Author: markt
Date: Sun Apr 28 19:01:41 2013
New Revision: 1476815

URL: http://svn.apache.org/r1476815
Log:
Slightly shorter timeout. Add a note.

Modified:
    tomcat/trunk/test/org/apache/catalina/loader/TestVirtualContext.java

Modified: tomcat/trunk/test/org/apache/catalina/loader/TestVirtualContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/loader/TestVirtualContext.java?rev=1476815&r1=1476814&r2=1476815&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/loader/TestVirtualContext.java 
(original)
+++ tomcat/trunk/test/org/apache/catalina/loader/TestVirtualContext.java Sun 
Apr 28 19:01:41 2013
@@ -323,7 +323,13 @@ public class TestVirtualContext extends 
     private void assertPageContains(String pageUrl, String expectedBody,
         int expectedStatus) throws IOException {
         ByteChunk res = new ByteChunk();
-        int sc = getUrl("http://localhost:"; + getPort() + pageUrl, res, 30000,
+        // Note: With a read timeout of 3s the ASF CI buildbot was consistently
+        //       seeing failures with this test. The failures were due to the
+        //       JSP initialisation taking longer than the read timeout. The
+        //       root cause of this is the frequent poor IO performance of the
+        //       VM running the buildbot instance. Increasing this to 10s 
should
+        //       avoid these failures.
+        int sc = getUrl("http://localhost:"; + getPort() + pageUrl, res, 10000,
                 null, null);
 
         assertEquals(expectedStatus, sc);



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to