Bug 1338834

Summary: Option to override cluster migration policy in vm level is absent
Product: [oVirt] ovirt-engine Reporter: sefi litmanovich <slitmano>
Component: BLL.VirtAssignee: Alexander Wels <awels>
Status: CLOSED CURRENTRELEASE QA Contact: sefi litmanovich <slitmano>
Severity: high Docs Contact:
Priority: high    
Version: 4.0.0CC: awels, bugs, gklein, michal.skrivanek, tjelinek
Target Milestone: ovirt-4.0.0-rcFlags: rule-engine: ovirt-4.0.0+
rule-engine: blocker+
rule-engine: planning_ack+
tjelinek: devel_ack+
rule-engine: testing_ack+
Target Release: ---   
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: 2016-07-05 08:04:35 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screenshot none

Description sefi litmanovich 2016-05-23 13:52:22 UTC
Created attachment 1160639 [details]
screenshot

Description of problem:

New cluster migration policies were introduced on ovirt-4.0 beta.
The plan was to include the option to override the policy in vm level - under 'host' sub tab in new/edit vm menu.
The option is not there.

Version-Release number of selected component (if applicable):
ovirt-engine-4.0.0-0.7.master.el7ev.noarch

How reproducible:
always

Steps to Reproduce:
1. set cluster migration policy (to whatever you like)
2. create/edit a vm in the cluster, look for migration policy option


Actual results:
no migration policy option exists in vm.

Expected results:
Policy is changeable from this menu

Comment 1 Tomas Jelinek 2016-05-23 14:38:27 UTC
It is a regression introduced by Ia95e6aba555e09d36643b091a9428b6edf366495 "userportal,webadmin: tab dialog scroll".

@Alex, could you please have a look? The migrationPolicyEditor, overrideMigrationPolicyEditorWithDetachable etc have disappeared from the dialog.

Comment 2 Alexander Wels 2016-05-23 14:59:00 UTC
Of course, can you show me what it should look like? I tried to make 100% sure I didn't mangle anything but obviously I did. I am happy to add stuff I broke back, but I am not sure what I broke.

Comment 3 Tomas Jelinek 2016-05-24 06:58:59 UTC
this section has disappeared:

<g:FlowPanel addStyleNames="{style.migrationFieldsPanel}">
<ge:EntityModelDetachableWidget ui:field="overrideMigrationPolicyEditorWithDetachable" addStyleNames="{style.overrideMigrationFields} avmpw_overrideMigrationFields_pfly_fix" contentWidgetContainerStyle="{style.overrideMigrationFieldsContentEditor}" />
<l:EnableableFormLabel ui:field="overrideMigrationPolicyLabel" text="{constants.overrideMigrationPolicyLabel}" addStyleNames="{style.migrationFieldsLabel} avmpw_migrationFieldsLabel_pfly_fix" />
d:InfoIcon ui:field="migrationPolicyInfoIcon" addStyleNames="{style.migrationFieldsInfoIcon} avmpw_infoIcon_pfly_fix" />
<e:ListModelListBoxOnlyEditor ui:field="migrationPolicyEditor" addStyleNames="{style.migrationFields}" />
</g:FlowPanel>

and the migrationPolicyInfoIcon has been placed to the overrideMigrationDowntimeLabel (instead of the migrationDowntimeInfoIcon)

Comment 4 Alexander Wels 2016-05-24 17:37:21 UTC
Added the appropriate fields back in. Sorry for the mistake.

Comment 5 Tomas Jelinek 2016-05-25 07:45:13 UTC
np, thank you for the quick response

Comment 6 sefi litmanovich 2016-06-05 10:39:16 UTC
Verified with ovirt-engine-4.0.0.2-0.1.el7ev.noarch, option exists on both UI and API.

Comment 7 Sandro Bonazzola 2016-07-05 08:04:35 UTC
oVirt 4.0.0 has been released, closing current release.