Skip to content

test(k8s): remove duplicate args in pytest runs of 'kubernetes_tests'#48616

Merged
potiuk merged 1 commit intoapache:mainfrom
insomnes:unique-k8s-pytest-args
Apr 1, 2025
Merged

test(k8s): remove duplicate args in pytest runs of 'kubernetes_tests'#48616
potiuk merged 1 commit intoapache:mainfrom
insomnes:unique-k8s-pytest-args

Conversation

@insomnes
Copy link
Contributor

@insomnes insomnes commented Apr 1, 2025

Because of the copy and adding defaults some pytest args may be duplicated in kubernetes_tests helper runs.

This PR ensures the uniqueness of args on the level of the _run_tests function to help lower the visual clutter.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
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.

@insomnes insomnes force-pushed the unique-k8s-pytest-args branch from 69b816f to 659ee1d Compare April 1, 2025 10:32
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@potiuk potiuk merged commit 8d4977a into apache:main Apr 1, 2025
91 checks passed
diogotrodrigues pushed a commit to diogotrodrigues/airflow that referenced this pull request Apr 6, 2025
simonprydden pushed a commit to simonprydden/airflow that referenced this pull request Apr 8, 2025
@insomnes insomnes deleted the unique-k8s-pytest-args branch April 18, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants