Skip to content

Fix Celery worker JSON logging config for teams - #69139

Merged
hussein-awala merged 1 commit into
apache:mainfrom
henry3260:fix-celery-multi-conf-ci-error
Jun 29, 2026
Merged

Fix Celery worker JSON logging config for teams#69139
hussein-awala merged 1 commit into
apache:mainfrom
henry3260:fix-celery-multi-conf-ci-error

Conversation

@henry3260

@henry3260 henry3260 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

We should use config instead of conf

Error:
https://github.com/apache/airflow/actions/runs/28381801726

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.

@hussein-awala hussein-awala left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I closed and reopened the PR to run all the tests.

I wonder why the CI check which failed doesn't run when we change the providers code source, we need to change that in a separate PR.

I'll wait for the CI checks before merging

@hussein-awala
hussein-awala merged commit 926a8c6 into apache:main Jun 29, 2026
219 of 220 checks passed
@henry3260
henry3260 deleted the fix-celery-multi-conf-ci-error branch June 30, 2026 01:13
karenbraganz pushed a commit to karenbraganz/airflow that referenced this pull request Jun 30, 2026
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