Skip to content
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

Issue with CLI/Airflow API #39197

Closed
2 tasks done
fakirAyoub opened this issue Apr 23, 2024 · 4 comments · Fixed by #39224
Closed
2 tasks done

Issue with CLI/Airflow API #39197

fakirAyoub opened this issue Apr 23, 2024 · 4 comments · Fixed by #39224
Assignees
Labels
kind:bug This is a clearly a bug kind:documentation

Comments

@fakirAyoub
Copy link

What do you see as an issue?

In this link, the CLI for adding roles is either incomplete or outdated (for instance, it is not shown that resources should be prefixed with their types => DAG:...), etc...

There is no documentation on how to query the API in a deployed Airflow environment (Kubernetes or else)

Solving the problem

Have a clearer and working examples for the Airflow CLI, and the API to interact with the Airflow components.

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@fakirAyoub fakirAyoub added kind:bug This is a clearly a bug kind:documentation needs-triage label for new issues that we didn't triage yet labels Apr 23, 2024
Copy link

boring-cyborg bot commented Apr 23, 2024

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@fakirAyoub fakirAyoub changed the title Issue with CLI/Airflow APi Issue with CLI/Airflow API Apr 23, 2024
@potiuk
Copy link
Member

potiuk commented Apr 23, 2024

Indeed. Link should be added to https://airflow.apache.org/docs/apache-airflow-providers-fab/stable/cli-ref.html where it has been moved - similarly to executors CLIs - can you please add a PR with the link ?

@potiuk
Copy link
Member

potiuk commented Apr 23, 2024

(should be easy - by looking as other links are added after clicking "Suggest a change on this page".

@jscheffl jscheffl removed the needs-triage label for new issues that we didn't triage yet label Apr 23, 2024
potiuk added a commit to potiuk/airflow that referenced this issue Apr 24, 2024
@potiuk
Copy link
Member

potiuk commented Apr 24, 2024

Added it quickly to make sure it will make it for 2.9.1

eladkal pushed a commit that referenced this issue Apr 24, 2024
jedcunningham pushed a commit that referenced this issue Apr 26, 2024
Fixes: #39197
(cherry picked from commit fd1b5f5)
RodrigoGanancia pushed a commit to RodrigoGanancia/airflow that referenced this issue May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug This is a clearly a bug kind:documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants