Bug 1302271 - Provisioning request approval - incorrect flash message text
Summary: Provisioning request approval - incorrect flash message text
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.5.0
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: GA
: 5.6.0
Assignee: Roman Blanco
QA Contact: Jan Krocil
URL:
Whiteboard: flash_msg:service
: 1305733 1313521 (view as bug list)
Depends On:
Blocks: 1305954
TreeView+ depends on / blocked
 
Reported: 2016-01-27 10:46 UTC by Jan Krocil
Modified: 2016-06-29 15:33 UTC (History)
7 users (show)

Fixed In Version: 5.6.0.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1305954 (view as bug list)
Environment:
Last Closed: 2016-06-29 15:33:46 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:1348 0 normal SHIPPED_LIVE CFME 5.6.0 bug fixes and enhancement update 2016-06-29 18:50:04 UTC

Description Jan Krocil 2016-01-27 10:46:35 UTC
Description of problem:
When a request for configured system reprovision/reconfiguration is approved, an incorrect flash message is displayed stating that the request has been denied.
The request is approved and works fine but the test of the flash message is wrong.

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

How reproducible:
Always

Steps to Reproduce:
1. Add a configuration manager
2. Reprovision a configured system
3. Apporove the request
4. Check the flash message

Actual results:
'Request "Red Hat Satellite install on [<hostname>]" was denied'

Expected results:
'Request "Red Hat Satellite install on [<hostname>]" was approved'

Additional info:

Comment 2 Shveta 2016-01-27 19:42:55 UTC
Assigning to add test case

Comment 4 CFME Bot 2016-02-01 17:40:58 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/0b1fbf4e62fffc643bc7b2aed5ce954a53dd8136

commit 0b1fbf4e62fffc643bc7b2aed5ce954a53dd8136
Author:     Roman Blanco <rblanco>
AuthorDate: Mon Feb 1 14:37:31 2016 +0100
Commit:     Roman Blanco <rblanco>
CommitDate: Mon Feb 1 14:37:31 2016 +0100

    Corrected condition for request approval flash message
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1302271

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

Comment 5 Dan Clarizio 2016-02-09 14:45:42 UTC
*** Bug 1305733 has been marked as a duplicate of this bug. ***

Comment 8 CFME Bot 2016-02-15 15:38:23 UTC
Detected commit referencing this ticket while ticket status is MODIFIED.

Comment 9 CFME Bot 2016-02-29 23:58:09 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=077716129c9d37f4fe4c20846c1252d1d216b983

commit 077716129c9d37f4fe4c20846c1252d1d216b983
Author:     Roman Blanco <rblanco>
AuthorDate: Mon Feb 1 14:37:31 2016 +0100
Commit:     Roman Blanco <rblanco>
CommitDate: Mon Feb 29 17:13:53 2016 +0100

    Corrected condition for request approval flash message
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1302271

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

Comment 10 CFME Bot 2016-02-29 23:58:23 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=f6db2d6feac988d8b833f7b19ceb544cf2f89f2e

commit f6db2d6feac988d8b833f7b19ceb544cf2f89f2e
Merge: adddc6a 0777161
Author:     Dan Clarizio <dclarizi>
AuthorDate: Mon Feb 29 18:40:24 2016 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Mon Feb 29 18:40:24 2016 -0500

    Merge branch 'bz1305954' into '5.5.z'
    
    Corrected condition for request approval flash message
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1302271
    
    clean cherry-pick
    
    See merge request !808

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

Comment 11 Harpreet Kataria 2016-03-01 19:04:12 UTC
*** Bug 1313521 has been marked as a duplicate of this bug. ***

Comment 12 Jan Krocil 2016-04-19 10:53:22 UTC
Verified fixed in 5.6-beta2 - 5.6.0.1-beta2.20160413141124_e25ac0e.

Comment 14 errata-xmlrpc 2016-06-29 15:33:46 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/RHBA-2016:1348


Note You need to log in before you can comment on or make changes to this bug.