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

    https://github.com/apache/spark/pull/19437#discussion_r143221450
  
    --- Diff: 
resource-managers/mesos/src/test/scala/org/apache/spark/scheduler/cluster/mesos/Utils.scala
 ---
    @@ -105,4 +108,108 @@ object Utils {
       def createTaskId(taskId: String): TaskID = {
         TaskID.newBuilder().setValue(taskId).build()
       }
    +
    +  def configEnvBasedRefSecrets(secretConfig: MesosSecretConfig): 
Map[String, String] = {
    +    val secretName = "/path/to/secret,/anothersecret"
    +    val envKey = "SECRET_ENV_KEY,PASSWORD"
    +    Map(
    +      secretConfig.SECRET_NAME.key -> secretName,
    +      secretConfig.SECRET_ENVKEY.key -> envKey
    +    )
    +  }
    +
    +  def verifyEnvBasedRefSecrets(launchedTasks: List[TaskInfo]): Unit = {
    +    val envVars = launchedTasks.head
    +      .getCommand
    +      .getEnvironment
    +      .getVariablesList
    +      .asScala
    +    assert(envVars
    +      .filter(!_.getName.startsWith("SPARK_")).length == 2)  // 
user-defined secret env vars
    --- End diff --
    
    assert(envVars.count(!_.getName.startsWith("SPARK_")) == 2)


---

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

Reply via email to