Bug 1220882 - Orchestration Configuration button missing
Summary: Orchestration Configuration button missing
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.4.0
Assignee: Milan Zázrivec
QA Contact: Shveta
URL:
Whiteboard:
: 1222838 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-05-12 18:02 UTC by Shveta
Modified: 2015-06-16 13:02 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-06-16 13:02:13 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
error (132.99 KB, image/png)
2015-05-12 18:02 UTC, Shveta
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2015:1100 0 normal SHIPPED_LIVE CFME 5.4.0 bug fixes, and enhancement update 2015-06-16 16:28:42 UTC

Description Shveta 2015-05-12 18:02:17 UTC
Created attachment 1024714 [details]
error

Description of problem:


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


How reproducible:


Steps to Reproduce:
1. Add a cloud provider and go to Catalogs - Orchestration template
2. Configuration bar and create service dialog options are missing from the template
3.

Actual results:


Expected results:


Additional info:
This feature is present in 5.4 builds

===========================
master.20150512135457_7caaea5

Comment 2 Shveta 2015-05-12 18:18:59 UTC
Just checked in latest 5.4 build .
Not present in 5.4.0.0.26.20150511144816_f924bd4  also .

Comment 3 Shveta 2015-05-12 21:10:35 UTC
Configuration button also missing from Catalog and Catalog item page

Comment 4 Milan Zázrivec 2015-05-14 14:44:21 UTC
https://github.com/ManageIQ/manageiq/pull/2917

Comment 5 CFME Bot 2015-05-14 15:16:05 UTC
New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/919f81b94b6773bff9743acf6f957320c855e79a

commit 919f81b94b6773bff9743acf6f957320c855e79a
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Thu May 14 11:58:14 2015 +0200
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Thu May 14 16:31:05 2015 +0200

    Revert "Added curly braces in the if block."
    
    This reverts commit c82b2de54e904aa478521da9713be9f45f018cec.
    
    We need to properly initialize all required toolbars.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220882

 vmdb/app/helpers/application_helper.rb | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

Comment 6 CFME Bot 2015-05-14 15:16:09 UTC
New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/9a16ac670c0d7c9378d17b22b8b74bcca78c14e4

commit 9a16ac670c0d7c9378d17b22b8b74bcca78c14e4
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Thu May 14 16:33:41 2015 +0200
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Thu May 14 16:38:16 2015 +0200

    Create parent div for toolbar to be created, in case it doesn't exist
    
    This fix is to make sure that the subsequent
    
        new dhtmlXToolbarObject
    
    succeeds. Without the parent object in place,
    the dhtmlXToolbarObject constructor would fail.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220882

 vmdb/app/helpers/application_helper.rb | 6 ++++++
 1 file changed, 6 insertions(+)

Comment 8 John Prause 2015-05-14 18:37:36 UTC
This will need to be cherry-pick to 5.4.z since the fix was made after the 5.4.z branch was created yesterday.

Comment 9 CFME Bot 2015-05-14 20:00:56 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c07e97e26fda987738a457bfac4c06e6b1c0ce49

commit c07e97e26fda987738a457bfac4c06e6b1c0ce49
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Thu May 14 11:58:14 2015 +0200
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Thu May 14 17:42:21 2015 +0200

    Revert "Added curly braces in the if block."
    
    This reverts commit c82b2de54e904aa478521da9713be9f45f018cec.
    
    We need to properly initialize all required toolbars.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220882
    (cherry picked from commit 919f81b94b6773bff9743acf6f957320c855e79a)

 vmdb/app/helpers/application_helper.rb | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

Comment 10 CFME Bot 2015-05-14 20:00:59 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=a6ede013c3698466734494be0ede48146662f513

commit a6ede013c3698466734494be0ede48146662f513
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Thu May 14 16:33:41 2015 +0200
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Thu May 14 17:42:27 2015 +0200

    Create parent div for toolbar to be created, in case it doesn't exist
    
    This fix is to make sure that the subsequent
    
        new dhtmlXToolbarObject
    
    succeeds. Without the parent object in place,
    the dhtmlXToolbarObject constructor would fail.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220882
    (cherry picked from commit 9a16ac670c0d7c9378d17b22b8b74bcca78c14e4)

 vmdb/app/helpers/application_helper.rb | 6 ++++++
 1 file changed, 6 insertions(+)

Comment 11 CFME Bot 2015-05-14 20:01:03 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=29c6c3126bd1c4d3a5876a91fa93b79dcd18546c

commit 29c6c3126bd1c4d3a5876a91fa93b79dcd18546c
Merge: d37f04a a6ede01
Author:     Dan Clarizio <dclarizi>
AuthorDate: Thu May 14 11:51:45 2015 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Thu May 14 11:51:45 2015 -0400

    Merge branch 'fix_orch_templates_toolbars' into '5.4.z'
    
    Fix orchestration templates toolbars
    
    The first commit is to make sure the our toolbars in UI are created correctly
    (orch. templates toolbar for example).
    
    The second commit is to make sure the parent for the dhtmlx toolbar
    to be created is in place (i.e. we create it, if it doesn't exist), otherwise
    the toolbar creation would fail.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220882
    
    See merge request !56

 vmdb/app/helpers/application_helper.rb | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

Comment 13 Dave Johnson 2015-05-19 18:34:16 UTC
*** Bug 1222838 has been marked as a duplicate of this bug. ***

Comment 14 Shveta 2015-05-27 18:25:59 UTC
Fixed in 
5.4.0.2.20150519003830_673f996

Comment 16 errata-xmlrpc 2015-06-16 13:02:13 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://rhn.redhat.com/errata/RHBA-2015-1100.html


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