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

Added csm delete validation steps in E2E #839

Merged
merged 13 commits into from
Jan 2, 2025
Merged

Added csm delete validation steps in E2E #839

merged 13 commits into from
Jan 2, 2025

Conversation

mgandharva
Copy link
Contributor

@mgandharva mgandharva commented Dec 27, 2024

Description

Added E2E validation step to verify if the CSM got deleted after the deletion step.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#1655

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
  • 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
  • I have executed the relevant end-to-end test scenarios

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Install PowerScale Driver(Standalone)
  • Install PowerScale Driver(With Replication)
  • Install PowerScale Driver(Standalone), Enable/Disable Replication
  • Uninstall PowerScale Driver
  • Install PowerScale Driver(With Observability)
  • Install PowerMax Driver(Standalone)
  • Install PowerScale Driver (With Authorization V2)
  • Install Powermax Driver (With Authorization v2)
  • Install PowerStore Driver(Standalone), Enable Resiliency

Sorry, something went wrong.

ashleyvjoy
ashleyvjoy previously approved these changes Jan 1, 2025
@mgandharva mgandharva requested a review from ashleyvjoy January 1, 2025 07:03
karthikk92
karthikk92 previously approved these changes Jan 1, 2025
tests/e2e/testfiles/minimal-testfiles/scenarios.yaml Outdated Show resolved Hide resolved
ashleyvjoy
ashleyvjoy previously approved these changes Jan 1, 2025
@mgandharva mgandharva requested a review from harishp8889 January 2, 2025 09:41
@mgandharva mgandharva merged commit 80b88ce into main Jan 2, 2025
7 checks passed
@mgandharva mgandharva deleted the add-e2e-tests branch January 2, 2025 09:43
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.

None yet

5 participants