Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22760#discussion_r230223741
  
    --- 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
    --- End diff --
    
    Unnecessary comment.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to