Yikun opened a new pull request #35299: URL: https://github.com/apache/spark/pull/35299
### What changes were proposed in this pull request? This reverts commit c4a9772f741836cdb399e35a45b3b5df48d9eea6 which is merged in https://github.com/apache/spark/pull/34983 . ### Why are the changes needed? Before this revert, there are two expected behavior: 1. **Can't submit application with specified namespace** the driver pod would be stuck in `ContainerCreating` due to `MountVolume.SetUp failed for volume "spark-conf-volume-driver" : configmap "spark-drv-ed33d87e8b2eb017-conf-map" not found`, because the configFileMap doesn't contains 2. **`configFileMap` doesn't allow `namespace` key as confimap.data** (but kubernetes doesn't have this limit). The configmap of executor is still set correctly after revert this patch (the original PR solved case): ```scala // Create configmap in ns2 namespace val builder2 = new ConfigMapBuilder() .withNewMetadata() // We don't specify ns2 in here like `.withNamespace("ns2")` .withName("configmap2") .endMetadata() .addToData("testkey", "testvalue") .build() val configMap2 = client.inNamespace("ns2").resource(builder2).createOrReplace() // the namespace is still set to ns2 as expected. assert(configMap2.getMetadata.getNamespace === "ns2") ``` That is even if we don't need to explicitly set the namespace for configmap.metadata, it will set correctly. That means we don't need the original PR. ### Does this PR introduce _any_ user-facing change? NO ### How was this patch tested? - Manual test - existing K8S UT -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
