Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 899466 (JBPAPP6-638)

Summary: Undeployment operation using the Management Console of an application deployed by filesystem copy is not persisted
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Dominik Pospisil <dpospisi>
Component: Web ConsoleAssignee: Heiko Braun <ike.braun>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: 6.0.0CC: brian.stansberry, dpospisi, ike.braun, ssilvert
Target Milestone: ---   
Target Release: EAP 6.0.0   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/JBPAPP6-638
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-10 13:18:23 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:
Embargoed:
Attachments:
Description Flags
deployments.png
none
UseFileSystem.png
none
missing-use-fs.jpg none

Description Dominik Pospisil 2011-08-16 11:03:43 UTC
Steps to Reproduce: 1. Start EAP 6.0.0 Alpha in standalone configuration
2. deploy web application by copying app war file to EAP_HOME/standalone/deployments
3. undeploy the application using Admin console's Manage Deployments -> Remove command
4. restart the server

Now the application is redeployed.

project_key: JBPAPP6

Undeployment operation using the Management Console in standalone configuration is not persisted. After restarting the server, application is redeployed.

Comment 1 Dominik Pospisil 2011-08-16 11:07:50 UTC
Steps to Reproduce: Removed: 1. Start EAP 6.0.0 Alpha in standalone configuration
2. deploy web application
3. undeploy the application using Admin console's Manage Deployments -> Remove command
4. restart the server

Now the application is redeployed.
 Added: 1. Start EAP 6.0.0 Alpha in standalone configuration
2. deploy web application by copying app war file to EAP_HOME/standalone/deployments
3. undeploy the application using Admin console's Manage Deployments -> Remove command
4. restart the server

Now the application is redeployed.



Comment 2 Dominik Pospisil 2011-08-16 12:09:14 UTC
Link: Added: This issue relates to JBQA-5289


Comment 3 Rajesh Rajasekaran 2012-02-02 21:48:14 UTC
Dominik, is there an upstream JIRA for this issue? 

Comment 4 Dominik Pospisil 2012-02-03 09:33:36 UTC
Link: Added: This issue is related to AS7-3595


Comment 5 Dominik Pospisil 2012-02-03 17:22:57 UTC
Link: Added: This issue is related to AS7-784


Comment 6 Dominik Pospisil 2012-02-07 10:39:05 UTC
AS7-784 is the right upstream JIRA.

Comment 7 Brian Stansberry 2012-03-12 16:44:01 UTC
Link: Added: This issue is duplicated by JBPAPP-7922


Comment 8 Stan Silvert 2012-04-18 11:51:39 UTC
This is not a console problem.  Once AS7-784 is resolved, this should work correctly in console and CLI.

However, if you want me to hide filesystem deployments we can look into that.

Reassigning to Brian for his comments.

Comment 9 Brian Stansberry 2012-04-18 15:24:24 UTC
AS7-784 is not going to be resolved for EAP 6.0.0.GA, so disabling the ability to control their lifecycle in the console makes sense.

I wouldn't advocate hiding them completely though. That seems like a cure that's worse than the disease.

Comment 10 Stan Silvert 2012-04-18 15:34:52 UTC
Thanks Brian.  I'm taking this back.  I'd like to get some opinions on it though.

I could try just hiding the "Remove" and "Update Content" links for filesystem deployments.  But that would look strange and the user wouldn't understand why those operations are not available.  Ideally, I'd like a way to show the user that these are filesystem deployments and they must be controlled from the filesystem.  

Any thoughts on how best to communicate the situtation to the user?

Comment 11 Stan Silvert 2012-04-18 15:49:02 UTC
Attached file showing what deployment screen looks like currently.

Comment 12 Stan Silvert 2012-04-18 15:49:02 UTC
Attachment: Added: deployments.png


Comment 13 Brian Stansberry 2012-04-18 16:05:11 UTC
Thanks for the screen shot!

Perhaps replace the links in the columns with "N/A" and a little question mark icon? Or instead of the question mark add the "Need help?" thing above the box. For text, something like:

"This deployment is managed by a file system deployment scanner. To disable or remove the deployment or update its content, update the directory containing the deployment."

BTW, I don't see "Update content" ever being supported for scanner deployments. I don't think we want to start having to support uploading files to the server and then dropping them into the deployments/ dir. Theoretically we could -- unless the admin didn't give the server account write perms to the file, which may very well happen in some odd cases. 

Comment 14 Stan Silvert 2012-04-18 17:01:56 UTC
Yea, supporting "Update Content" would be very problematic for filesystem deployments.

I'm thinking of replacing the links with a file icon.  I like your explanatory text.  I'll use it as long as you promise not to sue for copyright infringement. :-)

Comment 16 Stan Silvert 2012-04-24 23:23:03 UTC
https://github.com/heiko-braun/as7-console/pull/104

After thinking this over for awhile, I think I came up with a really simple and obvious way to communicate with the user.  I just put static text "Use File System" in place of the command link.

I've attached a screen shot so you can see how this looks in context.

Comment 17 Stan Silvert 2012-04-24 23:23:24 UTC
Attachment: Added: UseFileSystem.png


Comment 18 Rajesh Rajasekaran 2012-04-25 22:27:32 UTC
Based on the commit date, setting fix version to ER7

Comment 19 Dominik Pospisil 2012-05-10 12:41:23 UTC
I am not seeing "Use File System" hints in ER7 build. The relevant fields are empty.

Comment 20 Dominik Pospisil 2012-05-10 12:43:53 UTC
Attaching screenshot of ER7 build with three apps deployed - deployment0 was deployed using console and the other two using filesystem copy.

Comment 21 Dominik Pospisil 2012-05-10 12:43:53 UTC
Attachment: Added: missing-use-fs.jpg


Comment 22 Stan Silvert 2012-05-10 12:56:34 UTC
Someone changed this so that it says "File System Deployment" under the name and it leaves the other columns blank.  

I assume Heiko made these changes.  I'll assign to him and you guys can discuss what you want to do.

For the record, I'm fine with either approach.

Comment 23 Heiko Braun 2012-05-10 13:04:58 UTC
You'd need to elaborate. A FS deployment can not be changed through the console anymore. And it shouldn't.

Comment 24 Dominik Pospisil 2012-05-10 13:18:23 UTC
My assumption was that it is missing by mistake. But I am fine with how it looks right now. Closing, reopen if anyone has some objections.

Comment 25 Anne-Louise Tangring 2012-11-05 17:45:11 UTC
Docs QE Status: Removed: NEW