Bug 1155954

Summary: [RFE] big delay on checkbox in Manage Repositories
Product: Red Hat Enterprise Linux 6 Reporter: Lubos Kocman <lkocman>
Component: subscription-managerAssignee: Michael Stead <mstead>
Status: CLOSED ERRATA QA Contact: John Sefler <jsefler>
Severity: medium Docs Contact:
Priority: high    
Version: 6.8CC: alikins, bkearney, candlepin-bugs, csnyder, dgoodwin, jgalipea, jmolet, mcepl, mstead, redakkan, salmy
Target Milestone: rcKeywords: FutureFeature, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
: 1161157 (view as bug list) Environment:
Last Closed: 2016-05-10 20:35:42 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:
Bug Depends On:    
Bug Blocks: 1161157, 1209535, 1275344    

Description Lubos Kocman 2014-10-23 09:00:34 UTC
Version-Release number of selected component (if applicable):

subscription-manager-gui-1.12.14-7.el6.x86_64

How reproducible:


Steps to Reproduce:
1. sudo subscription-manager-gui
2. goto Manage/Repositories (Ctrl + s)
3. Enable few repositories (in my case rhel-6-workstation-optional-rpms 

Actual results:

Enabled checbox is checked several seconds after clicking (up to 10).
I understand that (un)checking it after (un)registering channel solves issues with e.g. click and close 

Expected results:

Check immidiatelly  or with minimal delay.

Additional info:
It's a really unpleasant user experience with mentioned delay.
Sometimes you check it twice since nothing happens and ... result is 10seconds until checked and another 10 seconds to uncheck it :-) ... you know customers will try and do that.

Comment 2 Matěj Cepl 2014-11-06 08:58:47 UTC
Should I make a clone of this bug to RHEL 7, where situation is exactly the same?

Comment 3 Bryan Kearney 2014-11-06 13:23:45 UTC
Sure, that would be great. Thank you!

Comment 4 Devan Goodwin 2014-12-01 15:24:05 UTC
Will be tough to fix but seems worthwhile. Probably need to lock the UI and show a progress bar to prevent things getting weird, async server communication would be nice but will get error prone and very weird here if the requests start failing.

Comment 5 Adrian Likins 2015-03-02 22:23:16 UTC
It looks like one of the bottlenecks here is that we check the consumer systems overrides and it's systems 'release' every time we toggle a repo enabled. 

We also completely regenerate the redhat.repo file before returning.

So each time we toggle 'enabled', we:

1) ask the server for override info
2) ask the server for the latest system release
3) regenerate the redhat.repo applying overrides
(and spin off threads to write out the client side offline caches)

Using the cached release here would help some.


Better use of the event loop as well, and some sort of progress indicator
to indicate the wait time. So we'd flip the check box instantly, and flip on the progress, and flip it off when it completes and we've done the steps mentioned above.

The network stuff would need to get switched to AsyncBackend which that screen isn't currently using.

Comment 6 J.C. Molet 2015-04-01 20:25:13 UTC
As a normal user of things, I would expect the state of the checkboxes to be saved in memory until I hit the 'Close' button (should be renamed 'Save' in my scenario), and only then would it consult with candlepin and rewrite the redhat.repo.

Comment 9 Michael Stead 2015-08-06 12:04:55 UTC
Fixed and will be available in:

subascription-manager-1.16.1+


Commit Details:

3ab798843011b05f9a24dd0c82db96466a7e72f4

https://github.com/candlepin/subscription-manager/pull/1235

Comment 10 Chris Snyder 2016-01-18 17:00:48 UTC
In upstream master and the latest builds of subman for RHEL 6.8.
Setting version to 6.8 and moving to the MODIFIED status in order to add this BZ to the errata.

Comment 12 Rehana 2016-01-19 09:41:14 UTC
Retested on ,

subscription management server: 0.9.51.14-1
subscription management rules: 5.15.1
subscription-manager: 1.16.7-1.el6
python-rhsm: 1.16.5-1.el6

Tried to enable/disable the multiple repo from subscription-manager-gui , checkbox are checked/unchecked without any delay. Marking the bug to verified.

Comment 14 errata-xmlrpc 2016-05-10 20:35:42 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.

https://rhn.redhat.com/errata/RHBA-2016-0797.html