Bug 1508869 - HA VM hotplug lease double meaning message
Summary: HA VM hotplug lease double meaning message
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.2.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ovirt-4.1.9
: ---
Assignee: Eyal Shenitzky
QA Contact: Avihai
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-11-02 12:25 UTC by Liran Rotenberg
Modified: 2018-01-24 10:41 UTC (History)
4 users (show)

Fixed In Version: ovirt-engine-4.1.9
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-01-24 10:41:08 UTC
oVirt Team: Storage
Embargoed:
rule-engine: ovirt-4.1+


Attachments (Terms of Use)
engine log (5.46 MB, text/plain)
2017-11-02 12:25 UTC, Liran Rotenberg
no flags Details
edit VM to be HA (71.44 KB, image/png)
2017-11-27 13:56 UTC, Eyal Shenitzky
no flags Details
edit VM to be HA confirmation window (69.92 KB, image/png)
2017-11-27 13:57 UTC, Eyal Shenitzky
no flags Details
edit VM to be HA 4.1 (61.50 KB, image/png)
2017-11-30 06:21 UTC, Eyal Shenitzky
no flags Details
edit VM to be HA confirmation window 4.1 (66.73 KB, image/png)
2017-11-30 06:22 UTC, Eyal Shenitzky
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 84998 0 ovirt-engine-4.1 MERGED engine: Display VM Lease changes only as Hot Plugged 2020-03-30 21:10:57 UTC

Description Liran Rotenberg 2017-11-02 12:25:48 UTC
Created attachment 1347012 [details]
engine log

Description of problem:
When trying to enable\disable HA VM lease(hotplug) a message is shown:

Changes that can be applied immediately:
VM Lease
-apply later option
Changes that require Virtual Machine restart:
VM Lease

Version-Release number of selected component (if applicable):
4.2.0-0.0.master.20171025204923.git6f4cbc5.el7.centos

How reproducible:
Always

Steps to Reproduce:
1. Create a VM and run it.
2. Enable or disable the HA option, with lease.

Actual results:
A window message is shown with double meaning.

Expected results:
Only the next message(or other appropriate):
VM Lease
-apply later option

Additional info:
engine log attached

Comment 1 Eyal Shenitzky 2017-11-27 13:56:29 UTC
Created attachment 1359472 [details]
edit VM to be HA

Comment 2 Eyal Shenitzky 2017-11-27 13:57:25 UTC
Created attachment 1359473 [details]
edit VM to be HA confirmation window

Comment 3 Eyal Shenitzky 2017-11-27 13:58:06 UTC
Hi Liran,

It seems like the issue fixed in master 4.2 from commit - 97ca6b571b34c07eb2f6d4686b4e106402c95882.

I've attached two screenshots:
1. Change the VM to be HA.
2. Following confirmation window

You can see that the 'Changes that require Virtual Machine restart:' was updated to include the 'Resume Behavior' updates instead of the double meaning - VM Lease.

Please close the bug in case it solved the issue.

Comment 4 Liran Rotenberg 2017-11-29 11:32:53 UTC
Verified on:
Software version:4.2.0-0.5.master.el7

The issue solved and can be seen in Eyal attachments.

Comment 5 Eyal Shenitzky 2017-11-30 05:14:09 UTC
Moving back to post until the relevant patch from 4.2 will backport to 4.1.

Comment 6 Eyal Shenitzky 2017-11-30 06:21:48 UTC
Created attachment 1360684 [details]
edit VM to be HA 4.1

Comment 7 Eyal Shenitzky 2017-11-30 06:22:17 UTC
Created attachment 1360685 [details]
edit VM to be HA confirmation window 4.1

Comment 8 Eyal Shenitzky 2017-11-30 06:22:40 UTC
Seems like the patch that responsible for the fix already found on 4.1 branch - 
commit - 8a9fe89c15cbfbf3914e6bc505d229943b1067c5

Screenshots of 4.1 were added.

Liran, 
Sorry about this but please verify (this time on 4.1) again.

Comment 9 Eyal Shenitzky 2017-12-03 09:38:06 UTC
This bug is a duplication of - https://bugzilla.redhat.com/show_bug.cgi?id=1492471 which target to 4.2 and already verified.

Keep this bug open and backport the fix to 4.1.

Comment 10 Avihai 2018-01-16 09:38:41 UTC
verified on 4.1.9-0.2.el7

Comment 11 Sandro Bonazzola 2018-01-24 10:41:08 UTC
This bugzilla is included in oVirt 4.1.9 release, published on Jan 24th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.1.9 release, published on Jan 24th 2018, 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.