Skip to content
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

432 Replication APIs to be moved from alpha phase to v1 #75

Merged
merged 6 commits into from
Feb 8, 2023

Conversation

santhoshatdell
Copy link
Contributor

@santhoshatdell santhoshatdell commented Feb 7, 2023

Description

Replication APIs are updated from v1alpha1 to v1. Further changes related to Init Container which migrates the existing RG's apiVersion & CRD's to v1 during upgrade, will be in the next PR (#76).

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#432

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate (81.8%)
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have maintained backward compatibility (via Init Container which will migrate RG & CRD versions during upgrade. With that, existing RGs continue to work).

How Has This Been Tested?

• Installed CSM 1.5 replication module with isilon driver, and provisioned few PV/RGs.
• Upgraded isilon & replicator sidecar images to latest on both k8s clusters.
   cd dell-csi-helm-installer && ./csi-install.sh --namespace isilon --values /home/santhosh/my-samples/csm-replication/isilon-values.yaml --upgrade
• Upgraded replication-controller images.
   ./repctl create -f ../deploy/controller.yaml
• Verified RGs and CRD for v1 version with init container logs and kubectl on both clusters.
• Deleted target PowerScale SyncIQ policies.
• Restarted pod to confirm no migration was run second time.
• Provisioned new RG and ran replication actions on existing & new RGs.

@santhoshatdell santhoshatdell merged commit 49fea26 into main Feb 8, 2023
@santhoshatdell santhoshatdell deleted the 432-alpha-api branch February 8, 2023 23:23
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.

5 participants