Skip to content

Catch Exception in git hook instantiation #55079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jedcunningham merged 1 commit into from
Aug 30, 2025
Merged

Catch Exception in git hook instantiation #55079

jedcunningham merged 1 commit into from
Aug 30, 2025

Conversation

@dstandish
Copy link
Contributor

@dstandish dstandish commented Aug 29, 2025

It won't always be AirflowException here. It could be ImportError, or other such things. Not catching an exception here could cause the dag processor to halt.

@dstandish
Catch Exception in git hook instantiation
a701b15 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
amoghrajesh
amoghrajesh approved these changes Aug 29, 2025
View reviewed changes
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.

Sounds good

@jedcunningham jedcunningham merged commit c5ccfd1 into apache:main Aug 30, 2025
109 checks passed
@jedcunningham jedcunningham deleted the broader-exc-for-git-bundle branch August 30, 2025 00:49
mangal-vairalkar pushed a commit to mangal-vairalkar/airflow that referenced this pull request Aug 30, 2025
@dstandish @mangal-vairalkar
Catch Exception in git hook instantiation (apache#55079)
f96f45f 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
nothingmin pushed a commit to nothingmin/airflow that referenced this pull request Sep 2, 2025
@dstandish @nothingmin
Catch Exception in git hook instantiation (apache#55079)
f858345 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
@eladkal eladkal mentioned this pull request Sep 5, 2025
Closed
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Sep 30, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
370b8c9 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 1, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
52aa9a9 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 2, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
8e4b1ec 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 3, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
79da86f 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 4, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
f4d81a8 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
de2875d 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 5, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
6a94458 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 7, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
873e280 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 8, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
995367c 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 9, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
0b3393c 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 10, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
d879ed0 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 11, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
ad45d03 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 12, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
329901e 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 14, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
0cf399e 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 15, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
34f4812 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 17, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
d3d2a50 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
abdulrahman305 bot pushed a commit to abdulrahman305/airflow that referenced this pull request Oct 19, 2025
@dstandish
Catch Exception in git hook instantiation (apache#55079)
cb82504 
It won't always be AirflowException here.  It could be ImportError, or other such things.  Not catching an exception here could cause the dag processor to halt.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ephraimbuddy ephraimbuddy ephraimbuddy approved these changes

@amoghrajesh amoghrajesh amoghrajesh approved these changes

@jedcunningham jedcunningham jedcunningham approved these changes

Assignees

No one assigned

Labels

area:providers provider:git

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dstandish @ephraimbuddy @amoghrajesh @jedcunningham