jiangzho opened a new pull request, #414:
URL: https://github.com/apache/spark-kubernetes-operator/pull/414
### What changes were proposed in this pull request?
Enhance existing test assertGeneratedCRDMatchesHelmChart to give diff upon
failure
### Why are the changes needed?
For better readability.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Self-contained upon success & manually tested failure scenario - e.g. remove
some fields from CRD inhelm repo, the test would fail with message like
```
* What went wrong:
Execution failed for task
':spark-operator-api:assertGeneratedCRDMatchesHelmChart'.
> Generated CRD yaml does not match the staged version in Helm Chart, please
keep the chart updated.
=== SparkApplication CRD Differences ===
--- /dev/fd/63 2025-10-31 14:33:55
+++ /dev/fd/62 2025-10-31 14:33:55
@@ -6355,11 +6355,6 @@
properties:
currentAttemptSummary:
properties:
- attemptInfo:
- properties:
- id:
- type: integer
- type: object
stateTransitionHistory:
additionalProperties:
properties:
=== SparkCluster CRD Differences ===
--- /dev/fd/63 2025-10-31 14:33:55
+++ /dev/fd/62 2025-10-31 14:33:55
@@ -7218,8 +7218,6 @@
type: string
lastTransitionTime:
type: string
- message:
- type: string
type: object
type: object
type: object
```
Which gives a more clear indication about the exact failure
### Was this patch authored or co-authored using generative AI tooling?
No
--
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]
