You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/04/02 20:53:08 UTC
[GitHub] [airflow] blag opened a new issue #22708: Rewrite Airflow CLI commands with Click
blag opened a new issue #22708:
URL: https://github.com/apache/airflow/issues/22708
### Body
Converting the Airflow CLI to use [Click](https://click.palletsprojects.com) and [rich-click](https://pypi.org/project/rich-click/) is a large task and will be much easier and quicker if other people pitch in.
I'm creating this issue to track volunteers and progress as we rewrite the Airflow CLI to use Click.
If you would like to help, you can do so by commenting below. I will try to update this list with the people who have volunteered to pitch in.
## Proposed Development Process ##
Once you drop a comment below, you can get to work converting a command. Please submit PRs against [my branch (`astronomer:convert-to-click-cli`)](https://github.com/astronomer/airflow/tree/convert-to-click-cli), that way we can review any changes separately from the apache/airflow review process and submit all of the conversions against `apache:airflow` in one go in PR #22613 (and I _think_ Git/GitHub will/should handle all of the attributions automatically).
I'd like to curtail the scope of this issue to just task management/assignments (and related issues). So if you run into an issue while coding, please drop your comment in #22613 and we can handle development issues there.
If you have any suggestions on how to improve this process, this would probably be a better place for it.
## Conversion Volunteers ##
* [ ] celery
* [ ] cheat_sheet
* [ ] config
* [ ] connection
* [ ] dag
* [x] db - @blag
* [ ] info
* [ ] jobs
* [ ] kerberos
* [ ] kubernetes
* [ ] legacy
* [ ] plugins
* [ ] pool
* [ ] provider
* [ ] role
* [ ] rotate_fernet_key
* [x] scheduler - @blag
* [x] standalone - @blag
* [ ] sync_perm
* [ ] task
* [x] triggerer - @blag
* [ ] user
* [ ] variable
* [ ] version
* [x] webserver - @blag
## Testing Volunteers ##
In addition to fixing/writing automated tests for the commands, we're probably going to need people to manually test the new commands.
I'm separating out testing volunteers since we may have community members who can help test out code but won't be able to convert the code itself. This is a major change, so I'm hoping to get at least **two** volunteer testers per command.
* [ ] celery
* [ ] cheat_sheet
* [ ] config
* [ ] connection
* [ ] dag
* [ ] db
* [ ] info
* [ ] jobs
* [ ] kerberos
* [ ] kubernetes
* [ ] legacy
* [ ] plugins
* [ ] pool
* [ ] provider
* [ ] role
* [ ] rotate_fernet_key
* [ ] scheduler
* [ ] standalone
* [ ] sync_perm
* [ ] task
* [ ] triggerer
* [ ] user
* [ ] variable
* [ ] version
* [ ] webserver
### Committer
- [X] I acknowledge that I am a maintainer/committer of the Apache Airflow project.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] boring-cyborg[bot] commented on issue #22708: Rewrite Airflow CLI commands with Click
Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #22708:
URL: https://github.com/apache/airflow/issues/22708#issuecomment-1086720855
Thanks for opening your first issue here! Be sure to follow the issue template!
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] tirkarthi commented on issue #22708: Rewrite Airflow CLI commands with Click
Posted by GitBox <gi...@apache.org>.
tirkarthi commented on issue #22708:
URL: https://github.com/apache/airflow/issues/22708#issuecomment-1086786547
sync-perm command PR : https://github.com/astronomer/airflow/pull/1485
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] Bowrna commented on issue #22708: Rewrite Airflow CLI commands with Click
Posted by GitBox <gi...@apache.org>.
Bowrna commented on issue #22708:
URL: https://github.com/apache/airflow/issues/22708#issuecomment-1086900496
I will choose `connection` first followed by `config`, `dag` in that order. will finish this first and pick up further :)
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org