Bug 1464795

Summary: Cold Merge: Add an entry describing merge job to audit log
Product: [oVirt] ovirt-engine Reporter: Ala Hino <ahino>
Component: BLL.StorageAssignee: Ala Hino <ahino>
Status: CLOSED CURRENTRELEASE QA Contact: Kevin Alon Goldblatt <kgoldbla>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 4.1.2CC: amureini, bugs, gwatson, stirabos, tnisan
Target Milestone: ovirt-4.1.4Flags: rule-engine: ovirt-4.1+
Target Release: 4.1.4   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-07-28 14:18:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Ala Hino 2017-06-25 20:48:04 UTC
Description of problem:
In the fix of BZ 1420401, we showed the cold merge info (base and top volume names and the host running the merge job) in the UI in the tasks panel (and obviously in engine log file).
This bug is to address a request to add same info to audit log.

How reproducible:
100%

Steps to Reproduce:
Run cold merge

Actual results:
No entry added to audit log (audit_log table in the database)

Expected results:
An entry with merge job info to be added to audit log table. Following is a sample message:

"
Merging snapshots (vm03-s1 into vm03-s2) of disk vm03_Disk1 on host rich-el73-host01
"

Comment 1 Red Hat Bugzilla Rules Engine 2017-06-25 20:49:22 UTC
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.

Comment 2 Kevin Alon Goldblatt 2017-07-11 13:35:37 UTC
Verified with the following code:
---------------------------------------
ovirt-engine-4.1.4-0.2.el7.noarch
rhevm-4.1.4-0.2.el7.noarch
vdsm-4.19.21-1.el7ev.x86_64


Verified with the following scenario:
---------------------------------------
1. Create Vm with disk
2. Create 3 snapshots snapt1, snapt2, snapt3
3. While vm is down perform cold merge by deleting snapshot snapt2 >>>>> An entry is added to the audit log as follows:

Merging snapshots (snapt3 into snapt2) of disk GlanceDisk-d8282c2 on host purple-vds1.qa.lab.tlv.redhat.com


Moving to VERIFIED!