Bug 1506268

Summary:	Maintenance tag should be shown in Host table during provision
Product:	Red Hat CloudForms Management Engine	Reporter:	Leo Khomenko <lkhomenk>
Component:	UI - OPS	Assignee:	Drew Bomhof <dbomhof>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Dave Johnson <dajohnso>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.9.0	CC:	dbomhof, gmccullo, hkataria, jhardy, jocarter, mpovolny, obarenbo
Target Milestone:	GA	Keywords:	FutureFeature, TestOnly
Target Release:	5.10.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	host:provision
Fixed In Version:	5.10.0.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1514116 (view as bug list)		Environment:
Last Closed:	2018-06-21 20:23:26 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1514116

Description Leo Khomenko 2017-10-25 14:09:35 UTC

Description of problem:as per https://bugzilla.redhat.com/show_bug.cgi?id=1454880 
Maintenance State should be reflected during provision. as it's possible to select such Host and get faulty provision


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

How reproducible:100%


Steps to Reproduce:
1.Put Host into Maintenance Mode
2.Provision VM using this Host


Actual results:Provision failure


Expected results: Warning Massage or at least UI change that would let user know about host state

Comment 3 Greg McCullough 2017-11-13 17:20:32 UTC

Drew - We can pair on this, I think it will end up being a small change on the model and UI side.

Comment 4 CFME Bot 2017-11-13 20:46:57 UTC

https://github.com/ManageIQ/manageiq/pull/16464

Comment 5 CFME Bot 2017-11-13 20:56:24 UTC

https://github.com/ManageIQ/manageiq-ui-classic/pull/2717

Comment 6 CFME Bot 2017-11-14 16:17:01 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/a1893e955dc58bbf96f1a1a6558a6153490c3f85

commit a1893e955dc58bbf96f1a1a6558a6153490c3f85
Author:     Drew Bomhof <dbomhof>
AuthorDate: Mon Nov 13 15:26:30 2017 -0500
Commit:     Drew Bomhof <dbomhof>
CommitDate: Mon Nov 13 15:26:30 2017 -0500

    Added a Maintenance key to the hash struct
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1506268

 app/models/miq_request_workflow.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 7 CFME Bot 2017-11-14 16:24:13 UTC

New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/553b8603d0a022945852172550f9fb8ea5df9a20

commit 553b8603d0a022945852172550f9fb8ea5df9a20
Author:     Drew Bomhof <dbomhof>
AuthorDate: Mon Nov 13 15:32:39 2017 -0500
Commit:     Drew Bomhof <dbomhof>
CommitDate: Mon Nov 13 15:32:39 2017 -0500

    Allow the Maintenance mode to be displayed
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1506268

 app/controllers/application_controller/miq_request_methods.rb | 1 +
 app/views/miq_request/_prov_host_grid.html.haml               | 1 +
 2 files changed, 2 insertions(+)

Comment 9 Greg McCullough 2017-11-16 15:17:00 UTC

Adding cfme‑5.9 flag