markusthoemmes commented on a change in pull request #3788: Store and pass
variant data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r196834822
##########
File path:
core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala
##########
@@ -354,16 +354,19 @@ class ContainerProxy(
.flatMap { initInterval =>
val parameters = job.msg.content getOrElse JsObject()
+ val authkey = job.msg.user.authkey.toEnvironment
+
val environment = JsObject(
- "api_key" -> job.msg.user.authkey.compact.toJson,
"namespace" -> job.msg.user.namespace.name.toJson,
"action_name" -> job.msg.action.qualifiedNameWithLeadingSlash.toJson,
"activation_id" -> job.msg.activationId.toString.toJson,
// compute deadline on invoker side avoids discrepancies inside
container
// but potentially under-estimates actual deadline
"deadline" -> (Instant.now.toEpochMilli +
actionTimeout.toMillis).toString.toJson)
- container.run(parameters, environment,
actionTimeout)(job.msg.transid).map {
+ logging.info(this, s"*******MH env JSON: ${JsObject(authkey.fields ++
environment.fields)}")
Review comment:
Please remove this logline.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
