Skip to content

[AIP-94] Create a CLI airflowctl client and adopt it in existing commands#68175

Merged
bugraoz93 merged 7 commits into
apache:mainfrom
bugraoz93:64924
Jun 11, 2026
Merged

[AIP-94] Create a CLI airflowctl client and adopt it in existing commands#68175
bugraoz93 merged 7 commits into
apache:mainfrom
bugraoz93:64924

Conversation

@bugraoz93

@bugraoz93 bugraoz93 commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

closes: #64924

Deprecation of the local bespoke api client replaced with airflowctl.
Existing members are migrated.
This should enable migrating other commands easily.


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

  • 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.

@bugraoz93 bugraoz93 changed the title Create a CLI airflowctl client and adopt it in existing commands [AIP-94] Create a CLI airflowctl client and adopt it in existing commands Jun 7, 2026
@bugraoz93 bugraoz93 added full tests needed We need to run full set of tests for this PR to merge and removed backport-to-v3-2-test labels Jun 7, 2026
@bugraoz93 bugraoz93 closed this Jun 7, 2026
@bugraoz93 bugraoz93 reopened this Jun 7, 2026
@bugraoz93 bugraoz93 added this to the Airflow 3.3.0 milestone Jun 7, 2026
Comment thread airflow-core/src/airflow/cli/api_client.py
@jscheffl

jscheffl commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

One question: Was it not planned also to add a deprecation note that in future these commands will be removed and be an alias only? Would it be beneficial when switching to add the deprecation note early?

@bugraoz93

Copy link
Copy Markdown
Contributor Author

One question: Was it not planned also to add a deprecation note that in future these commands will be removed and be an alias only? Would it be beneficial when switching to add the deprecation note early?

Great point Jens! Many thanks for the review! Will address this before merging :)

@bugraoz93

Copy link
Copy Markdown
Contributor Author
Screenshot from 2026-06-08 20-45-12 Screenshot from 2026-06-08 20-45-05 Screenshot from 2026-06-08 20-44-43

Some of local tests

One question: Was it not planned also to add a deprecation note that in future these commands will be removed and be an alias only? Would it be beneficial when switching to add the deprecation note early?

Added with the airflowctl command replacement in the log to see and get used to them :)

@bugraoz93

Copy link
Copy Markdown
Contributor Author

CI was green before rebase
Should be fixed in #68300

@bugraoz93

Copy link
Copy Markdown
Contributor Author

UI go wrong and merged the changes rather rebase. I will rebase before merging. If no objection will merge today

@bugraoz93 bugraoz93 merged commit a0be5d8 into apache:main Jun 11, 2026
145 checks passed
imrichardwu pushed a commit to imrichardwu/airflow that referenced this pull request Jun 16, 2026
…ands (apache#68175)

* Route airflow dags/pools/assets CLI commands through the API server via the airflowctl client

* Add auth-manager get_cli_user tests and tidy CLI migration newsfragment

* Rename significant with PR number

* Add deprecation warnings with airflowctl command replacement

* Fix failure when the auth manager isn't initialized

* Clarify deprecation wording for migrated airflow CLI commands

* Fix airflow assets list with watchers and CLI migration test failures
dingo4dev pushed a commit to dingo4dev/airflow that referenced this pull request Jun 16, 2026
…ands (apache#68175)

* Route airflow dags/pools/assets CLI commands through the API server via the airflowctl client

* Add auth-manager get_cli_user tests and tidy CLI migration newsfragment

* Rename significant with PR number

* Add deprecation warnings with airflowctl command replacement

* Fix failure when the auth manager isn't initialized

* Clarify deprecation wording for migrated airflow CLI commands

* Fix airflow assets list with watchers and CLI migration test failures
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:CLI area:dev-tools area:providers full tests needed We need to run full set of tests for this PR to merge provider:fab provider:keycloak

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AIP-94] Initiate airflowctl as a client in the airflow CLI

2 participants