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

Add metro e2e tests #356

Merged
merged 19 commits into from
Oct 15, 2024
Merged

Add metro e2e tests #356

merged 19 commits into from
Oct 15, 2024

Conversation

lukeatdell
Copy link
Contributor

@lukeatdell lukeatdell commented Oct 14, 2024

Description

Adding e2e tests for powerstore metro replication.
The test suit tests:

  • The metro volume is mounted with multiple device mount paths on the Kubernetes worker node for the provisioned csi persistent volume (PV).
  • Multiple host paths exist between the Kubernetes worker node and the PowerStore array for each side of the metro volume.
  • All host paths are in 'active ready' state.
  • The persistent volume is writable.
  • The persistent volume continues to be writable after taking down all host paths for one of the two PowerStore arrays in the metro replication session.

Additional info:

  • Updated kubernetes e2e test module versions and necessary implementations.
  • Relocated k8s e2e tests under csi-powerstore/test/e2e/k8s

GitHub Issues

GitHub Issue #
dell/csm#1443

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Backward compatibility is not broken

How Has This Been Tested?

  • Metro e2e test executed
    image

donatwork
donatwork previously approved these changes Oct 15, 2024
santhoshatdell
santhoshatdell previously approved these changes Oct 15, 2024
tests/e2e/driver/resources/sc.yaml Outdated Show resolved Hide resolved
tests/e2e/driver/resources/sc.yaml Outdated Show resolved Hide resolved
tests/e2e/driver/resources/deployment.yaml Outdated Show resolved Hide resolved
tests/e2e/driver/resources/deployment.yaml Outdated Show resolved Hide resolved
mbasha-dell
mbasha-dell previously approved these changes Oct 15, 2024
@santhoshatdell santhoshatdell dismissed stale reviews from mbasha-dell and themself via 84120a1 October 15, 2024 17:43
lukeatdell

This comment was marked as off-topic.

@lukeatdell lukeatdell merged commit eee3ec6 into main Oct 15, 2024
5 checks passed
@lukeatdell lukeatdell deleted the usr/lukeatdell/e2e-metro branch October 15, 2024 17:48
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