Skip to content

[AIP-94] Mark dags list-import-errors as migrated to airflowctl - #68602

Merged
potiuk merged 1 commit into
apache:mainfrom
yuseok89:migrate-dags-list-import-errors-to-airflowctl
Jul 4, 2026
Merged

[AIP-94] Mark dags list-import-errors as migrated to airflowctl#68602
potiuk merged 1 commit into
apache:mainfrom
yuseok89:migrate-dags-list-import-errors-to-airflowctl

Conversation

@yuseok89

@yuseok89 yuseok89 commented Jun 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

related: #68402

Marks airflow dags list-import-errors as migrated to airflowctl dags list-import-errors by adding the deprecated_for_airflowctl marker.

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    • 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.

@yuseok89
yuseok89 marked this pull request as ready for review June 16, 2026 23:55
@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Jun 17, 2026
@yuseok89
yuseok89 force-pushed the migrate-dags-list-import-errors-to-airflowctl branch from d29eb39 to 68c01c3 Compare June 24, 2026 00:49
@yuseok89

yuseok89 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

@bugraoz93
Following the converted direction, I've reworked this PR to be marker-only.
I dropped the earlier change that routed the command through the API server.
Does this match what you had in mind?
If so, I'd like to quickly roll the same change out to the other open PRs.

@bugraoz93

bugraoz93 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

@bugraoz93 Following the converted direction, I've reworked this PR to be marker-only. I dropped the earlier change that routed the command through the API server. Does this match what you had in mind? If so, I'd like to quickly roll the same change out to the other open PRs.

Yes, thanks a lot!

@bugraoz93 bugraoz93 added the backport-to-v3-3-test Backport to v3-3-test label Jun 24, 2026
@bugraoz93 bugraoz93 mentioned this pull request Jun 24, 2026
Open
41 tasks
@bugraoz93 bugraoz93 changed the title [AIP-94] Route airflow dags list-import-errors through the API server [AIP-94] Add Maintainer deprecation to airflow dags list-import-errors Jun 24, 2026
@yuseok89 yuseok89 changed the title [AIP-94] Add Maintainer deprecation to airflow dags list-import-errors [AIP-94] Mark dags list-import-errors as migrated to airflowctl Jun 25, 2026
@henry3260

henry3260 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Need resolve conflicts

@yuseok89
yuseok89 force-pushed the migrate-dags-list-import-errors-to-airflowctl branch from 68c01c3 to e08dc4a Compare July 2, 2026 11:09
@yuseok89
yuseok89 requested a review from henry3260 as a code owner July 2, 2026 11:09
@yuseok89

yuseok89 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Need resolve conflicts

Resolved. Thanks !

@henry3260

henry3260 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Need resolve conflicts

Resolved. Thanks !

Could we deprecate all ⁠dags⁠ commands in this PR? If possible, that would be great.

potiuk
potiuk approved these changes Jul 4, 2026
View reviewed changes

@potiuk potiuk 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.

LGTM. Minor: the PR description is copy-pasted from a larger routing PR; the actual change is only the migration marker (the title is accurate).

@potiuk
potiuk merged commit 924f022 into apache:main Jul 4, 2026
72 checks passed
@github-actions github-actions Bot added this to the Airflow 3.3.1 milestone Jul 4, 2026
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Hi maintainer, this PR was merged without a milestone set.
We've automatically set the milestone to Airflow 3.3.1 based on: backport label targeting v3-3-test
If this milestone is not correct, please update it to the appropriate milestone.

This comment was generated by Milestone Tag Assistant.

@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-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-3-test PR Link

github-actions Bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 4, 2026
@yuseok89 Your friendly bot
[v3-3-test] [AIP-94] Mark dags list-import-errors as migrated to airf…
563145c 
…lowctl (apache#68602)

(cherry picked from commit 924f022)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
aws-airflow-bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jul 4, 2026
@yuseok89 Your friendly bot
[v3-3-test] [AIP-94] Mark dags list-import-errors as migrated to airf…
de98d0b 
…lowctl (apache#68602)

(cherry picked from commit 924f022)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
@YAshhh29 YAshhh29 mentioned this pull request Jul 4, 2026
Merged
@yuseok89

yuseok89 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

@henry3260
Sorry, I saw this late. It had already merged by then. When I get the chance, I'll check whether the rest of the dags commands have been marked as migrated, and take care of any that haven't.

henry3260 pushed a commit that referenced this pull request Jul 5, 2026
@github-actions @yuseok89
[v3-3-test] [AIP-94] Mark dags list-import-errors as migrated to airf…
ecfc6f9 
…lowctl (#68602) (#69390)

(cherry picked from commit 924f022)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request Jul 7, 2026
@github-actions @yuseok89 @vatsrahul1001
[v3-3-test] [AIP-94] Mark dags list-import-errors as migrated to airf…
0f700f2 
…lowctl (#68602) (#69390)

(cherry picked from commit 924f022)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request Jul 9, 2026
@github-actions @yuseok89 @vatsrahul1001
[v3-3-test] [AIP-94] Mark dags list-import-errors as migrated to airf…
f866f03 
…lowctl (#68602) (#69390)

(cherry picked from commit 924f022)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request Aug 5, 2026
@github-actions @yuseok89 @vatsrahul1001
[v3-3-test] [AIP-94] Mark dags list-import-errors as migrated to airf…
862cb26 
…lowctl (#68602) (#69390)

(cherry picked from commit 924f022)

Co-authored-by: Yuseok Jo <yuseok89@gmail.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
@Lee-W Lee-W Lee-W approved these changes
@bugraoz93 bugraoz93 bugraoz93 approved these changes
@dheerajturaga dheerajturaga Awaiting requested review from dheerajturaga dheerajturaga is a code owner
@henry3260 henry3260 Awaiting requested review from henry3260 henry3260 is a code owner

Assignees

No one assigned

Labels

area:CLI backport-to-v3-3-test Backport to v3-3-test ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Milestone

Airflow 3.3.1

Development

Successfully merging this pull request may close these issues.

5 participants

@yuseok89 @bugraoz93 @henry3260 @potiuk @Lee-W