-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pb-5019: Added logic to fetch the default encryption type for FBDA, if #325
Conversation
License Evaluation Results:
Total License Issues: 19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
it is not set in the kdmp-config cm. - frst case, if the exclude file list key is empty in the kdmp-config cm, we will try to set the default exclude file list for FBDA, if the storage class is FBDA. - second case, if the exclude file list key is not empty in the kdmp-cpnfig cm, if it did not had rule for FBDA storage class, we will try to set the degault execlude file list for the FBD, if the storage class is FBDA.
License Evaluation Results:
Total License Issues: 19 |
9e4bb4a
to
724ad49
Compare
License Evaluation Results:
Total License Issues: 19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The assumption made should be documented.
What this PR does / why we need it:
Which issue(s) this PR fixes (optional)
Closes # pb-5019
Special notes for your reviewer:
Testing:
Tested below two cases:
Note: Need to vendor it to stork repo.