Bug 601175

Summary: Bundle wizard does not tell why upload failed
Product: [Other] RHQ Project Reporter: Heiko W. Rupp <hrupp>
Component: ContentAssignee: Jay Shaughnessy <jshaughn>
Status: CLOSED CURRENTRELEASE QA Contact: Sunil Kondkar <skondkar>
Severity: medium Docs Contact:
Priority: urgent    
Version: 3.0.0   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 2.4 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-08-12 16:48:56 UTC Type: ---
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: 577210, 593121    
Attachments:
Description Flags
Red Dot none

Description Heiko W. Rupp 2010-06-07 12:01:42 UTC
Tried to upload a .war file as step 1 of the bundle wizard.

After clicking 'next' this shows a red exclamation mark with a  hover message of "Upload failed"
Server console show 
13:59:30,800 INFO  [[/coregui]] org.rhq.enterprise.gui.coregui.CoreGUI BundleDistributionFileUploadServlet: file was uploaded: test-dist-vhosts.war

- The UI should not only show a hover in case of failure, but a plain text field
- The message should include a reason for failure (e.g. in my case ".war files are no valid bundles")

Comment 1 Heiko W. Rupp 2010-06-07 12:02:19 UTC
Created attachment 421800 [details]
Red Dot

Comment 2 Jay Shaughnessy 2010-07-07 19:54:53 UTC
fix commit: 91c9d1fb7849b9966f60a2cdeb8bca1903e3389c

There is new messaging and is displayed within the wizard.

Comment 3 Sunil Kondkar 2010-07-08 10:57:42 UTC
Verified on jon-2.4.0.GA_QA build#52

When user tries to upload invalid bundle distribution file, the bundle wizard
box displays the message with reason as expected (Failed to upload bundle
distribution file
java.lang.IllegalArgumentException: Invalid bundle distribution file.
BundleType/Recipe not recognized by any deployed server bundle plugin.)

Comment 4 Corey Welton 2010-08-12 16:48:56 UTC
Mass-closure of verified bugs against JON.