Bug 2096923 - [Hybrid Backup] Backup succeed but with no clear message when creating a snapshot during backup finalization
Summary: [Hybrid Backup] Backup succeed but with no clear message when creating a sna...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.5.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ovirt-4.5.2
: ---
Assignee: Benny Zlotnik
QA Contact: Ilia Markelov
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-06-14 14:58 UTC by Evelina Shames
Modified: 2022-08-30 08:49 UTC (History)
4 users (show)

Fixed In Version: ovirt-engine-4.5.2.2
Clone Of:
Environment:
Last Closed: 2022-08-18 13:23:25 UTC
oVirt Team: Storage
Embargoed:
pm-rhel: ovirt-4.5?


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github oVirt ovirt-engine pull 467 0 None Merged core: produce successful event if backup finished successfully 2022-08-16 08:09:49 UTC
Red Hat Issue Tracker RHV-46422 0 None None None 2022-06-14 15:35:42 UTC

Description Evelina Shames 2022-06-14 14:58:21 UTC
Description of problem:
further to bug 2080766, when creating a snapshot during backup finalization it causes the auto-generated snapshot not to be removed with no clear message.

The message we see in the UI is that there was some possible failure during the snapshot removal:
"Possible failure while removing a snapshot that was generated automatically during backup of VM 'golden_env_mixed_virtio_2_0'. The snapshot may be removed manually"

and that the backup failed:
"Backup c64ddaec-0d1f-45e0-9269-91e805db84d2 for VM golden_env_mixed_virtio_2_0 failed"

Version-Release number of selected component (if applicable):
engine-4.5.1-0.62.el8ev

How reproducible:
always.

Steps to Reproduce:
1. Create a VM from rhel8.6 template and run it
2. Run backup_vm.py SDK example
3. When it starts finalizing the backup, create a live snapshot

Actual results:
The backup operation succeeds but the backup command fails:
SDK script finishes successfully, engine throws an error that backup failed.

Expected results:
We should see a clear message on the UI that the backup operation succeeded and the problem is only with removing the auto-generated snapshot.

Additional info:

Comment 1 Arik 2022-06-14 15:43:48 UTC
Evelina, you have got the following message:
"Possible failure while removing a snapshot that was generated automatically during backup of VM 'golden_env_mixed_virtio_2_0'. The snapshot may be removed manually"
didn't it appear in the UI?

Comment 2 Michal Skrivanek 2022-06-15 07:30:39 UTC
I would let go of "possible" - we did detect an error in removing snapshot, so it's not "possible" - it happened.
Perhaps also change the "may" to "can", as to me "may" implies that the system might do that automatically which is not the case. Or "You may remove...."

Comment 3 Arik 2022-06-15 09:02:58 UTC
(In reply to Michal Skrivanek from comment #2)
> I would let go of "possible" - we did detect an error in removing snapshot,
> so it's not "possible" - it happened.
> Perhaps also change the "may" to "can", as to me "may" implies that the
> system might do that automatically which is not the case. Or "You may
> remove...."

The reason for phrasing it that way was that we cannot say for sure what happened to the volumes (unless we check them) - we know that we didn't get an indication of a successful removal but it may be that the operation succeeded and we didn't get the task in succeed status so we wanted to be more careful with the message but the likelihood of such a case is pretty low, we can change it if you think it's important

the more important part of this issue (now reflected in the title) is that the backup command appears to fail.
we thought of two ways to handle that:
1. to wait for an on going operation to finish before initiating remove-snapshot
2. to generate an audit log that indicates the backup has succeeded (or no audit log at all) in case we just failed to remove the auto generated snapshot for the backup

Comment 4 Arik 2022-08-16 08:09:50 UTC
(In reply to Arik from comment #3)
> 2. to generate an audit log that indicates the backup has succeeded (or no
> audit log at all) in case we just failed to remove the auto generated
> snapshot for the backup

The option above was chosen, the fix is included in 4.5.2 already

Comment 5 Lukas Svaty 2022-08-18 13:23:25 UTC
This bug has low overall severity and passed an automated regression suite, and is not going to be further verified by QE. If you believe special care is required, feel free to re-open to ON_QA status.

Comment 6 Sandro Bonazzola 2022-08-30 08:49:07 UTC
This bugzilla is included in oVirt 4.5.2 release, published on August 10th 2022.
Since the problem described in this bug report should be resolved in oVirt 4.5.2 release, it has been closed with a resolution of CURRENT RELEASE.
If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.