= Abstract = Loading sync management page generates potentially thousands of notices While debugging bug 798007, found that each time a user loads the sync management page, a separate warning notice is being created for each of the errors that occured while syncing the repos. In the case of bug 798007, this resulted in over 3000 notices being generated per page load. This eventually affect the performance of the app. This needs to be investigated. If a notice is to be generated, it should probably be a single notices with a summary of the issues found in the notice details. Alternatively, in this case, perhaps those notices should be generated at sync time and not when loading the page. = Software Tested = * candlepin-0.5.22-1.el6.src.rpm * katello-0.1.300-1.el6.src.rpm * katello-certs-tools-1.0.2-2.el6.src.rpm * katello-cli-0.1.100-2.el6.src.rpm * katello-configure-0.1.100-7.el6.src.rpm * katello-httpd-ssl-key-pair-1.0-1.src.rpm * katello-qpid-broker-key-pair-1.0-1.src.rpm * katello-selinux-0.1.7-1.el6.src.rpm * katello-trusted-ssl-cert-1.0-1.src.rpm * pulp-0.0.267-2.el6.src.rpm
Created attachment 566379 [details] production log containing errors The errors in this log were also generated in the form of notices. Note: over 3000 notices would be generated based on this log.
QA note: This bug can be verified together with https://bugzilla.redhat.com/show_bug.cgi?id=798007
866d297c388460342ab956f86b6189c7bacb4404 This bug fixes makes a number of changes to how the most recent sync status is determined and stored in the database. Whenever a sync is started an object will be created in the databse to track the pulp task and be updated as the sync status is updated from pulp. Further, upon completion of a sync, a delayed job will kickoff that will generate a success or error notice and if there are errors dump the errors to the log. This change also reduces the number of calls to pulp when calculating the overall size of a product in the UI.798376 - Sync management page reworked to generate error and success notices once upon sync completion.
Fixing small problem in commit 995d64fd2e9c06a70784d830e50f5da0a4dfbbfb: Calling repo discovery on CLI caused 'undefined method `length' for 0:Fixnum' error. Setting default value for error_details to [] instead of 0
Another regressions found in system-tests fixed in commit 2ca51d8b96037a289428d9e8abca26b8abd54dd9
Another small fix in commit 54c6c3e01b18e3737b057480b3b07603b7ede355: ordering the tasks to get the most recent in UI sync management
Validated: * candlepin-0.5.26-1.el6.noarch * candlepin-tomcat6-0.5.26-1.el6.noarch * katello-0.1.304-1.el6.noarch * katello-all-0.1.304-1.el6.noarch * katello-candlepin-cert-key-pair-1.0-1.noarch * katello-certs-tools-1.0.4-1.el6.noarch * katello-cli-0.1.105-1.el6.noarch * katello-cli-common-0.1.105-1.el6.noarch * katello-common-0.1.304-1.el6.noarch * katello-configure-0.1.106-1.el6.noarch * katello-glue-candlepin-0.1.304-1.el6.noarch * katello-glue-foreman-0.1.304-1.el6.noarch * katello-glue-pulp-0.1.304-1.el6.noarch * katello-qpid-broker-key-pair-1.0-1.noarch * katello-qpid-client-key-pair-1.0-1.noarch * katello-selinux-0.1.9-1.el6.noarch * pulp-1.0.0-5.el6.noarch * pulp-common-1.0.0-5.el6.noarch * pulp-selinux-server-1.0.0-5.el6.noarch
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/RHEA-2012-0665.html
getting rid of 6.0.0 version since that doesn't exist