Bug 1145195

Summary: Bundle destination view: deployments table shows partial results and purge button is disabled even if there is a live deployment
Product: [JBoss] JBoss Operations Network Reporter: Armine Hovsepyan <ahovsepy>
Component: Provisioning, UIAssignee: Thomas Segismont <tsegismo>
Status: CLOSED CURRENTRELEASE QA Contact: Armine Hovsepyan <ahovsepy>
Severity: medium Docs Contact:
Priority: unspecified    
Version: JON 3.3.0CC: jkremser, jshaughn, mfoley, tsegismo
Target Milestone: ER04   
Target Release: JON 3.3.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-12-11 14:04:49 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: 1145648    
Bug Blocks:    
Attachments:
Description Flags
deployments
none
purge_btn
none
purge_btn_deployment_count_300 none

Description Armine Hovsepyan 2014-09-22 14:17:59 UTC
Created attachment 940038 [details]
deployments

Description of problem:
only last 50 deployments to destination are visible on GUI

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

How reproducible:
always

Steps to Reproduce:
1. create a bundle
2. deploy to destination (e.g. 'test') >200 times
3. Navigate to destination and click Deployments sub-tab

Actual results:
only last 50 deployments are listed

Expected results:
all deployments are listed


Additional info:
screen-shot attached

Comment 1 Armine Hovsepyan 2014-09-23 08:56:56 UTC
Created attachment 940320 [details]
purge_btn

Comment 3 Jirka Kremser 2014-09-23 12:28:54 UTC
this was fixed by my work on bug 1129051

All we need to do is to cherry-pick the commit from master
104ff1c86

Comment 4 Thomas Segismont 2014-09-23 13:07:08 UTC
(In reply to Jirka Kremser from comment #3)
> this was fixed by my work on bug 1129051
> 
> All we need to do is to cherry-pick the commit from master
> 104ff1c86

The 50 rows limitation should be fixed along with bug 1129051.

But the Purge button issue remains and I'm working on it.

Comment 5 Thomas Segismont 2014-09-23 13:09:27 UTC
By the way, Bug 1129051 is the upstream bug. The product bug is Bug 1145648

Comment 6 Thomas Segismont 2014-09-23 13:59:28 UTC
Fixed in master

commit b8764388e55def65d9bb2dba62c7830ad4783bfa
Author: Thomas Segismont <tsegismo>
Date:   Tue Sep 23 15:57:32 2014 +0200
    
    This would happen when a destination has more than 200 deployments.
    
    Now #checkIfDisabled method does not load all deployments for destination but only one live deployment if it exists.
    
    Side work:
    * removed call to deprecated method

Comment 7 Jay Shaughnessy 2014-09-24 19:36:44 UTC
Release/jon3.3.x commit 3ed7ce007ad34d45c44c0a544f17ff0477fda6cb
Author: Thomas Segismont <tsegismo>
Date:   Tue Sep 23 15:57:32 2014 +0200

    (cherry picked from commit b8764388e55def65d9bb2dba62c7830ad4783bfa)
    Signed-off-by: Jay Shaughnessy <jshaughn>

Comment 8 Simeon Pinder 2014-10-01 21:33:33 UTC
Moving to ON_QA as available for test with build:
https://brewweb.devel.redhat.com/buildinfo?buildID=388959

Comment 9 Armine Hovsepyan 2014-10-07 09:43:04 UTC
Created attachment 944488 [details]
purge_btn_deployment_count_300

Comment 10 Armine Hovsepyan 2014-10-07 09:43:20 UTC
verified in JON 3.3 ER04