[GitHub] houshengbo commented on a change in pull request #3010: Adapt timeouts of controller and nginx.

[GitBox <gi...@apache.org>]

houshengbo commented on a change in pull request #3010: Adapt timeouts of controller and nginx.
URL: https://github.com/apache/incubator-openwhisk/pull/3010#discussion_r153232430
 
 

 ##########
 File path: core/controller/src/main/resources/application.conf
 ##########
 @@ -15,7 +15,7 @@ akka.http {
     # The controller holds connections up to 60s for blocking invokes, and
     # all other operations are expected to complete quickly. We allow a grace
     # period in addition to the blocking invoke timeout.
-    request-timeout = 90s
+    request-timeout = 65s
 
 Review comment:
   system.basic.WskRestSequenceTests.Wsk Sequence should execute a sequence in blocking fashion and finish execution even if longer than blocking response timeout
   
   This test case sets the payload exactly to 65000 milliseconds. I am not sure request-timeout = 65s is sufficient for this test case to complete.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services