This is an automated email from the ASF dual-hosted git repository.

dongjoon-hyun pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/spark-kubernetes-operator.git


The following commit(s) were added to refs/heads/main by this push:
     new 9209d2d6 [SPARK-59957] Use `spark` for `meta.helm.sh/release-name` of 
the dynamic config E2E fixtures
9209d2d6 is described below

commit 9209d2d6485b11b6c3fb8c1634d2ebd3de4daab6
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Fri Oct 2 12:43:02 2026 -0700

    [SPARK-59957] Use `spark` for `meta.helm.sh/release-name` of the dynamic 
config E2E fixtures
    
    ### What changes were proposed in this pull request?
    
    This PR aims to use `spark` for `meta.helm.sh/release-name` of the dynamic 
configuration `ConfigMap` fixtures of `watched-namespaces` and 
`watched-namespaces-file` E2E tests.
    
    ### Why are the changes needed?
    
    SPARK-53639 
([#323](https://github.com/apache/spark-kubernetes-operator/pull/323)) missed 
these fixtures. Since the `ConfigMap` is kept by `helm.sh/resource-policy: 
keep`, the stale annotation makes a later `helm install spark` fail on the same 
cluster.
    
    ```
    Error: INSTALLATION FAILED: ... invalid ownership metadata; annotation 
validation error: key "meta.helm.sh/release-name" must equal "spark": current 
value is "spark-kubernetes-operator"
    ```
    
    ### Does this PR introduce _any_ user-facing change?
    
    No. This is a test-only change.
    
    ### How was this patch tested?
    
    Pass the CIs.
    
    ### Was this patch authored or co-authored using generative AI tooling?
    
    Generated-by: Claude Opus 5.5
    
    Closes #927 from dongjoon-hyun/SPARK-59957.
    
    Authored-by: Dongjoon Hyun <[email protected]>
    Signed-off-by: Dongjoon Hyun <[email protected]>
---
 tests/e2e/watched-namespaces-file/spark-operator-dynamic-config-1.yaml | 2 +-
 tests/e2e/watched-namespaces/spark-operator-dynamic-config-1.yaml      | 2 +-
 tests/e2e/watched-namespaces/spark-operator-dynamic-config-2.yaml      | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/tests/e2e/watched-namespaces-file/spark-operator-dynamic-config-1.yaml 
b/tests/e2e/watched-namespaces-file/spark-operator-dynamic-config-1.yaml
index 791bb642..e44d2f60 100644
--- a/tests/e2e/watched-namespaces-file/spark-operator-dynamic-config-1.yaml
+++ b/tests/e2e/watched-namespaces-file/spark-operator-dynamic-config-1.yaml
@@ -23,7 +23,7 @@ kind: ConfigMap
 metadata:
   annotations:
     helm.sh/resource-policy: keep
-    meta.helm.sh/release-name: spark-kubernetes-operator
+    meta.helm.sh/release-name: spark
     meta.helm.sh/release-namespace: default
   labels:
     app.kubernetes.io/component: operator-dynamic-config-overrides
diff --git a/tests/e2e/watched-namespaces/spark-operator-dynamic-config-1.yaml 
b/tests/e2e/watched-namespaces/spark-operator-dynamic-config-1.yaml
index d0fe2758..bc944616 100644
--- a/tests/e2e/watched-namespaces/spark-operator-dynamic-config-1.yaml
+++ b/tests/e2e/watched-namespaces/spark-operator-dynamic-config-1.yaml
@@ -22,7 +22,7 @@ kind: ConfigMap
 metadata:
   annotations:
     helm.sh/resource-policy: keep
-    meta.helm.sh/release-name: spark-kubernetes-operator
+    meta.helm.sh/release-name: spark
     meta.helm.sh/release-namespace: default
   labels:
     app.kubernetes.io/component: operator-dynamic-config-overrides
diff --git a/tests/e2e/watched-namespaces/spark-operator-dynamic-config-2.yaml 
b/tests/e2e/watched-namespaces/spark-operator-dynamic-config-2.yaml
index 112a9ce5..8d4235c9 100644
--- a/tests/e2e/watched-namespaces/spark-operator-dynamic-config-2.yaml
+++ b/tests/e2e/watched-namespaces/spark-operator-dynamic-config-2.yaml
@@ -22,7 +22,7 @@ kind: ConfigMap
 metadata:
   annotations:
     helm.sh/resource-policy: keep
-    meta.helm.sh/release-name: spark-kubernetes-operator
+    meta.helm.sh/release-name: spark
     meta.helm.sh/release-namespace: default
   labels:
     app.kubernetes.io/component: operator-dynamic-config-overrides


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to