Github user liyinan926 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21669#discussion_r223851661
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/hadooputils/HadoopKerberosLogin.scala
---
@@ -48,10 +48,8 @@ private[spark] object HadoopKerberosLogin {
submissionSparkConf,
hadoopConf)
require(tokenData.nonEmpty, "Did not obtain any delegation tokens")
- val currentTime = tokenManager.getCurrentTime
- val initialTokenDataKeyName =
s"$KERBEROS_SECRET_KEY_PREFIX-$currentTime"
- val newSecretName =
-
s"$kubernetesResourceNamePrefix-$KERBEROS_DELEGEGATION_TOKEN_SECRET_NAME.$currentTime"
+ val initialTokenDataKeyName = KERBEROS_SECRET_KEY_PREFIX
--- End diff --
Nit: you don't need this variable and you can rename
`KERBEROS_SECRET_KEY_PREFIX` to `KERBEROS_SECRET_KEY`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
