-
Notifications
You must be signed in to change notification settings - Fork 70
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
OADP-3050: Remove unwanted BSL/VSLs on create/edit DPA #1525
base: master
Are you sure you want to change the base?
Conversation
@stillalearner: This pull request references OADP-3050 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.18.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
@stillalearner: This pull request references OADP-3050 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target the "4.18.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
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.
Have to double check but hopefully this doesn't break the workaround in
https://issues.redhat.com/browse/OADP-3630
21fd06f
to
5b7750d
Compare
I think it should not, as these changes only deal with the BSLs having all of the 3 labels. While the workaround is only adding one of those. What do you think @kaovilai ? |
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.
While the workaround is only adding one of those. What do you think @kaovilai ?
works for me
5b7750d
to
024548a
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: stillalearner The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: stillalearner The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
024548a
to
008db51
Compare
008db51
to
7aa3a88
Compare
@stillalearner: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Fixes the issue #1469 of unwanted BSL or VSLs in the cases:
hope that covers all cases.