Bug 537100 - [raw-config] clicking on Stop on Upload config UI does not stop the upload
Summary: [raw-config] clicking on Stop on Upload config UI does not stop the upload
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: unspecified
Hardware: All
OS: Linux
low
low
Target Milestone: ---
: ---
Assignee: John Sanda
QA Contact: Sudhir D
URL:
Whiteboard:
Depends On:
Blocks: RHQ_RawConfig
TreeView+ depends on / blocked
 
Reported: 2009-11-12 14:26 UTC by Preethi Thomas
Modified: 2010-06-30 13:33 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-06-30 13:33:24 UTC
Embargoed:


Attachments (Terms of Use)

Description Preethi Thomas 2009-11-12 14:26:04 UTC
Description of problem:
Upload UI displays 2 stop button while uploading a file. But clicking on either doesn't stop the upload.

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


How reproducible:


Steps to Reproduce:
1.go to a resource with raw-config support (raw-config-server)
2.click on configuration tab and edit file in file based mode 
3.click on upload
4. Add a file and click on stop while uploading. 
5. status says done and it automatically navigates to the edit ui
  
Actual results:
stop does not stop the upload

Expected results:
stop should stop the upload and user should be able to stay in the same upload page.

Additional info:

Comment 1 Adam Young 2009-11-20 21:14:34 UTC
It's worse than that.  If you push the stop button, you get:  An error.  I'm attaching the stack trace below.  


I'd like to get rid of the stop button altogether.


java.lang.OutOfMemoryError: Java heap space
	java.io.Writer.write(Writer.java:189)
	java.io.PrintWriter.write(PrintWriter.java:429)
	com.sun.faces.renderkit.html_basic.HtmlResponseWriter.write(HtmlResponseWriter.java:636)
	com.sun.facelets.FaceletViewHandler.renderView(FaceletViewHandler.java:628)
	org.rhq.enterprise.gui.common.framework.FaceletRedirectionViewHandler.renderView(FaceletRedirectionViewHandler.java:64)
	org.ajax4jsf.application.ViewHandlerWrapper.renderView(ViewHandlerWrapper.java:100)
	org.ajax4jsf.application.AjaxViewHandler.renderView(AjaxViewHandler.java:176)
	com.sun.faces.lifecycle.RenderResponsePhase.execute(RenderResponsePhase.java:110)
	com.sun.faces.lifecycle.Phase.doPhase(Phase.java:100)
	com.sun.faces.lifecycle.LifecycleImpl.render(LifecycleImpl.java:139)
	javax.faces.webapp.FacesServlet.service(FacesServlet.java:266)
	org.ajax4jsf.webapp.BaseFilter.doFilter(BaseFilter.java:532)
	org.rhq.enterprise.gui.legacy.WebUserTrackingFilter.doFilter(WebUserTrackingFilter.java:47)
	org.rhq.enterprise.gui.legacy.AuthenticationFilter.doFilter(AuthenticationFilter.java:129)
	org.rhq.enterprise.gui.common.upload.MultipartFilter.doFilter(MultipartFilter.java:63)

Comment 2 Adam Young 2010-02-17 21:04:38 UTC
Assigning to John since he has been working on the Upload functionality

Comment 3 John Sanda 2010-02-24 17:24:52 UTC
Preethi, I am marking this bug as modified to indicate that I have worked on it; however, I do not think there is a bug. It did not seem like the stop button was working, but I failed to take a couple things into consideration. First, the files that I have been uploading are very small. Secondly, I have only been testing on my own machine. Both/either of these could result in the entire file being sent across the wire before the stop/cancel operation has taken effect. 

I just now tried uploading a file that was 141 MB in size, and clicking on the stop button did in fact stop the upload. Can you retest with a large file?

Comment 4 Corey Welton 2010-03-26 12:34:49 UTC
qa -> sdharane

Comment 5 Sudhir D 2010-03-29 14:06:09 UTC
Has the raw-config stuff been merged to master? I did not seem to find a resource with raw config enabled?

Comment 6 John Sanda 2010-03-31 19:26:33 UTC
There are changes in the raw-config branch that have not been merged into master. I talked with Charles last week, and he asked that I hold off on merging them into master since it has not been decided if/when raw-config will be supported. With that said, it might still make sense to go ahead and merge into master since most of the raw-config work is already in master and since there are some fixes in the raw-config branch.

Sudhir, there are only a couple plugins that support raw config as far as I know - cobbler and nss. Adam Young wrote the nss plugin, but he is no longer working on RHQ. There is also the raw-config-test plugin that basically a mock plugin that we used for testing during development.

Comment 8 Corey Welton 2010-06-30 13:33:24 UTC
QA Closing.


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