Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Apr 23, 2025


^ 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.

@potiuk potiuk requested review from amoghrajesh, eladkal and kaxil April 23, 2025 11:12
@potiuk potiuk force-pushed the fail-script-after-failing-a-number-of-attempts branch from 35b1dee to ea02349 Compare April 23, 2025 11:12
@potiuk potiuk force-pushed the fail-script-after-failing-a-number-of-attempts branch from ea02349 to 36e321b Compare April 23, 2025 11:33
@kaxil kaxil changed the title Actually fail when number of attempts failed for breeze command Fail when number of attempts failed for breeze command Apr 23, 2025
@potiuk potiuk merged commit c0f2ce1 into apache:main Apr 23, 2025
42 checks passed
@potiuk potiuk deleted the fail-script-after-failing-a-number-of-attempts branch April 23, 2025 11:39
potiuk added a commit to potiuk/airflow that referenced this pull request Apr 23, 2025
…ommand (apache#49615)

(cherry picked from commit c0f2ce1)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
potiuk added a commit that referenced this pull request Apr 23, 2025
Copy link
Contributor

@amoghrajesh amoghrajesh 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 added a commit to potiuk/airflow that referenced this pull request Apr 23, 2025
…ommand (apache#49615)

(cherry picked from commit c0f2ce1)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
prabhusneha pushed a commit to astronomer/airflow that referenced this pull request Apr 25, 2025
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.

3 participants