Bug 1402995

Summary:	do not render service start/stop buttons (and status field?) if start and stop actions are missing
Product:	Red Hat CloudForms Management Engine	Reporter:	Satoe Imaishi <simaishi>
Component:	Automate	Assignee:	Drew Bomhof <dbomhof>
Status:	CLOSED ERRATA	QA Contact:	Leo Khomenko <lkhomenk>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.7.0	CC:	adahms, cpelland, dajohnso, dbomhof, gmccullo, hkataria, jhardy, lkhomenk, mkanoor, mpovolny, obarenbo, simaishi, sshveta, tfitzger
Target Milestone:	GA	Keywords:	Reopened, ZStream
Target Release:	5.7.2
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	ssui:power
Fixed In Version:	5.7.2.0	Doc Type:	Known Issue
Doc Text:	Currently, power operations performed through the Service User Interface work for services created from catalog bundles, but will not work for services created from a single catalog item.	Story Points:	---
Clone Of:	1394202	Environment:
Last Closed:	2017-04-12 14:33:48 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Embargoed:
Bug Depends On:	1394202
Bug Blocks:	1405055

Comment 6 Dave Johnson 2016-12-16 04:16:16 UTC

Leo, Satoe is right, if you are re-opening, please move it back to ON_DEV or ASSIGNED. 

Drew, in terms of the empty state, look at the screenshot attachment.  The is no power state value in the field.  Leo, please confirm that is what you mean.

Leo, please confirm happy path of all VMs in the same state, does the power button work correctly up and down (with a correct field value).  And then confirm if we have an issue when multiple VMs get into separate power state thereby confusing the app (which would most likely be a separate BZ).

Comment 8 Aparna Karve 2016-12-16 16:38:46 UTC

*** Bug 1405055 has been marked as a duplicate of this bug. ***

Comment 9 CFME Bot 2016-12-16 22:51:28 UTC

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

Comment 11 Drew Bomhof 2016-12-20 14:38:20 UTC

The PR from comment 9 is really a PR for the upstream BZ - https://bugzilla.redhat.com/show_bug.cgi?id=1394202

Comment 13 CFME Bot 2017-01-23 23:20:45 UTC

New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/a5486c586099a8312e5ac8dddd4969b3980e5b99

commit a5486c586099a8312e5ac8dddd4969b3980e5b99
Author:     Greg McCullough <gmccullo>
AuthorDate: Mon Jan 23 16:59:02 2017 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Mon Jan 23 18:15:34 2017 -0500

    Merge pull request #13232 from syncrou/assume_when_all_nil_actions
    
    Allow a service power state to correctly handle nil actions
    (cherry picked from commit 656bbc8e32f9b2d8a64d2e998350fb6f73b75a14)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1402995

 app/models/service.rb       | 34 +++++++++++++++++----
 spec/models/service_spec.rb | 74 +++++++++++++++++++++++++++++++++++++++------
 2 files changed, 92 insertions(+), 16 deletions(-)

Comment 18 Drew Bomhof 2017-01-26 15:08:53 UTC

It is the expected behavior - though - There is another PR - https://github.com/ManageIQ/manageiq-ui-service/pull/406 That once implemented (see: https://github.com/ManageIQ/manageiq-ui-service/pull/406#issuecomment-271744784) will speed up the power_status without requiring the extra reload (or navigation back into the service details page.)

Comment 19 Drew Bomhof 2017-02-06 16:59:07 UTC

Backport PR - https://github.com/ManageIQ/manageiq-ui-service/pull/493

Comment 23 errata-xmlrpc 2017-04-12 14:33:48 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2017:0898