Bug 1176435 - Installing large number of packages causes ISE
Summary: Installing large number of packages causes ISE
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite 5
Classification: Red Hat
Component: WebUI
Version: 570
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
Assignee: Stephen Herr
QA Contact: Pavel Studeník
URL:
Whiteboard:
Depends On:
Blocks: sat570-blockers
TreeView+ depends on / blocked
 
Reported: 2014-12-21 20:04 UTC by Dimitar Yordanov
Modified: 2015-01-13 10:25 UTC (History)
3 users (show)

Fixed In Version: spacewalk-java-2.3.8-94
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-01-13 10:25:39 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Dimitar Yordanov 2014-12-21 20:04:43 UTC
Description of problem:
Installing large number (2000 rpm) of packages causes ISE.

Version-Release number of selected component (if applicable):
spacewalk-java-2.3.8-84

How reproducible:
100%

Steps to Reproduce:
1. Register Client to Satellite 5 named test_client.
2. Systems -> Systems -> test_client -> Software -> Install -> Select All(2000 rpms) -> Install Selected Packages -> Confirm
3.Events -> Package Install scheduled by xxxx 

Actual results:
ISE


Additional info:
Regression.
This problem is not present on Satellite 5.6 when installing 3500 rpms.

Comment 1 Stephen Herr 2014-12-22 15:19:53 UTC
Committing to Spacewalk master:
e8f52d3f534af8b2cec10dfc4cbc24c886f3a0e3

This slowness was caused by inefficient hibernate lookup after the perl -> java migration of this page. The original java code was inefficient because:

1) it looked up the list of packages twice and stored them in two different lists (unnecessarily)
2) when building the package nevra string for display it would throw out the already-loaded Name / EVR / Arch information and force hibernate to look up the nevra info by nameId / evrId / archId, for every package in the list.

Fixing those two things makes the page load much faster (a second or two for two thousand packages).

Comment 3 Stephen Herr 2014-12-22 15:30:57 UTC
One additional note: this change will affect all types of package actions (install, upgrade, etc) but nothing else (like errata actions). If problems are noted in other types of actions then that should probably be filed as a different bug.

Comment 6 Clifford Perry 2015-01-13 10:25:39 UTC
With the release of Red Hat Satellite 5.7 on January 12th 2015 this bug is
being moved to a Closed Current Release state. 

The Satellite 5.7 GA Errata:
 - https://rhn.redhat.com/errata/RHSA-2015-0033.html 

Satellite 5.7 Release Notes:
 -
https://access.redhat.com/documentation/en-US/Red_Hat_Satellite/5.7/html-single/Release_Notes/index.html

Satellite Customer Portal Blog announcement for release:
 - https://access.redhat.com/blogs/1169563/posts/1315743 

Cliff

NOTE: This bug has not been re-verified (moved to RELEASE_PENDING) prior to release. We assume that the bug has indeed been fixed and not regressed since we initially verified it. Please re-open in the future if needed.


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