Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/22760#discussion_r230223780 --- 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) + } + + def containerHasVolumeMounts(container: Container, vms: Map[String, String]): Boolean = { + vms.toSet.subsetOf(container.getVolumeMounts.asScala + .map { vm => (vm.getName, vm.getMountPath)}.toSet) + } + + def podHasLabels(pod: Pod, labels: Map[String, String]): Boolean = { + labels.toSet.subsetOf(pod.getMetadata.getLabels.asScala.toSet) + } + + def podHasVolumes(pod: Pod, volumes: Seq[Volume]): Boolean = { + volumes.toSet.subsetOf(pod.getSpec.getVolumes.asScala.toSet) + } + + // Kerberos Specific Test utils + + // Upon use of bootstrapHadoopConfDir --- End diff -- Not sure I understand what this comment means.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org