mohamed-mis opened a new pull request, #56063:
URL: https://github.com/apache/airflow/pull/56063
This PR addresses an issue in the CustomObjectLauncher operator within the
Kubernetes provider, where submitting a SparkApplication manifest -using the
SparkKubernetesOperator- results in the loss of existing metadata labels and
annotations.
The current implementation in get_body() overwrites the entire metadata
object:
`self.body.metadata = {"name": self.name, "namespace": self.namespace}
`
This discards any pre-existing metadata such as labels, annotations, or
custom fields.
This PR replaces the overwrite with an in-place update using dict.update().
**Testing:**
This fix has been validated with the following setup:
- Airflow Version: 3.0.6
- Deployment Method: Official Helm chart 1.18.0
- Kubernetes Provider Version:
apache-airflow-providers-cncf-kubernetes==10.7.0
- Python Version: 3.12.11
SparkApplication resources submitted via the SparkKubernetesOperator now
retain their original metadata labels as expected.
---
**^ Add meaningful description above**
Read the **[Pull Request
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
for more information.
In case of fundamental code changes, an Airflow Improvement Proposal
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party
License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in a
newsfragment file, named `{pr_number}.significant.rst` or
`{issue_number}.significant.rst`, in
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
--
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]
