Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1197836 - Content VIew json is missing the repository content counts
Content VIew json is missing the repository content counts
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Content Management (Show other bugs)
Nightly
Unspecified Unspecified
unspecified Severity medium (vote)
: Unspecified
: Unused
Assigned To: Walden Raines
Kedar Bidarkar
http://projects.theforeman.org/issues...
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2015-03-02 12:15 EST by Walden Raines
Modified: 2017-02-23 15:24 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-08-12 01:28:22 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
prior to adding cv to composite cv (54.81 KB, image/png)
2015-03-17 04:11 EDT, Kedar Bidarkar
no flags Details
after adding cv to composite cv (60.84 KB, image/png)
2015-03-17 04:12 EDT, Kedar Bidarkar
no flags Details
the repo count on cv list tab is fine now (70.34 KB, image/png)
2015-05-06 12:25 EDT, Kedar Bidarkar
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
Foreman Issue Tracker 9608 None None None 2016-04-22 11:49 EDT
Red Hat Product Errata RHSA-2015:1592 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 05:04:35 EDT

  None (edit)
Description Walden Raines 2015-03-02 12:15:43 EST
Commit https://github.com/waldenraines/katello/commit/0603abcadd03186cf0a969274cd42c538e2a1170 removed the inclusion of the entire repositories rabl to speed up the content view index page but neglected to include the repository counts in the attribute list.  We need to add these counts back so they can be properly displayed in the UI.

h3. Steps to Reproduce

1. Ensure you have at least one published content view with at least repository or puppet module included
2. Create a composite content view
3. Add the repository via the content views tab
4. Click on list/remove
5. Note that the repository/puppet module count is not displayed
Comment 1 Walden Raines 2015-03-02 12:15:44 EST
Created from redmine issue http://projects.theforeman.org/issues/9608
Comment 2 Walden Raines 2015-03-02 12:15:46 EST
Upstream bug assigned to walden@redhat.com
Comment 3 Walden Raines 2015-03-02 12:32:51 EST
PR: https://github.com/Katello/katello/pull/5068
Comment 5 Bryan Kearney 2015-03-04 14:05:32 EST
Moving to POST since upstream bug http://projects.theforeman.org/issues/9608 has been closed
-------------
Walden Raines
Applied in changeset commit:katello|952d6d268ba90ca5d182ea4e0708e7b8df84a10d.
Comment 8 Kedar Bidarkar 2015-03-17 04:08:41 EDT
Now it does show the repository/puppet-module count, but the repo count is wrong.

Before adding: checking via the add Tab: the cv show 3 repos
After adding: checking via the list/view tab the cv now shows 9 repos.

Also clicking on the 9 repos , just shows 3 repos.
Comment 9 Kedar Bidarkar 2015-03-17 04:11:58 EDT
Created attachment 1002687 [details]
prior to adding cv to composite cv
Comment 10 Kedar Bidarkar 2015-03-17 04:12:30 EDT
Created attachment 1002688 [details]
after adding cv to composite cv
Comment 11 Walden Raines 2015-03-17 09:56:08 EDT
PR: https://github.com/Katello/katello/pull/5115
Comment 12 Walden Raines 2015-03-17 16:18:12 EDT
commit bc604653a55de7f15e1b10667570517eeccb9eb2
Merge: ee9bf59 bf84eac
Author: Walden Raines <walden@redhat.com>
Date:   Tue Mar 17 13:37:54 2015 -0400

    Merge pull request #5115 from waldenraines/9608
    
    Fixes #9608: correct repository count, BZ 1197836.

commit bf84eacfa15a5dc7f625da7f4196875ac5f8fb19
Author: Walden Raines <walden@redhat.com>
Date:   Tue Mar 17 09:47:05 2015 -0400

    Fixes #9608: correct repository count, BZ 1197836.
    
    The respository counts were wrong on the content view
    composite list page.  This commit fixes the counts.
    
    http://projects.theforeman.org/issues/9608
    https://bugzilla.redhat.com/show_bug.cgi?id=1197836
Comment 14 Kedar Bidarkar 2015-05-06 12:25:07 EDT
Created attachment 1022734 [details]
the repo count on cv list tab is fine now

VERIFIED with Sat6.1.0-GA-SNAP2
Comment 15 Bryan Kearney 2015-08-11 09:30:03 EDT
This bug is slated to be released with Satellite 6.1.
Comment 16 errata-xmlrpc 2015-08-12 01:28:22 EDT
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-2015:1592

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