justinpakzad commented on code in PR #69409:
URL: https://github.com/apache/airflow/pull/69409#discussion_r3524952272
##########
airflow-core/src/airflow/cli/commands/dag_command.py:
##########
@@ -727,6 +729,7 @@ def dag_report(args) -> None:
)
+@deprecated_for_airflowctl("airflowctl dags list-jobs")
Review Comment:
This one doesn't either. There is a `airflowctl jobs list` command, but it
can't filter by dag, so it's not fully matched with the cli.
##########
airflow-core/src/airflow/cli/commands/dag_command.py:
##########
@@ -758,6 +761,7 @@ def dag_list_jobs(args, dag: DAG | None = None, *, session:
Session = NEW_SESSIO
)
+@deprecated_for_airflowctl("airflowctl dags list-runs")
Review Comment:
Same here. We currently have `airflowctl dagrun list` which lists Dag runs
(optionally filtered by dag id, state, etc), so it's very similar. That being
said, if there are plans to extend the dag operations to match the command
names exactly, then maybe it makes sense as is.
##########
airflow-core/src/airflow/cli/commands/dag_command.py:
##########
@@ -692,6 +693,7 @@ def dag_list_import_errors(args, *, session: Session =
NEW_SESSION) -> None:
sys.exit(1)
+@deprecated_for_airflowctl("airflowctl dags report")
Review Comment:
This command doesn't exist yet in airflowctl.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
