Project

General

Profile

Bug #9608

Content VIew json is missing the repository content counts

Added by Walden Raines about 8 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Walden Raines
Category:
API
Target version:
Katello 2.3.0
Difficulty:
easy
Triaged:
Yes
Bugzilla link:
1197836
Pull request:
https://github.com/Katello/katello/pull/5115
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

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.

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

Associated revisions

Revision 952d6d26 (diff)
Added by Walden Raines about 8 years ago

Fixes #9608 add repository/puppet module to CV components BZ 1197836.

We removed the inclusion of the entire repsoitories rabl in 0603abc in order
to speed up the page but repository and puppet module counts are being used
in the UI and thus need to be included in the rabl.

http://projects.theforeman.org/issues/9608
https://bugzilla.redhat.com/show_bug.cgi?id=1197836

Revision 9374bc58
Added by Walden Raines about 8 years ago

Merge pull request #5068 from waldenraines/9608

Fixes #9608 add repository/puppet module to CV components BZ 1197836.

Revision bf84eacf (diff)
Added by Walden Raines about 8 years ago

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

Revision bc604653
Added by Walden Raines about 8 years ago

Merge pull request #5115 from waldenraines/9608

Fixes #9608: correct repository count, BZ 1197836.

History

#1 Updated by Walden Raines about 8 years ago

  • Bugzilla link set to 1197836

#2 Updated by The Foreman Bot about 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/5068 added
  • Pull request deleted ()

#3 Updated by Walden Raines about 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#4 Updated by Walden Raines about 8 years ago

  • Status changed from Closed to Assigned
  • Target version changed from 67 to 68
  • Pull request added
  • Pull request deleted (https://github.com/Katello/katello/pull/5068)

(In reply to Kedar Bidarkar from comment #8)

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.

#5 Updated by The Foreman Bot about 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/5115 added
  • Pull request deleted ()

#6 Updated by Walden Raines about 8 years ago

  • Status changed from Ready For Testing to Closed

#7 Updated by Eric Helms almost 8 years ago

  • Legacy Backlogs Release (now unused) set to 31

Also available in: Atom PDF