Bug 723248 - subscription-manager-gui is ignoring the quanties>1 on a pool with multi-entitlement
Summary: subscription-manager-gui is ignoring the quanties>1 on a pool with multi-enti...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: subscription-manager
Version: 5.7
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: beta
: 5.8
Assignee: Bryan Kearney
QA Contact: John Sefler
URL:
Whiteboard:
Depends On:
Blocks: 715031
TreeView+ depends on / blocked
 
Reported: 2011-07-19 14:17 UTC by John Sefler
Modified: 2012-02-21 06:41 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
No description necessary
Clone Of:
Environment:
Last Closed: 2012-02-21 06:41:32 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
some observations to reproduce/avoid this bug (24.63 KB, image/png)
2011-07-19 15:00 UTC, John Sefler
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2012:0154 0 normal SHIPPED_LIVE subscription-manager bug fix update 2012-02-20 15:06:04 UTC

Description John Sefler 2011-07-19 14:17:17 UTC
Description of problem:
When using the gui to subscribe to a pool that is "multi-entitlement" enabled, the contract selection dialog allows me to increase the quantity higher than 1.

Problem 1: Currently, using the spinner to increase the quantity to a valid value, like 2, the subscribe seems to only consume 1 subscription. (I'm not sure what happened.  this appeared to be working in the sprint demo last week).

This appears to be a gui problem only.  The cli allows me to subscribe to the same pool with --quantity=2 and it consumes two subscriptions.

Problem 2: If the contract is for up to 13 subscriptions, the spinner button should be topped off at 13.


Version-Release number of selected component (if applicable):
[root@jsefler-onprem-62server ~]# rpm -qa  | grep subscription-manager
subscription-manager-firstboot-0.96.4-1.git.40.158de0a.el6.x86_64
subscription-manager-0.96.4-1.git.40.158de0a.el6.x86_64
subscription-manager-gnome-0.96.4-1.git.40.158de0a.el6.x86_64


Additional info:
If testing with the TESTDATA imported into candlepin, then the "Awesome OS Server Basic (multi-entitlement)" pool can be used for this test.

Comment 1 John Sefler 2011-07-19 14:37:23 UTC
[root@jsefler-onprem-62server ~]# rpm -q python-rhsm
python-rhsm-0.96.7-1.git.3.f3ce5d1.el6.noarch

Comment 2 John Sefler 2011-07-19 15:00:32 UTC
Created attachment 513825 [details]
some observations to reproduce/avoid this bug

In the attached screenshot, the user has incremented the spinner and if he then clicks Subscribe, only 1 of the subscriptions will be consumed.

It appears that the new quantity value in the spinner is not actually committed yet.  If the user re-clicks the row, then the new spinner value seems to get comitted for quantity and a click on Subscribe correct consumes the desired quantity.


I also realize that there was a recent usability design review (on 7/15/2011) regarding the workflow ro specify a quantity in the gui and this implementation is subject to change.

Comment 3 J.C. Molet 2011-07-20 17:07:43 UTC
(In reply to comment #0)
> 
> Problem 2: If the contract is for up to 13 subscriptions, the spinner button
> should be topped off at 13.
> 

The spinbutton should be limited here and do built in checking.  Its max needs to be the difference of the quantity consumed and the quantity available.  Its min needs to be 0 or 1 (not sure which would be better).   Currently you can select negative numbers, or numbers too great.  When this happens the contract selection dialog closes and you either have to discover your error by going to the "My subscriptions" tab or by the error pop up. In either case you have to search for the subscription and start the subscribe process over again. We could save the user a lot of time here.

Comment 4 Michael Stead 2011-07-25 20:06:48 UTC
(In reply to comment #2)
> In the attached screenshot, the user has incremented the spinner and if he then
> clicks Subscribe, only 1 of the subscriptions will be consumed.
> 
> It appears that the new quantity value in the spinner is not actually committed
> yet.  If the user re-clicks the row, then the new spinner value seems to get
> comitted for quantity and a click on Subscribe correct consumes the desired
> quantity.

Problem #1

This is now fixed in the master branch at 75efb0acc19648bfdde1ca9987a91a1837f69b79

Comment 5 Michael Stead 2011-07-25 20:13:40 UTC
(In reply to comment #3)
> (In reply to comment #0)
> > 
> > Problem 2: If the contract is for up to 13 subscriptions, the spinner button
> > should be topped off at 13.
> > 
> 
> The spinbutton should be limited here and do built in checking.  Its max needs
> to be the difference of the quantity consumed and the quantity available.  Its
> min needs to be 0 or 1 (not sure which would be better).   Currently you can
> select negative numbers, or numbers too great.  When this happens the contract
> selection dialog closes and you either have to discover your error by going to
> the "My subscriptions" tab or by the error pop up. In either case you have to
> search for the subscription and start the subscribe process over again. We
> could save the user a lot of time here.

As of 32614970381e53f323503a82e35683f1031a11eb if the quantity is NOT greater than 0, an error box is presented and the contract selector/subscription assistant dialog stays open.

NOTE: We still DO NOT enforce a maximum value, and the bug should remain open until we fix that problem.

Comment 9 spandey 2011-11-22 12:59:52 UTC
*** Bug 755861 has been marked as a duplicate of this bug. ***

Comment 10 J.C. Molet 2011-12-12 15:17:50 UTC
Testing against build: subscription-manager-gnome-0.98.7-1.git.2.a7a5a30.el5_7

I've tested this this in the following scenarios:
 - Subscription Assistant
 - All available Subscriptions
 - Contract Selections

In each case I've verified that the max/min values are set correctly, and that
the quantity can be changed with both text entry and the arrow buttons.  When subscribed to, it actually subscribes the requested number of entitlements.

Though the max values are set correctly for simple cases, you quickly run into edge cases.  Will open a bug on this separately.

As the max value has been pushed to the backlog in comment 8, I'm marking this VERIFIED as basic functionality works here.

Comment 11 William Poteat 2012-01-20 18:47:22 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
No description necessary

Comment 12 errata-xmlrpc 2012-02-21 06:41:32 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, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2012-0154.html


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