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

Fixed DeleteStorageProtectionGroup function response #407

Merged
merged 4 commits into from
Jan 10, 2025

Conversation

chaganti-rajitha
Copy link
Contributor

@chaganti-rajitha chaganti-rajitha commented Jan 10, 2025

Description

When we delete rg from cluster it is getting stuck in deleting state because of mismatch in the response from error message so added the correct string now in DeleteStorageProtectionGroup function.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#1669

Checklist:

  • Have you run format,vet & lint checks against your submission?
  • Have you made sure that the code compiles?
  • Did you run the unit & integration tests successfully?
  • Have you maintained at least 90% code coverage?
  • Have you commented your code, particularly in hard-to-understand areas
  • Have you done corresponding changes to the documentation
  • Did you run tests in a real Kubernetes cluster?
  • Backward compatibility is not broken

How Has This Been Tested?

We built the image with code changes and used it to test delete rg scenario in ocp cluster environment.
image
image
image
image

@chaganti-rajitha chaganti-rajitha changed the title fixed UT ECS01D-1: Fixed DeleteStorageProtectionGroup function response Jan 10, 2025
@chaganti-rajitha chaganti-rajitha marked this pull request as ready for review January 10, 2025 07:42
@chaganti-rajitha chaganti-rajitha changed the title ECS01D-1: Fixed DeleteStorageProtectionGroup function response Fixed DeleteStorageProtectionGroup function response Jan 10, 2025
alankar-verma
alankar-verma previously approved these changes Jan 10, 2025
satyakonduri
satyakonduri previously approved these changes Jan 10, 2025
@adarsh-dell
Copy link

adarsh-dell commented Jan 10, 2025

Checks are failing, please fix them.
image

@chaganti-rajitha
Copy link
Contributor Author

Checks are failing, please fix them. image

Fixed

@adarsh-dell
Copy link

@chaganti-rajitha Any associated github issue for this PR?

@chaganti-rajitha
Copy link
Contributor Author

@chaganti-rajitha Any associated github issue for this PR?

There is no existing issue, created now.

@chaganti-rajitha chaganti-rajitha merged commit c72952e into main Jan 10, 2025
6 checks passed
@chaganti-rajitha chaganti-rajitha deleted the usr/rajitha/ECS01D-1-fix branch January 10, 2025 10:59
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.

6 participants