Bug 976851 - compliance status cache is written multiple times
Summary: compliance status cache is written multiple times
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: subscription-manager
Version: 5.10
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
: ---
Assignee: Carter Kozak
QA Contact: IDM QE LIST
URL:
Whiteboard:
Depends On:
Blocks: rhsm-rhel510
TreeView+ depends on / blocked
 
Reported: 2013-06-21 16:08 UTC by Adrian Likins
Modified: 2013-10-01 13:47 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-01 13:47:31 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Adrian Likins 2013-06-21 16:08:38 UTC
Description of problem:

Related to https://bugzilla.redhat.com/show_bug.cgi?id=976848

When subman writes out the offline cache of compliance status, it is slow
and blocks. Compounding this is that it can happen multiple times in a single
subman invocation. 

It should only be neccasarry to do this once per invocation. Subman only needs to write to disk the last compliance status it receives, before existing.

Comment 1 Carter Kozak 2013-07-10 15:36:13 UTC
commit a6f551cebac16702a26950620995c718713cb532
Author: ckozak <ckozak>
Date:   Mon Jul 1 11:29:44 2013 -0400

    976848: 976851: thread cache write, limit disk reads, singleton

Comment 2 Carter Kozak 2013-07-17 17:50:34 UTC
Another performance fix, if it worked correctly there should be no difference in testing.

Comment 3 John Sefler 2013-07-17 18:19:52 UTC
I do not have any explicit test coverage to confirm that comment 0 was a problem nor assert that comment 0 is now fixed. 
I also do not have any metrics to confirm that performance has improved with this bug fix.

As suggested in comment 2, I can say that our automated tests have not revealed a regression in behavior with respect to compliance status that has not already been addressed in other bugzillas.  Therefore, I am flipping this bug to VERIFIED without any adverse regression.

[root@jsefler-5 ~]# rpm -q subscription-manager
subscription-manager-1.8.13-1.el5


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