-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve the error when DAG does not exist when running dag pause command #13900
Conversation
When running `airflow dags unpause` with a DAG that does not exist, it currently shows this error ``` root@6f086ba87198:/opt/airflow# airflow dags unpause example_bash_operatoredd Traceback (most recent call last): File "/usr/local/bin/airflow", line 33, in <module> sys.exit(load_entry_point('apache-airflow', 'console_scripts', 'airflow')()) File "/opt/airflow/airflow/__main__.py", line 40, in main args.func(args) File "/opt/airflow/airflow/cli/cli_parser.py", line 48, in command return func(*args, **kwargs) File "/opt/airflow/airflow/utils/cli.py", line 92, in wrapper return f(*args, **kwargs) File "/opt/airflow/airflow/cli/commands/dag_command.py", line 160, in dag_unpause set_is_paused(False, args) File "/opt/airflow/airflow/cli/commands/dag_command.py", line 170, in set_is_paused dag.set_is_paused(is_paused=is_paused) AttributeError: 'NoneType' object has no attribute 'set_is_paused' ``` This commit changes the error to show a helpful error: ``` root@6f086ba87198:/opt/airflow# airflow dags unpause example_bash_operatoredd DAG: example_bash_operatoredd does not exit in 'dag' table ```
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest master or amend the last commit of the PR, and push it with --force-with-lease. |
…and (#13900) When running `airflow dags unpause` with a DAG that does not exist, it currently shows this error ``` root@6f086ba87198:/opt/airflow# airflow dags unpause example_bash_operatoredd Traceback (most recent call last): File "/usr/local/bin/airflow", line 33, in <module> sys.exit(load_entry_point('apache-airflow', 'console_scripts', 'airflow')()) File "/opt/airflow/airflow/__main__.py", line 40, in main args.func(args) File "/opt/airflow/airflow/cli/cli_parser.py", line 48, in command return func(*args, **kwargs) File "/opt/airflow/airflow/utils/cli.py", line 92, in wrapper return f(*args, **kwargs) File "/opt/airflow/airflow/cli/commands/dag_command.py", line 160, in dag_unpause set_is_paused(False, args) File "/opt/airflow/airflow/cli/commands/dag_command.py", line 170, in set_is_paused dag.set_is_paused(is_paused=is_paused) AttributeError: 'NoneType' object has no attribute 'set_is_paused' ``` This commit changes the error to show a helpful error: ``` root@6f086ba87198:/opt/airflow# airflow dags unpause example_bash_operatoredd DAG: example_bash_operatoredd does not exit in 'dag' table ``` (cherry picked from commit 8723b1f)
…and (#13900) When running `airflow dags unpause` with a DAG that does not exist, it currently shows this error ``` root@6f086ba87198:/opt/airflow# airflow dags unpause example_bash_operatoredd Traceback (most recent call last): File "/usr/local/bin/airflow", line 33, in <module> sys.exit(load_entry_point('apache-airflow', 'console_scripts', 'airflow')()) File "/opt/airflow/airflow/__main__.py", line 40, in main args.func(args) File "/opt/airflow/airflow/cli/cli_parser.py", line 48, in command return func(*args, **kwargs) File "/opt/airflow/airflow/utils/cli.py", line 92, in wrapper return f(*args, **kwargs) File "/opt/airflow/airflow/cli/commands/dag_command.py", line 160, in dag_unpause set_is_paused(False, args) File "/opt/airflow/airflow/cli/commands/dag_command.py", line 170, in set_is_paused dag.set_is_paused(is_paused=is_paused) AttributeError: 'NoneType' object has no attribute 'set_is_paused' ``` This commit changes the error to show a helpful error: ``` root@6f086ba87198:/opt/airflow# airflow dags unpause example_bash_operatoredd DAG: example_bash_operatoredd does not exit in 'dag' table ``` (cherry picked from commit 8723b1f)
When running
airflow dags unpause
with a DAG that does not exist, itcurrently shows this error
This commit changes the error to show a helpful error:
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.