Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22760#discussion_r227065151
--- Diff:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/KubernetesFeaturesTestUtils.scala
---
@@ -63,4 +63,40 @@ object KubernetesFeaturesTestUtils {
def containerHasEnvVar(container: Container, envVarName: String):
Boolean = {
container.getEnv.asScala.exists(envVar => envVar.getName == envVarName)
}
+
+ def podHasLabels(pod: Pod, labels: Map[String, String]): Boolean = {
+ labels.toSet.subsetOf(pod.getMetadata.getLabels.asScala.toSet)
+ }
+
+ // Kerberos Specific Test utils
+
+ // Upon use of bootstrapHadoopConfDir
+ def hConfBootPod(inputPod: SparkPod): SparkPod =
--- End diff --
Can we just name it `hadoopConfBootPod`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
