Skip to content

[AIP-94] Mark dags pause/unpause as migrated to airflowctl - #68650

Merged
henry3260 merged 1 commit into
apache:mainfrom
yuseok89:migrate-dags-pause-unpause-to-airflowctl
Jul 5, 2026
Merged

[AIP-94] Mark dags pause/unpause as migrated to airflowctl#68650
henry3260 merged 1 commit into
apache:mainfrom
yuseok89:migrate-dags-pause-unpause-to-airflowctl

Conversation

@yuseok89

@yuseok89 yuseok89 commented Jun 17, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

related: #68402

Marks airflow dags pause/unpause as migrated to airflowctl dags pause/unpause 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 17, 2026 01:45
@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Jun 17, 2026
bugraoz93
bugraoz93 reviewed Jun 17, 2026
View reviewed changes

@bugraoz93 bugraoz93 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Have a comment on the overall direction. Thanks for PR!

Comment thread airflow-core/src/airflow/cli/commands/dag_command.py Outdated
@yuseok89
yuseok89 force-pushed the migrate-dags-pause-unpause-to-airflowctl branch from 3b38350 to b4b6e92 Compare June 25, 2026 07:44
@yuseok89 yuseok89 changed the title [AIP-94] Route airflow dags pause/unpause through the API server [AIP-94] Mark dags pause/unpause as migrated to airflowctl Jun 25, 2026
@YAshhh29 YAshhh29 mentioned this pull request Jul 3, 2026
Merged
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 description reads as if it routes pause/unpause through the API server, but the change here is just the migration marker — worth trimming the body.

@potiuk

potiuk commented Jul 4, 2026

Copy link
Copy Markdown
Member

Conflicts as well

@yuseok89
yuseok89 force-pushed the migrate-dags-pause-unpause-to-airflowctl branch from b4b6e92 to fc32018 Compare July 5, 2026 08:20
@yuseok89
yuseok89 requested a review from henry3260 as a code owner July 5, 2026 08:20
@henry3260 henry3260 added the backport-to-v3-3-test Backport to v3-3-test label Jul 5, 2026
@henry3260
henry3260 merged commit 638c635 into apache:main Jul 5, 2026
73 checks passed
@henry3260

henry3260 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Thanks!

@github-actions github-actions Bot added this to the Airflow 3.3.1 milestone Jul 5, 2026
@github-actions

github-actions Bot commented Jul 5, 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 5, 2026

Copy link
Copy Markdown
Contributor

Backport failed to create: v3-3-test. View the failure log Run details

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 Commit Link

You can attempt to backport this manually by running:

cherry_picker 638c635 v3-3-test

This should apply the commit to the v3-3-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

If you don't have cherry-picker installed, see the installation guide.

@henry3260

henry3260 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

backport in #69412

henry3260 added a commit that referenced this pull request Jul 5, 2026
@henry3260 @yuseok89
[AIP-94] Mark dags pause/unpause as migrated to airflowctl (#68650) (#…
b228add 
…69412)

(cherry picked from commit 638c635)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request Jul 7, 2026
@henry3260 @yuseok89 @vatsrahul1001
[AIP-94] Mark dags pause/unpause as migrated to airflowctl (#68650) (#…
2a2dd10 
…69412)

(cherry picked from commit 638c635)

Co-authored-by: Yuseok Jo <yuseok89@gmail.com>
vatsrahul1001 pushed a commit that referenced this pull request Jul 9, 2026
@henry3260 @yuseok89 @vatsrahul1001
[AIP-94] Mark dags pause/unpause as migrated to airflowctl (#68650) (#…
0e60234 
…69412)

(cherry picked from commit 638c635)

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
@bugraoz93 bugraoz93 bugraoz93 approved these changes
@henry3260 henry3260 henry3260 approved these changes
@dheerajturaga dheerajturaga Awaiting requested review from dheerajturaga dheerajturaga 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.

4 participants

@yuseok89 @potiuk @henry3260 @bugraoz93