Bug 1035501

Summary: Remove Add Graph to Monitor View Menu Item in Resource Tree
Product: [JBoss] JBoss Operations Network Reporter: Mike Thompson <mithomps>
Component: UIAssignee: Jirka Kremser <jkremser>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: medium Docs Contact:
Priority: unspecified    
Version: JON 3.2CC: hrupp, lkrejci, loleary
Target Milestone: DR01   
Target Release: JON 3.2.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-05-08 17:44:22 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: 983644    
Bug Blocks:    
Attachments:
Description Flags
the patch file none

Description Mike Thompson 2013-11-27 23:22:33 UTC
Description of problem:


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


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Mike Thompson 2013-11-27 23:28:06 UTC
Description of problem:
Remove the old 'Add Graph to Monitor View' menu item in the TreeView context menu as that is no longer valid with the new metrics graphs. This has been replaced with 'Add to Dashboard' button.

NOTE: this also applies to the group tree as well.


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


How reproducible:
always


Steps to Reproduce:
1. Goto a resource detail view
2. Right click on the Resource Tree to bring up the context menu

Actual results:
Note the menu item: Add Graph to Monitor View


Expected results:
It should be removed now.

Comment 2 Jirka Kremser 2013-11-28 11:16:09 UTC
Created attachment 830158 [details]
the patch file

Removing the old way of adding the charts from the portal war. It is not in master / release-branch because its target release is post 3.2

Comment 3 Jirka Kremser 2013-12-04 19:16:32 UTC
7355462e593ce38a8a7b88e3a0e68b4f9da2a561 in master => needs to be cherry-picked over once the 3.2 is out

Comment 6 Jay Shaughnessy 2014-01-10 18:36:25 UTC
*** Bug 1033230 has been marked as a duplicate of this bug. ***

Comment 7 Jirka Kremser 2014-01-20 13:04:29 UTC
fixing the same issue for the context menu for groups

branch:  master
link:    http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=75ec4b37b
time:    2014-01-20 13:59:57 +0100
commit:  75ec4b37b6408822220b756186cbaa6dae361d3d
author:  Jirka Kremser - jkremser
message: [BZ 1035501] - Remove Add Graph to Monitor View Menu Item in Resource
         Tree - removing the zombie code for groups as well (commit
         7355462e593 solved the issue only for resource tree). Also
         removing unused I18N messages.

Comment 8 Lukas Krejci 2014-01-27 19:39:07 UTC
commit bba9df55b63a4169a735ae5d9e94beb1b6472ed3
Author: Lukas Krejci <lkrejci>
Date:   Mon Jan 27 20:37:41 2014 +0100

    [BZ 1035501] - Remove Add Graph to Monitor View Menu Item in Resource Tree - removing the zombie code for groups as well (commit 7355462e593 solved the issue only for resource tree). Also removing unused I18N 
    (cherry picked from commit 75ec4b37b6408822220b756186cbaa6dae361d3d)
    
    moved to release/jon3.2.x as part of BZ 1035501
    
    Signed-off-by: Lukas Krejci <lkrejci>

commit a2d9ad3c297ef33ee3c23944b775c36826631bc3
Author: Lukas Krejci <lkrejci>
Date:   Mon Jan 27 20:26:11 2014 +0100

    [BZ 1005922] - Selecting grouped fired alerts in chosen time range does not work - hiding the filters if the alerts are shown in the grouped mode and vice versa.
    
    moved to release/jon3.2.x as part of BZ 1035501
    
    (cherry picked from commit 3c5dc0d98419a209df10a832259e85cfc1608c98)
    
    Signed-off-by: Lukas Krejci <lkrejci>

Comment 9 Lukas Krejci 2014-02-14 22:05:04 UTC
I reverted the previous commits to release branch, because I seem to have messed it up. It seems like the second commit doesn't do what the commit message says.

Comment 10 Lukas Krejci 2014-02-14 22:28:54 UTC
New commits, now hopefully correctly cherry-picked from master:

commit 4c0315003664b4a62aca7efad40d909f30840a2c
Author: Jirka Kremser <jkremser>
Date:   Mon Jan 20 13:59:57 2014 +0100

    [BZ 1035501] - Remove Add Graph to Monitor View Menu Item in Resource Tree - removing the zombie code for groups as well (commit 7355462e593 solved the issue only f
    
    (cherry picked from commit 75ec4b37b6408822220b756186cbaa6dae361d3d)
    Signed-off-by: Lukas Krejci <lkrejci>

commit 94cb4612c9a0482faf7d14b4b3304d716dcea282
Author: Jirka Kremser <jkremser>
Date:   Wed Dec 4 19:17:46 2013 +0100

    [BZ 1035501] - Remove Add Graph to Monitor View Menu Item in Resource Tree - Removing the option leading to the old portal war page from the context menu.
    
    (cherry picked from commit 7355462e593ce38a8a7b88e3a0e68b4f9da2a561)
    Signed-off-by: Lukas Krejci <lkrejci>

Comment 11 Simeon Pinder 2014-02-18 15:08:43 UTC
Moving to ON_QA as available for testing in the following brew build:
https://brewweb.devel.redhat.com//buildinfo?buildID=336752

Note: the installed version is still JON 3.2.0.GA by design and this represents part of the payload for JON 3.2.1 also known as cumulative patch 1 for 3.2.0.GA.  How this will be delivered to customers is still being discussed.

Comment 12 Mike Foley 2014-02-18 19:08:04 UTC
verified jon 3.2.1 dr1

Comment 13 Mike Foley 2014-05-08 17:44:22 UTC
JON 3.2.1 released week of 5/5/2014