erikerlandson commented on a change in pull request #24879: [SPARK-28042][K8S]
Support using volume mount as local storage
URL: https://github.com/apache/spark/pull/24879#discussion_r307442258
##########
File path:
resource-managers/kubernetes/core/src/test/scala/org/apache/spark/deploy/k8s/features/LocalDirsFeatureStepSuite.scala
##########
@@ -116,4 +115,33 @@ class LocalDirsFeatureStepSuite extends SparkFunSuite {
.withValue(defaultLocalDir)
.build())
}
+
+ test("local dir on mounted volume") {
+ val volumeConf = KubernetesVolumeSpec(
+ "spark-local-dir-test",
+ "/tmp",
+ "",
+ false,
+ KubernetesHostPathVolumeConf("/hostPath/tmp")
+ )
+ val mountVolumeConf = KubernetesTestConf.createDriverConf(volumes =
Seq(volumeConf))
+ val mountVolumeStep = new MountVolumesFeatureStep(mountVolumeConf)
+ val configuredPod = mountVolumeStep.configurePod(SparkPod.initialPod())
+
+ val sparkConf = new SparkConfWithEnv(Map())
+ val localDirConf = KubernetesTestConf.createDriverConf(sparkConf)
Review comment:
That is the idea, to keep building out one conf. (There were some early
proposals around making these stages "more functional" but those were
considered less broadly familiar and less spark idiomatic)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]