-
Notifications
You must be signed in to change notification settings - Fork 184
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
Sends VolumeGroupSnapshotClass
to client
#2661
Conversation
4bebb4c
to
652e5f1
Compare
/cc @rewantsoni |
/cc @Madhu-1 |
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.
Take a look at Jira, this PR is standalone right even if we use or not use the feature, like, creating resource in the client side will not have any side-effects & dependencies?
/hold |
/unhold for a sec I thought ocs-client-op repo and put a hold. |
/retest |
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
/lgtm |
@raaizik can you please check why the e2e test failed? |
Signed-off-by: raaizik <[email protected]>
/test ocs-operator-bundle-e2e-aws |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: leelavg, Madhu-1, nb-ohad, raaizik The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
fbe323d
into
red-hat-storage:main
Sends
VolumeGroupSnapshotClass
as an external resource to the client via the storage claim response.Must be merged prior to ocs-client-operator/pull/168.
RHSTOR-5795