Skip to content

Detect duplicate remote-logging schemes in provider.yaml check - #70921

Merged
potiuk merged 1 commit into
apache:mainfrom
jason810496:ci/logging/detect-remote-logging-scheme-collisions
Aug 3, 2026
Merged

Detect duplicate remote-logging schemes in provider.yaml check#70921
potiuk merged 1 commit into
apache:mainfrom
jason810496:ci/logging/detect-remote-logging-scheme-collisions

Conversation

@jason810496

@jason810496 jason810496 commented Aug 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

What

Each provider can register a remote-logging handler in its provider.yaml:

remote-logging:
  - classpath: airflow.providers.amazon.aws.log.s3_task_handler.S3RemoteLogIO
    scheme: s3
  • Please note that the runtime did handle the collisions.
  • This PR just added a static detector to prevent further schema collisions in provider (CI-tooling change only and no user-facing behaviour changes)
Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below) With the help of Opus 4.8 following the guidelines

@jason810496 jason810496 self-assigned this Aug 2, 2026
@jason810496
jason810496 marked this pull request as ready for review August 3, 2026 09:02
@jason810496 jason810496 removed the backport-to-v3-3-test Backport to v3-3-test label Aug 3, 2026
@jason810496

jason810496 commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

This is a static CI for provider so no need to backport.

@potiuk
potiuk merged commit 32be8fa into apache:main Aug 3, 2026
304 checks passed
@jason810496
jason810496 deleted the ci/logging/detect-remote-logging-scheme-collisions branch August 5, 2026 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@potiuk potiuk potiuk approved these changes
@ashb ashb Awaiting requested review from ashb ashb is a code owner
@gopidesupavan gopidesupavan Awaiting requested review from gopidesupavan gopidesupavan is a code owner
@amoghrajesh amoghrajesh Awaiting requested review from amoghrajesh amoghrajesh is a code owner
@jscheffl jscheffl Awaiting requested review from jscheffl jscheffl is a code owner
@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 is a code owner

Assignees

@jason810496 jason810496

Labels

area:dev-tools

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jason810496 @potiuk