Bug 908351

Summary: [Power Management]: Unable to edit power management fields, engine mistakenly thinks that host is operational.
Product: Red Hat Enterprise Virtualization Manager Reporter: Tareq Alayan <talayan>
Component: ovirt-engineAssignee: Eli Mesika <emesika>
Status: CLOSED CURRENTRELEASE QA Contact: Tareq Alayan <talayan>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.2.0CC: acathrow, dyasny, iheim, lpeer, oramraz, pstehlik, Rhev-m-bugs, sgrinber, yeylon, ykaul, yzaslavs
Target Milestone: ---   
Target Release: 3.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: infra
Fixed In Version: sf8 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-06-11 09:13:22 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 915537    
Attachments:
Description Flags
screenshot
none
engine log none

Description Tareq Alayan 2013-02-06 13:43:36 UTC
Description of problem:
Unable to edit power management fields, engine mistakenly thinks that host is operational. 

Version-Release number of selected component (if applicable):
sf5

How reproducible:
always

Steps to Reproduce:
1. Add host to engine. 
2. configure primary and secondary power managements
3. put host to maintenance
4. drop secondary power managements values
5. click ok - 
  
Actual results:
Error message saying that parameters couldn't be modified while host is operational. 
the host is in maintenance see screen shot 

Expected results:
parameters modified correctly

Additional info:
2013-02-06 15:23:24,245 ERROR [org.ovirt.engine.core.utils.ObjectIdentityChecker] (ajp-/127.0.0.1:8702-2) [6c440242] Field pmSecondaryuser can not be updated when status is Up
2013-02-06 15:23:24,245 WARN  [org.ovirt.engine.core.utils.ObjectIdentityChecker] (ajp-/127.0.0.1:8702-2) [6c440242] ObjectIdentityChecker.IsUpdateValid:: Not updatable field 'pmSecondaryuse
r' was updated
2013-02-06 15:23:24,245 WARN  [org.ovirt.engine.core.bll.UpdateVdsCommand] (ajp-/127.0.0.1:8702-2) [6c440242] CanDoAction of action UpdateVds failed. Reasons:VAR__ACTION__UPDATE,VAR__TYPE__H
OST,VDS_STATUS_NOT_VALID_FOR_UPDATE

Comment 1 Tareq Alayan 2013-02-06 13:44:10 UTC
Created attachment 693939 [details]
screenshot

Comment 2 Tareq Alayan 2013-02-06 13:45:24 UTC
Created attachment 693940 [details]
engine log

Comment 3 Eli Mesika 2013-02-12 12:22:24 UTC
fixed in commit : 30b6f9d

Comment 5 Oded Ramraz 2013-03-03 19:51:21 UTC
Verified sf9 , managed to remove secondary PM address from UI.

Comment 6 Itamar Heim 2013-06-11 09:13:22 UTC
3.2 has been released

Comment 7 Itamar Heim 2013-06-11 09:39:42 UTC
3.2 has been released