Bug 1023392

Summary: Defer Quickstarts target dir creation to installation process
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Petr Kremensky <pkremens>
Component: InstallerAssignee: Francisco Canas <fcanas>
Status: CLOSED CURRENTRELEASE QA Contact: Petr Kremensky <pkremens>
Severity: low Docs Contact: Russell Dickenson <rdickens>
Priority: unspecified    
Version: 6.2.0CC: fcanas, thauser
Target Milestone: CR1   
Target Release: EAP 6.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-12-15 16:18:21 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Petr Kremensky 2013-10-25 10:32:37 UTC
Description of problem:
 Quickstarts target dir is created on Quickstarts screen now. If user returns and change the dir, new dir is created. This way, user can end with bunch of empty folders designated for quickstarts installation.

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

How reproducible:
 Always

Steps to Reproduce:
 1. Start GUI installation
 2. Use default values to get to Quickstarts screen
 3. Push Next and Ok to create a new directory
 4. Return to Quicktarts screen, change the quisktarts installation path.
 5. Repeat the step 3. & 4. for a few times and Quit the installation

Actual results:
 Empty folder was created for each entered installation path directory.

Expected results:
 Installer didn't create any directory. Quickstarts directory is created in time when quickstarts are installed.

Comment 1 Francisco Canas 2013-10-25 16:10:44 UTC
I've modified izpack to hold off on creating directories until the files are laid down. This applies to all panels that make use of path inputs, as the issue was not with quickstarts itself but with how userInputPanel validates paths.

http://git.app.eng.bos.redhat.com/izpack.git/commit/?h=izpack-wip&id=51d293760ae7de0864d1ec56f66890aec369a3fa

Comment 2 Petr Kremensky 2013-11-01 11:27:28 UTC
Hi,
path input panel still creates new directory (at least one for quickstarts). Just follow the issue description.

Comment 3 Francisco Canas 2013-11-01 13:56:54 UTC
Apologies. This BZ shouldn't have been set to ON_QA as the izpack fix for the issue did not actually make it into the izpack build for ER7.

The fix will be present in the next build.

Comment 4 Petr Kremensky 2013-11-01 14:00:52 UTC
Changing Target milestone to CR1. According to the latest information there will be no more ER builds.

Comment 5 Petr Kremensky 2013-11-14 14:54:28 UTC
Verified on EAP 6.2.0.CR1 installer.