mhenke1 commented on a change in pull request #3752: Move uuid into namespace 
to decouple it from authkey
URL: 
https://github.com/apache/incubator-openwhisk/pull/3752#discussion_r194765182
 
 

 ##########
 File path: tests/src/test/scala/whisk/core/limits/ActionLimitsTests.scala
 ##########
 @@ -245,7 +245,7 @@ class ActionLimitsTests extends TestHelpers with 
WskTestHelpers {
     val allowedSize = ActivationEntityLimit.MAX_ACTIVATION_ENTITY_LIMIT.toBytes
 
     // Needs some bytes grace since activation message is not only the payload.
-    val args = Map("p" -> ("a" * (allowedSize - 700).toInt).toJson)
+    val args = Map("p" -> ("a" * (allowedSize - 750).toInt).toJson)
 
 Review comment:
   Not really. Serializing the uuid as an additional part of the namespace has 
added some crucial bytes to the message so that the "grace bytes" where not 
enough to get the message passed.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to