Bug 758753 - CLI Method updateBackingContent Does Not Persist Information Correctly in Database
Summary: CLI Method updateBackingContent Does Not Persist Information Correctly in Dat...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: CLI
Version: 4.3
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: RHQ 4.3.0
Assignee: Stefan Negrea
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On: RHQ-2387 602475 727959 761593 767247 767393 769986 771418 771777 772722 773061 781763 782207 790753 801997 802787 802793 824503
Blocks: jon310-sprint11, rhq44-sprint11 788629
TreeView+ depends on / blocked
 
Reported: 2011-11-30 16:27 UTC by Stefan Negrea
Modified: 2018-11-27 21:09 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
: 788629 (view as bug list)
Environment:
Last Closed: 2013-09-01 10:07:59 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 788629 0 high CLOSED CLI Method updateBackingContent Does Not Persist Information Correctly in Database 2021-02-22 00:41:40 UTC

Internal Links: 788629

Description Stefan Negrea 2011-11-30 16:27:55 UTC
Description of problem:
CLI command updateBackingContent(fileName) for a resource does not persist correctly provided files.

How reproducible:
Every time

Steps to Reproduce:
1. Create four versions of a WAR files to upload to a resource.
2. Use resource.updateBackingContent(fileName) consecutively to upload the versions of the file.
  
Actual results:
The WAR files are not replaced correctly with new content on the third and subsequent attempts.

Expected results:
The WAR file should be updated correctly each time updateBackingContent is invoked.


Additional info:
This problem is exclusive only to the CLI; the similar UI functionality works as expected. A code inspection uncovered signification implementation differences between UI and CLI. 

The solution is to update the CLI code to use the same APIs as the UI. Such update requires code changes in the CLI, server, and remote interfaces.

Comment 1 Stefan Negrea 2011-12-08 15:59:35 UTC
After further investigation this issue affects both the UI and CLI for deploying packages to content type resources. The CLI problem is more visible because the code relies on the existing version of a package to find the version for the current deployment. The UI has a less visible/obvious problem because the user is required to type the version of the package to be deployed and thus avoids retrieval of inconsistent data (see explanation below). 

The current deployment request life-cycle request is very complex; the big picture is this:
1) A package deployment is requested by the user.
2) The new packages are stored in the database.
3) Based on database information a request is generated for the agent
4) The agent receives the request and:
  a) copies the new packages to the correct location and sends the result of the deployment action back to the server. 
  b) starts discovery processes for content, service and server; each of thse processes will then send their results back to the server. 
5) The server receives the deployment response and does some database updates.
6) The server receives the discovery response and does some database updates.

The root cause is missing/incorrect installed package information on the server. The server actions from 5 and 6 operate over the same data but they update slightly different content. The defect stems from the fact that steps 5 and 6 leave the server data inconsistent with the actual agent-side state. The issue is even more visible on Tomcat 6 because of unimplemented SHA computation functionality. 

Based on the current investigation code fixes are required only in the backend, agent, and plugins. This would resolve both the UI and CLI issues without explicit updates.

Comment 3 Stefan Negrea 2012-02-08 16:26:27 UTC
For verification please execute and refer to test cases for dependent bugs (in particular bug 761593, bug 767247, and bug 767393).

Comment 6 Heiko W. Rupp 2013-09-01 10:07:59 UTC
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.


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