-
Notifications
You must be signed in to change notification settings - Fork 61
Refactor labelling of CG PVCs based on PeerClass #2100
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
base: main
Are you sure you want to change the base?
Conversation
9b7ba28
to
843fa5d
Compare
@ELENAGER please rebase to fix the CI errors. |
f1d1d1d
to
3c9abd0
Compare
…roupreplications CRD Signed-off-by: Elena Gershkovich <elenage@il.ibm.com>
…ly volumegroupsnapshots CRD Signed-off-by: Elena Gershkovich <elenage@il.ibm.com>
…Rep PVC lists Signed-off-by: Elena Gershkovich <elenage@il.ibm.com>
Signed-off-by: Elena Gershkovich <elenage@il.ibm.com>
return annotations[IsCGEnabledAnnotation] == "true" | ||
// It checks whether the VolumeGroupReplication CRD is installed. | ||
// This condition must be true for VolRep CG protection to be considered enabled. | ||
func IsCGEnabled(ctx context.Context, apiReader client.Reader) bool { |
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.
Maybe now is the time to make this function specific to VGR and change the name to IsCGEnabledForVolRep
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
No description provided.