Commit 1aef8b7
[SPARK-40869][K8S] Resource name prefix should not start with a hyphen
### What changes were proposed in this pull request?
Strip leading - from resource name prefix
### Why are the changes needed?
leading - are not allowed for resource name prefix (especially spark.kubernetes.executor.podNamePrefix)
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Unit test
Closes apache#38331 from tobiasstadler/fix-SPARK-40869.
Lead-authored-by: Tobias Stadler <[email protected]>
Co-authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit 7f3b598)
Signed-off-by: Dongjoon Hyun <[email protected]>1 parent b9d22ac commit 1aef8b7
File tree
2 files changed
+5
-0
lines changed- resource-managers/kubernetes/core/src
- main/scala/org/apache/spark/deploy/k8s
- test/scala/org/apache/spark/deploy/k8s
2 files changed
+5
-0
lines changedLines changed: 1 addition & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
233 | 233 | | |
234 | 234 | | |
235 | 235 | | |
| 236 | + | |
236 | 237 | | |
237 | 238 | | |
238 | 239 | | |
| |||
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
178 | 178 | | |
179 | 179 | | |
180 | 180 | | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
181 | 185 | | |
0 commit comments