Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 589231 - when uploading a new package via an EAR or WAR Resource's Content>New subtab, user should not be required to select a repo
Summary: when uploading a new package via an EAR or WAR Resource's Content>New subtab,...
Alias: None
Product: RHQ Project
Classification: Other
Component: Content
Version: 1.3.1
Hardware: All
OS: All
medium vote
Target Milestone: ---
: ---
Assignee: Simeon Pinder
QA Contact: Sunil Kondkar
Depends On:
Blocks: jon24-content
TreeView+ depends on / blocked
Reported: 2010-05-05 15:49 UTC by Ian Springer
Modified: 2013-08-06 00:37 UTC (History)
1 user (show)

Fixed In Version: 2.4
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2010-08-12 16:47:25 UTC

Attachments (Terms of Use)
screenshot showing the issue (deleted)
2010-05-05 16:34 UTC, Ian Springer
no flags Details
Screenshot for the message (deleted)
2010-05-19 11:53 UTC, Sunil Kondkar
no flags Details

Description Ian Springer 2010-05-05 15:49:01 UTC
Description of problem:

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. Create a WAR Resource (e.g. helloworld.war)
2. Go to the WAR Resource's Content>New subtab.
3. Click the Upload New Package button.
4. Click the Upload File button and upload a new version of the WAR (e.g. helloworld_v2.war)
5. Try to click the Continue button.

Actual results:
You'll get an error that says "A repository deployment option must be specified." (see attached screenshot)

Expected results:
The original WAR package associated with the WAR Resource is not associated with a repo or a content source. I verified this by peeking at the DB using dbvis. So the package for the updated v2 WAR really should not require a repo be specified either.

I propose we add a 4th radio button labeled "None" to the select repository options, and have this radio button selected by default.

We could also potentially not require selecting a repository at all in the case where the Resource is a package-backed Resource such as an EAR or WAR (this would assume that a package-backed Resource would never have other poackages associated with it, besides its backing package).

Comment 1 Ian Springer 2010-05-05 16:34:16 UTC
Created attachment 411669 [details]
screenshot showing the issue

Comment 2 Simeon Pinder 2010-05-18 01:12:08 UTC
Made suggested changes to add None option for repo selection and select by default.

Available in successful master build >= 307, and 

commit hash:

Comment 3 Sunil Kondkar 2010-05-19 11:14:26 UTC
Verified on jon build#154 (Revision: 10620)

Observed that now there is a 'None' option for repository selection and the 'None' option is selected by default.

However, when clicked on 'Continue' button after uploading a .war file and selecting null option, it displays below message on the screen:

Failed to associate package [rhq-postinstaller.war] with repository ID [null]. Cause: java.lang.NumberFormatException:null.

Please find attached the screenshot for more details.

Comment 4 Sunil Kondkar 2010-05-19 11:53:08 UTC
Created attachment 415091 [details]
Screenshot for the message

Comment 5 Simeon Pinder 2010-05-26 09:03:38 UTC
Yep. This was caused by a curious reversion of the applied fix after checkin. I'm calling it a gitastrophy.  I re-applied the fix and the error message from the screenshot should be fixed now in successful build of master >= 355, and 
git hash: bd8ee1ee291e656dfec223110ed51d0cb24d2219.

Comment 6 Sunil Kondkar 2010-05-27 11:35:09 UTC
Verified on Jon build170 (Revision: 10621)

Observed that there is a 'None' option for repository selection and the
'None' option is selected by default. No exception is observed after clicking on 'Continue' button after uploading a .war file. New package is deployed successfully.

Comment 7 Corey Welton 2010-08-12 16:47:25 UTC
Mass-closure of verified bugs against JON.

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