[SPARK-53545] Enforce style check at project level for operator

### What changes were proposed in this pull request?

This PR enforces 'misc' spotless check at project level instead of submodules

### Why are the changes needed?

In this way, style checks are enforced for not only files inside submodules, but also for project files including helm and e2e test

### Does this PR introduce any user-facing change?

No

### How was this patch tested?

CIs

### Was this patch authored or co-authored using generative AI tooling?

No

Closes apache#315 from jiangzho/style.

Authored-by: Zhou JIANG <[email protected]>
Signed-off-by: Peter Toth <[email protected]>

Author: jiangzho

.github/workflows/build_and_test.yml

@@ -131,7 +131,7 @@ jobs:
           minikube docker-env --unset
       - name: Run E2E Test with Dynamic Configuration Disabled
         if: matrix.mode == 'static'
-        run: |        
+        run: |
           chainsaw test --test-dir ./tests/e2e/${{ matrix.test-group }} --parallel 2
       - name: Run Spark K8s Operator on K8S with Dynamic Configuration Enabled
         if: matrix.mode == 'dynamic'

build.gradle

@@ -116,11 +116,15 @@ subprojects {
       trimTrailingWhitespace()
       removeUnusedImports()
     }
-    format 'misc', {
-      target '*.md', '*.gradle', '**/*.properties', '**/*.xml', '**/*.yaml', '**/*.yml'
-      endWithNewline()
-      trimTrailingWhitespace()
-    }
+  }
+}
+
+apply plugin: 'com.diffplug.spotless'
+spotless {
+  format 'misc', {
+    target '*.md', '*.gradle', '**/*.properties', '**/*.xml', '**/*.yaml', '**/*.yml'
+    endWithNewline()
+    trimTrailingWhitespace()
   }
 }
 

tests/e2e/assertions/spark-application/spark-state-transition.yaml

@@ -29,4 +29,4 @@ status:
       - "DriverReady"
       - "RunningHealthy"
       - "Succeeded"
-      - "ResourceReleased"
+      - "ResourceReleased"

tests/e2e/assertions/spark-driver-config-map.yaml

@@ -18,4 +18,4 @@ apiVersion: v1
 kind: ConfigMap
 metadata:
   name: ($SPARK_DRIVER_CONFIG_MAP_NAME)
-  namespace: default
+  namespace: default

tests/e2e/assertions/spark-driver-pod.yaml

@@ -27,4 +27,3 @@ spec:
   (volumes[?configMap.name=='spark-conf-volume-driver'] != []): true
 status:
   phase: Running
-

tests/e2e/helm/dynamic-config-values-2.yaml

@@ -36,4 +36,4 @@ operatorRbac:
   clusterRole:
     name: "spark-operator-clusterrole-2"
   clusterRoleBinding:
-    name: "spark-operator-clusterrolebinding-2"
+    name: "spark-operator-clusterrolebinding-2"

tests/e2e/python/pyspark-example.yaml

@@ -28,4 +28,4 @@ spec:
     spark.kubernetes.container.image: ($IMAGE)
     spark.kubernetes.authenticate.driver.serviceAccountName: "spark"
   runtimeVersions:
-    sparkVersion: ($V_SPARK_VERSION)
+    sparkVersion: ($V_SPARK_VERSION)

tests/e2e/resource-retain-duration/chainsaw-test.yaml

@@ -61,4 +61,4 @@ spec:
             value: ($SPARK_APPLICATION_NAME)
         timeout: 120s
         content: |
-          kubectl delete sparkapplication $SPARK_APPLICATION_NAME --ignore-not-found=true 
+          kubectl delete sparkapplication $SPARK_APPLICATION_NAME --ignore-not-found=true