[AIP-94] Mark dags state and next-execution CLI commands as migrated to airflowctl - #69305
Conversation
potiuk
left a comment
There was a problem hiding this comment.
LGTM — verified both replacement strings against the airflowctl registry (dags state / dags next-execution); no-op marker, no drift.
One thing before this can merge: the branch currently has conflicts with main (mergeable: dirty). Please rebase on the latest main and resolve the conflict.
0c91649 to
3485fef
Compare
|
Thanks a lot for the review, @potiuk! Rebased on the latest Also, since this is one of my initial contributions to Airflow, I'd love any feedback on things I can improve - whether that's PR structure, commit message style, testing approach, or anything else. Happy to iterate. |
3485fef to
ad6ce5c
Compare
|
Gentle bump — this has been approved and green for a week. Happy to rebase again if it helps. |
henry3260
left a comment
There was a problem hiding this comment.
Appreciate your contribution!
f72f432 to
25d4e7d
Compare
Backport failed to create: airflow-ctl-v0-1-test. View the failure log Run detailsNote: As of Merging PRs targeted for Airflow 3.X In matter of doubt please ask in #release-management Slack channel.
You can attempt to backport this manually by running: cherry_picker 5628e83 airflow-ctl-v0-1-testThis should apply the commit to the airflow-ctl-v0-1-test branch and leave the commit in conflict state marking After you have resolved the conflicts, you can continue the backport process by running: cherry_picker --continueIf you don't have cherry-picker installed, see the installation guide. |
Add the
@deprecated_for_airflowctlmarker decorator to twodagsCLIcommands whose airflowctl equivalents already exist:
dags state→airflowctl dags statedags next-execution→airflowctl dags next-executionBoth airflowctl commands are already implemented and registered in
airflow-ctl, so functional parity is met. This follows the samemarker-only direction as #68932 (variables), #68958 (config), and
#68650 (dags pause/unpause).
related: #68402