Project

General

Profile

Bug #23903

Removing subscription from manifest still shows repos to enable

Added by Thomas McKay about 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
High
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

For my test:

+ import manifest w/ satellite subscription
+ enable and sync satellite rpms repo
+ refresh manifest from portal where satellite subscription had been removed
+ confirm that satellite repos are no longer listed to enable
+ confirm that the previously enabled repo is still present
+ confirm that the previously enabled repo is marked as "orphaned" in some way

These last three confirm steps are broken.

Associated revisions

Revision 97a11e1c (diff)
Added by Justin Sherrill about 3 years ago

Fixes #23903 - Hide repo sets without a valid subscription

This adds a new paramter to the repo sets api, to only return repo sets
that have an active subscription. This works well with the new UI which
seems to be built around this assumption. For hammer, we want to still
show either enabled repo sets or repo sets with an active sub, so the
new default behavior is to show only these.

With this change there is no way to view repo sets that either are
not enabled, or do not belong to a valid subscription.

History

#1 Updated by Thomas McKay about 3 years ago

  • Bugzilla link set to 1590404

#2 Updated by Justin Sherrill about 3 years ago

  • Legacy Backlogs Release (now unused) set to 338

#3 Updated by Andrew Kofink about 3 years ago

  • Priority changed from Normal to High

#4 Updated by Justin Sherrill about 3 years ago

  • Triaged set to No
  • Pull request https://github.com/Katello/katello/pull/7498 added

#5 Updated by Brad Buckingham about 3 years ago

  • Target version changed from 1.3.1 to Katello 3.7.0

#6 Updated by Brad Buckingham about 3 years ago

  • Assignee set to Justin Sherrill

#7 Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing

#8 Updated by Justin Sherrill about 3 years ago

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

Also available in: Atom PDF