slavkap opened a new pull request #5544: URL: https://github.com/apache/cloudstack/pull/5544
### Description If a snapshot is taken only on Primary storage with the option "snapshot.backup.to.secondary" set to` true`, when you set this option to `false` the revert will fail. Added check if there isn't a snapshot on Secondary storage to check for it on Primary. ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] New feature (non-breaking change which adds functionality) - [X] Bug fix (non-breaking change which fixes an issue) - [ ] Enhancement (improves an existing feature and functionality) - [ ] Cleanup (Code refactoring and cleanup, that may add test cases) #### Bug Severity - [ ] BLOCKER - [ ] Critical - [ ] Major - [X] Minor - [ ] Trivial ### How Has This Been Tested? 4.15 and main with KVM hypervisors set `snapshot.backup.to.secondary` to `false` take one or more snapshots set `snapshot.backup.to.secondary` to `true` try to revert the snapshot. The operation will fail with "snapshot [snapshot ID] does not exists in data store" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
