Skip to content

Update volume status received from IRI response #1342

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

Merged

Conversation

ushabelgur
Copy link
Contributor

Proposed Changes

  • Update volume status received from IRI response, instead of updating status only for volume list available in spec. In case of previous operation failures(like detach volume) will be missed in that case.
  • Add validation to check duplicate volume from status as well, in case of volume is removed from spec, but it has failed while detaching. So that user doesn't try to attach volume with same name again.

Fixes #1338 #982

@ushabelgur ushabelgur requested a review from a team as a code owner July 24, 2025 07:21
@github-actions github-actions bot added bug Something isn't working size/M labels Jul 24, 2025
@hardikdr hardikdr added the area/iaas Issues related to IronCore IaaS development. label Jul 25, 2025
@hardikdr hardikdr added this to Roadmap Jul 25, 2025
@Rohit-0505 Rohit-0505 marked this pull request as draft July 28, 2025 15:40
@Rohit-0505
Copy link
Contributor

Wait for this PR to get merged, after rebasing with the main, the tests should pass

@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/fix/machine_volume_status branch from 5716a3d to 37d905d Compare July 30, 2025 09:54
@ushabelgur ushabelgur marked this pull request as ready for review July 30, 2025 10:07
@opensovereigncloud-user opensovereigncloud-user force-pushed the osc/fix/machine_volume_status branch from 37d905d to 3e0e56e Compare August 13, 2025 09:49
@lukasfrank lukasfrank merged commit 3581ac0 into ironcore-dev:main Aug 14, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this to Done in Roadmap Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/iaas Issues related to IronCore IaaS development. bug Something isn't working size/M
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Machine Volume status reporting wrong status after detach volume fails
5 participants