Skip to content

Aggregate CI-image dependency groups #67130

Merged
potiuk merged 1 commit into
apache:mainfrom
bugraoz93:manage-extra-groups-from-toml
May 18, 2026
Merged

Aggregate CI-image dependency groups #67130
potiuk merged 1 commit into
apache:mainfrom
bugraoz93:manage-extra-groups-from-toml

Conversation

@bugraoz93
Copy link
Copy Markdown
Contributor

@bugraoz93 bugraoz93 commented May 18, 2026 •
edited
Loading

Providers can register non-default extras with a one-line change

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:dev-tools area:production-image Production image improvements and fixes backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch labels May 18, 2026
@bugraoz93 bugraoz93 changed the title Aggregate CI-image dependency groups so providers can register non-de… Aggregate CI-image dependency groups May 18, 2026
@bugraoz93 bugraoz93 added the full tests needed We need to run full set of tests for this PR to merge label May 18, 2026
@bugraoz93 bugraoz93 closed this May 18, 2026
@bugraoz93 bugraoz93 reopened this May 18, 2026
@bugraoz93
Copy link
Copy Markdown
Contributor Author

bugraoz93 commented May 18, 2026

Maybe solution to https://apache-airflow.slack.com/archives/C06K9Q5G2UA/p1779123673073269
Would be great to get your take @potiuk on whether the place looks good to manage these, or would you have any concern/dependencies that could break?

potiuk
potiuk approved these changes May 18, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@potiuk potiuk 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
Copy link
Copy Markdown
Member

potiuk commented May 18, 2026

Maybe solution to https://apache-airflow.slack.com/archives/C06K9Q5G2UA/p1779123673073269 Would be great to get your take @potiuk on whether the place looks good to manage these, or would you have any concern/dependencies that could break?

Good idae. Looked at it - I don't think there are any concerns:

  • the change only affects "sources" build - not installing from URL or Pip
  • It's only really used in CI-image - though PROD build locally built by breeze also uses ci-image group (but that's fine)

@potiuk potiuk merged commit e61640e into apache:main May 18, 2026
320 of 359 checks passed
@potiuk potiuk added this to the Airflow 3.2.2 milestone May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

Backport successfully created: v3-2-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
✅ v3-2-test PR Link

@vatsrahul1001 vatsrahul1001 mentioned this pull request May 19, 2026
Merged
@bugraoz93 bugraoz93 mentioned this pull request May 19, 2026
Open
1 task
@jscheffl jscheffl mentioned this pull request May 19, 2026
Open
vatsrahul1001 pushed a commit that referenced this pull request May 20, 2026
@bugraoz93 @vatsrahul1001
[v3-2-test] Aggregate CI-image dependency groups so providers can reg…
5dd0149 
…ister non-default extras with a one-line change (#67130)

(cherry picked from commit e61640e)

Co-authored-by: Bugra Ozturk <bugraoz93@users.noreply.github.com>
vatsrahul1001 pushed a commit that referenced this pull request May 21, 2026
@bugraoz93 @vatsrahul1001
[v3-2-test] Aggregate CI-image dependency groups so providers can reg…
2fa00e2 
…ister non-default extras with a one-line change (#67130)

(cherry picked from commit e61640e)

Co-authored-by: Bugra Ozturk <bugraoz93@users.noreply.github.com>
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

@jason810496 jason810496 Awaiting requested review from jason810496 jason810496 is a code owner

@jedcunningham jedcunningham Awaiting requested review from jedcunningham jedcunningham is a code owner

@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy is a code owner

@choo121600 choo121600 Awaiting requested review from choo121600 choo121600 is a code owner

@vatsrahul1001 vatsrahul1001 Awaiting requested review from vatsrahul1001 vatsrahul1001 is a code owner

Assignees

No one assigned

Labels

area:dev-tools area:production-image Production image improvements and fixes backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Milestone

Airflow 3.2.2

Development

Successfully merging this pull request may close these issues.

2 participants

@bugraoz93 @potiuk