Bug 771757

Summary: allow control for listing subscriptions which "match my system"
Product: Red Hat Satellite Reporter: Chris Duryee <cduryee>
Component: Content ManagementAssignee: Tom McKay <tomckay>
Status: CLOSED CURRENTRELEASE QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.0CC: bkearney, tomckay
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-04-27 00:32:02 UTC Type: ---
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:    
Bug Blocks: 703617, 747354, 804245    
Attachments:
Description Flags
sub-mgr-gui filtering
none
headpin available subscription search none

Description Chris Duryee 2012-01-04 20:51:00 UTC
Created attachment 550760 [details]
sub-mgr-gui filtering

Description of problem:

Occasionally, a user will want to consume entitlements that do not match what's installed on the system. One use case would be if the system is being used as a hypervisor and the sysadmin wants to subscribe simply to get access to bonus pools.

Right now, this is only possible via running subscription-manager on the host; headpin is not at feature parity with sub-mgr in this respect. See screenshots for more detail.

Comment 1 Chris Duryee 2012-01-04 20:53:04 UTC
Created attachment 550761 [details]
headpin available subscription search

the writing in the headpin screenshot is incorrect. The checkboxes in sub-mgr-gui are to disable filtering; not enable. I will post REST calls in a bit showing the difference.

Comment 2 Chris Duryee 2012-01-04 22:00:19 UTC
If you append a "&listall=true" to the end of the REST call to fetch the pools, you should get back an unfiltered list. This should be sufficient to show everything, so a user can subscribe to a pool to enable the bonus pools.

Comment 3 Tom McKay 2012-01-05 20:14:17 UTC
Email sent...

Kyle (and other UX opinions),

See screenshots off of https://bugzilla.redhat.com/show_bug.cgi?id=771757 for context

When viewing available subscriptions for a system in subscription-manager-gui, there are three checkboxes to help limit what is shown:
1 match my system (done in candlepin by not including param listall=true)
2 match my installed products (done in sub-mgr-gui)
3 have no overlap with existing subscriptions (done in sub-mgr-gui)

Currently, katello/headpin does not include the listall=true param so only subscriptions that candlepin considers matching for a system are considered. (eg. If a system has 16 sockets and the subscription only covers 8 sockets, it is not shown.)

Where and in what form should these options appear in the katello/headpin UI?

Comment 4 Tom McKay 2012-01-05 21:06:55 UTC
To reproduce

curl -k -u admin:admin https://localhost:8443/candlepin/pools/?consumer=1b144001-2aa6-4863-ab49-14b195022d72\&listall=true

substituting appropriate consumer id

Comment 5 Tom McKay 2012-01-09 14:50:04 UTC
Per conversation w/ Kyle Baker:

+ Add checkbox next to Available Subscriptions called "Match System"
+ Change font of this and auto-heal to be same as the Details, Facts, etc.
+ Right justify both checkboxes

Comment 6 Tom McKay 2012-01-23 19:11:31 UTC
  2c083f4... 771757 - toggle and callbacks added; still work to be done (*)
  3745c30... 771757 - candlepin called properly with saved user preferen (*)
  fa6cf37... 771757 - page refreshes on toggle checkbox and callback tri (*)
  9277621... Merge branch 'master' into BZ-771757 (*)
  c27a160... 771757 - added flag to api/systems_controller.rb and update (*)
  88086f2... Merge branch 'master' into BZ-771757 (*)

To confirm change, create a system w/ a fact of 64 cpu sockets and view that system using a manifest w/ a product that covers less sockets. Toggling checkbox will now show the lesser socket subscriptions where previously they were not shown.

Comment 7 Mike McCune 2012-01-26 19:08:14 UTC
mass ON_QA move

Comment 9 Jeff Weiss 2012-03-17 00:02:38 UTC
Feature works, but there is an unacceptable delay between the time you check the checkbox, and when the list updates.  Nothing indicates to the user that the app is busy. The "working" cursor appears for about 1/2 second, the normal cursor returns, and then finally the list updates about 3-4 seconds later.  

Verified, see dependent bug for further details