Bug 589173 - for a newly created WAR resource, several of the package details (on the Content>Deployed subtab) are blank or incorrect
Summary: for a newly created WAR resource, several of the package details (on the Cont...
Alias: None
Product: RHQ Project
Classification: Other
Component: Content
Version: 1.3.1
Hardware: All
OS: All
low vote
Target Milestone: ---
: ---
Assignee: Simeon Pinder
QA Contact: Sudhir D
: 583104 (view as bug list)
Depends On:
Blocks: jon24-content
TreeView+ depends on / blocked
Reported: 2010-05-05 15:06 UTC by Ian Springer
Modified: 2018-10-27 16:13 UTC (History)
5 users (show)

Clone Of:
Last Closed: 2010-08-12 16:53:30 UTC

Attachments (Terms of Use)
screenshot showing the issue (53.99 KB, image/png)
2010-05-05 15:06 UTC, Ian Springer
no flags Details
Helloworld ear file uploaded with correct information (140.79 KB, image/png)
2010-06-04 12:48 UTC, Sudhir D
no flags Details
testing after bug 596704 (172.63 KB, image/png)
2010-06-08 05:36 UTC, Sudhir D
no flags Details

External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Bugzilla 588507 None None None Never

Internal Trackers: 588507

Description Ian Springer 2010-05-05 15:06:35 UTC
Created attachment 411644 [details]
screenshot showing the issue

The File Name, SHA256, and Owner fields are blank, and the File Size field is incorrect (it's 0, rather than the actual file size).

See attached screenshot.

To reproduce, create a new WAR child Resource via RHQ, then go to that WAR Resource's Content>Deployed subtab.

Comment 1 Simeon Pinder 2010-05-26 08:54:00 UTC
This issue is a bit bigger than initially thought:
i) fields are invalid because agent side discovery had no way of mapping .war/.ear contents to specific RHQ PackageVersion instances.
ii) Some of the gui data was being pulled from incorrect places
iii) UI input fields for versioning were missing when inserting initial content instance. 
iv) Merge was not sufficiently connecting discovered and existing definitions.

The fix: 
i)Modify the discovery plugin element to additionally calculate package digests and update the newly discovered resources report returned.
ii)Modify the client side UI elements to take versioning details and to calculate package versioning information from the uploaded file elements.
iii)Modify the content discovery merge to lookup existing package version and installed package details an connect them to the discovered element information determined by the agent.

Available in successful master build >= 355, with 
commit hash: 5db6f43e511700a4de58dace6461a81b308ecc70 and 06136ac6f6ea806c3887cf2635ab0feb6b91378a.

Comment 2 Simeon Pinder 2010-05-26 08:58:09 UTC
When testing, the content upload and deploy process is asynchronous so there is a delay before newly uploaded packages are subsequently discovered and connected to correct version information.  

Deploying to an appserver typicaly shows the history items in progress.  For updates to installed content, the asynchronous updates and final completion is not as clear.

Comment 3 Simeon Pinder 2010-05-27 17:26:09 UTC
*** Bug 583104 has been marked as a duplicate of this bug. ***

Comment 4 Sudhir D 2010-06-04 12:47:25 UTC
I have verified this against the beta build jon-server-2.4.0.Beta0.

I'm attaching the screenshot.

Comment 5 Sudhir D 2010-06-04 12:48:52 UTC
Created attachment 421221 [details]
Helloworld ear file uploaded with correct information

Comment 6 Jay Shaughnessy 2010-06-04 18:36:37 UTC
This work was revised during the fix for 596704. See that bz for more info.

Comment 7 Sudhir D 2010-06-08 05:35:22 UTC
Verified this again on the 596704 environment and now the version shows as sha256. It shows a long sting for my app, but that's just mine. 

Marking this bug as verified. Also, attached the screenshot.

Comment 8 Sudhir D 2010-06-08 05:36:10 UTC
Created attachment 422029 [details]
testing after bug 596704

Comment 9 Jay Shaughnessy 2010-06-08 13:10:41 UTC
The sha256 is being set as the version, that is the correct behavior for an app that was not assigned a version manually (during, say, a Create New or Content update) and also does not have a manifest file including version information inside.

Comment 10 Corey Welton 2010-08-12 16:53:30 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.