canonical
Storage volume/instance snapshot edit restriction
Checklist
- [x] I have read the contributing guidelines and attest that all commits in this PR are signed off, cryptographically signed, and follow this project's commit structure.
- [x] I have checked and added or updated relevant documentation.
What Does This PR Do?
When editing a Storage Volume snapshot, modifications to writable fields are saved successfully. However, changes to read-only fields are simply ignored without notification, which can be confusing for users, as noted in issue #14423.
The same situation applies to instance snapshots. This PR ensures that if a snapshot is present within the two edit commands related to this situation, input validation is performed, and if a non-editable property is modified, an error is returned to the user.
Fixes #14423
the operation silently fails without providing any feedback to the user.
It doesn't silently fail, just the read-only fields are not modified, but the writable ones are still persisted.
@tugbataluy the tests are failing and this needs a rebase to be mergeable as other conflicting PRs have been merged in the interim.
You can open a draft PR, check the tests pass, before promoting to "reviewable".