Skip to content

Revert "fix(glue): Fix GlueJobOperator verbose logs not showing in deferrable mode (#63086)#64340

Merged
o-nikolas merged 1 commit intoapache:mainfrom
aws-mwaa:onioklas/revert_glue_trigger_verbose_logs
Mar 28, 2026
Merged

Revert "fix(glue): Fix GlueJobOperator verbose logs not showing in deferrable mode (#63086)#64340
o-nikolas merged 1 commit intoapache:mainfrom
aws-mwaa:onioklas/revert_glue_trigger_verbose_logs

Conversation

@o-nikolas
Copy link
Copy Markdown
Contributor

This reverts commit b086a22.

#63086 is causing PR tests (both static and providers) to fail. Example:

[1](https://github.com/apache/airflow/actions/runs/23673010180/job/68971470643?pr=64187#step:6:162)
Run 'ruff' for extremely fast Python linting..............................................................Failed
- hook id: ruff
- exit code: 1

  F821 Undefined name `AirflowException`
     --> providers/amazon/tests/unit/amazon/aws/triggers/test_glue.py:196:28
      |
  194 |             waiter_max_attempts=5,
  195 |         )
  196 |         with pytest.raises(AirflowException, match="FAILED"):
      |                            ^^^^^^^^^^^^^^^^
  197 |             async for _ in trigger.run():
  198 |                 pass
      |

  F821 Undefined name `AirflowException`
     --> providers/amazon/tests/unit/amazon/aws/triggers/test_glue.py:225:28
      |
  223 |             waiter_max_attempts=2,
  224 |         )
  225 |         with pytest.raises(AirflowException, match="max attempts"):
      |                            ^^^^^^^^^^^^^^^^
  226 |             async for _ in trigger.run():
  227 |                 pass
      |

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg bot added area:providers provider:amazon AWS/Amazon - related issues labels Mar 28, 2026
Copy link
Copy Markdown
Contributor

@ferruzzi ferruzzi left a comment

Choose a reason for hiding this comment

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

approved to fix main

@o-nikolas o-nikolas merged commit 48685e7 into apache:main Mar 28, 2026
96 of 97 checks passed
@o-nikolas o-nikolas deleted the onioklas/revert_glue_trigger_verbose_logs branch March 28, 2026 02:06
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Mar 30, 2026
Suraj-kumar00 pushed a commit to Suraj-kumar00/airflow that referenced this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants