Bug 605821 - ant launcher invoker deployer multiple times
Summary: ant launcher invoker deployer multiple times
Alias: None
Product: RHQ Project
Classification: Other
Component: Provisioning   
(Show other bugs)
Version: 3.0.0
Hardware: All
OS: All
medium vote
Target Milestone: ---
: ---
Assignee: Ian Springer
QA Contact: Corey Welton
Depends On:
Blocks: jon24-provisioning
TreeView+ depends on / blocked
Reported: 2010-06-18 20:58 UTC by John Mazzitelli
Modified: 2013-08-06 00:37 UTC (History)
1 user (show)

Fixed In Version: 2.4
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2010-08-12 16:52:23 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

Description John Mazzitelli 2010-06-18 20:58:32 UTC
AntLauncher calls ProjectHelper.configureProject() which is causing the BundleTask to call the Deployer (because BundleTask.execute is executed). Then later on, AntLauncher invokes the "" target which also invokes the Deployer.

This means we deploy twice, which is bad.


is a unit test that can illustrate this. I commented out the assert so Hudson doesn't fail this build right now (its in the release-3.0.0 branch), but to see this test fail, uncomment this assert:

// TODO: uncomment once we fix the problem that ant launcher invokes the deploy multiple times
//assert previousProps == null : "There should not be any previous deployment metadata";

Comment 1 John Mazzitelli 2010-06-21 20:51:48 UTC
i'm putting additional tests for upgrade, clean upgrade and revert testing.

Each has some lines commented out until this BZ is fixed. Once this is fixed, uncomment the lines in the test and confirm they pass. If they pass, this BZ should be considered fixed and ready for QA.

The comments to look for is this (they appear several times):

// TODO: uncomment once we fix the problem that ant launcher invokes the deploy multiple times

Comment 2 Ian Springer 2010-06-23 01:42:15 UTC
This is fixed by commit a75768dea4dd93403c2711838d0bc87262edee0c and has been pushed to the release-3.0.0 branch.

Comment 3 Corey Welton 2010-06-24 13:03:25 UTC
QA Closing.  This is basically a code change covered by unit tests.

Comment 4 Corey Welton 2010-08-12 16:52:23 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.