-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
CLI: Variables set should allow to set description #34756
Comments
Hi @eladkal , I am willing to take this issue and make a PR for this. Before I made any changes, I think this issue is related to this code snippet airflow/airflow/cli/commands/variable_command.py Lines 61 to 66 in 99eeb84
In order to achieve this, I will need to update the following line Variable.set(args.key, args.value, serialize_json=args.json) to something like this: Variable.set(args.key, args.value, args.description, serialize_json=args.json) I am wondering if this approach looks good to you and please feel free to assign me to this issue. Thanks! |
seems right. |
i wanted to solve this isuue .please assign me to solve |
Cool, let me do it real quick |
#34791 is ready for review |
#34791 is waiting for another approval, thanks! |
Body
The CLI:
airflow variables set [-h] [-j] [-v] key VALUE
https://airflow.apache.org/docs/apache-airflow/stable/cli-and-env-variables-ref.html#set_repeat1
Doesn't support adding description though column exists and we support it from Rest API:
https://airflow.apache.org/docs/apache-airflow/stable/stable-rest-api-ref.html#operation/post_variables
The Task:
Allow to set description from cli command
Committer
The text was updated successfully, but these errors were encountered: