Bug 1824959

Summary: [RFE] [sat-e-492] Activation Key repository list should be in 'Show All' mode by default with Organization Env Access aka Simple Content Access
Product: Red Hat Satellite Reporter: Mike McCune <mmccune>
Component: Subscription ManagementAssignee: Partha Aji <paji>
Status: CLOSED ERRATA QA Contact: Cole Higgins <chiggins>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.7.0CC: egolov, ehelms, jlenz, jturel, paji, rjerrido
Target Milestone: 6.9.0Keywords: FutureFeature, SubscriptionExperience, Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-katello-3.17.0-0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-04-21 13:12:29 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:
Attachments:
Description Flags
show-all.png none

Description Mike McCune 2020-04-16 18:10:55 UTC
In order to encourage customers to not add subscriptions to Activation Keys and Hosts when operating in relaxed enforcement (Organization Environment Access) mode we should have the UI default to 'Show All' on the "Activation keys -> Repository Sets" page.


Currently users have to know to check the 'Show All' selector to get the list of repositories populated without forcing the addition of subscriptions to the activation key.

Comment 3 Mike McCune 2020-04-16 18:11:13 UTC
Created attachment 1679455 [details]
show-all.png

Comment 4 Mike McCune 2020-06-24 16:59:53 UTC
Users with Simple Content Access are still adding subscriptions to Activation Keys because they are not aware that they need to click 'show all' to view the available repositories. The filtering makes sense in a subscription focused mode, but when SCA is on, the requirement to 'Show All' just gets in the way and it should be that way by default.

Comment 5 Jonathon Turel 2020-06-24 17:24:46 UTC
There's an API parameter for an Activation Key's product content called 'content_access_mode_all' which we can specify rather than using 'Show All'. This way the LCE and CV will be respected which is the behavior of SCA. That param was added a while back and it may have regressed since I do see references to it in the UI code but we aren't sending it. We can do the same for Content Hosts too (may be working already, but we should verify that).

https://github.com/Katello/katello/blob/b226cf98a8e16c338cda6d4117d690029d323657/app/controllers/katello/api/v2/activation_keys_controller.rb#L148
https://github.com/Katello/katello/blob/b226cf98a8e16c338cda6d4117d690029d323657/app/controllers/katello/api/v2/host_subscriptions_controller.rb#L172

Comment 6 Jeremy Lenz 2020-08-07 17:40:08 UTC
Created redmine issue https://projects.theforeman.org/issues/30592 from this bug

Comment 7 Partha Aji 2020-09-14 18:18:35 UTC
If the org has SCA I am planning to 
1) Always fetch the repo sets with content_access_mode_all = true
2) Hide the 'Show All' on activations key page.

Does that sound adequate ?

Comment 8 Jeremy Lenz 2020-09-14 20:25:44 UTC
^^ That will result in the following behavior:

In SCA mode:
- 'Show All' checkbox is not displayed
- Always behave as if the checkbox were checked

In non-SCA mode, no change.

I'm +1 to this

Comment 9 Bryan Kearney 2020-09-15 00:04:17 UTC
Upstream bug assigned to paji

Comment 10 Bryan Kearney 2020-09-15 00:04:19 UTC
Upstream bug assigned to paji

Comment 11 Bryan Kearney 2020-10-12 20:04:22 UTC
Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/30592 has been resolved.

Comment 17 errata-xmlrpc 2021-04-21 13:12:29 UTC
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 (Moderate: Satellite 6.9 Release), 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-2021:1313