Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22760#discussion_r230223616
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/KubernetesFeaturesTestUtils.scala
---
@@ -63,4 +63,54 @@ object KubernetesFeaturesTestUtils {
def containerHasEnvVar(container: Container, envVarName: String):
Boolean = {
container.getEnv.asScala.exists(envVar => envVar.getName == envVarName)
}
+
+ def containerHasEnvVars(container: Container, envs: Map[String,
String]): Boolean = {
+ envs.toSet.subsetOf(container.getEnv.asScala
+ .map { e => (e.getName, e.getValue)}.toSet)
--- End diff --
space before `}`
Also in other places.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
