8000 Refactor labelling of CG PVCs based on PeerClass by ELENAGER · Pull Request #2100 · RamenDR/ramen · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

ELENAGER
Copy link
Member

No description provided.

@ELENAGER ELENAGER requested a review from rakeshgm June 19, 2025 08:04
@ELENAGER ELENAGER force-pushed the enable_cg branch 6 times, most recently from 9b7ba28 to 843fa5d Compare June 23, 2025 12:15
@nirs
Copy link
Member
nirs commented Jun 23, 2025

@ELENAGER please rebase to fix the CI errors.

@ELENAGER ELENAGER force-pushed the enable_cg branch 3 times, most recently from f1d1d1d to 3c9abd0 Compare June 25, 2025 13:00
ELENAGER added 4 commits June 29, 2025 13:00
…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 {
Copy link
Member

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

Copy link
Member
@BenamarMk BenamarMk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0