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.

Sign up for GitHub

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

Jump to bottom

Replace references to airflow users #45936

Open
1 of 2 tasks
bbovenzi opened this issue Jan 22, 2025 · 1 comment
Open
1 of 2 tasks

Replace references to airflow users #45936

bbovenzi opened this issue Jan 22, 2025 · 1 comment
Labels
AIP-79 area:core area:dev-env CI, pre-commit, pylint and other changes that do not change the behavior of the final code kind:bug This is a clearly a bug kind:documentation

Comments

@bbovenzi
Copy link
Contributor

Apache Airflow version

main (development)

If "Other Airflow 2 version" selected, which one?

No response

What happened?

In main, we removed the airflow users command since that was connected to FAB. But we didn't actually remove that command from docker and so when a user does something like breeze start-airflow they will get an error and airflow won't run.

What you think should happen instead?

We need to ctrl+f for airflow users and remove all references to it in our dockerfile, scripts and documentation.

How to reproduce

run breeze start-airflow

Operating System

mac

Versions of Apache Airflow Providers

No response

Deployment

Official Apache Airflow Helm Chart

Deployment details

No response

Anything else?

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@bbovenzi bbovenzi added AIP-79 area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Jan 22, 2025
@vincbeck
Copy link
Contributor

As part of this task we also need to make sure there is an equivalent command provided by the FAB auth manager like: airflow fab users

@vincbeck vincbeck removed the needs-triage label for new issues that we didn't triage yet label Jan 22, 2025
@dosubot dosubot bot added area:dev-env CI, pre-commit, pylint and other changes that do not change the behavior of the final code kind:documentation labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
AIP-79 area:core area:dev-env CI, pre-commit, pylint and other changes that do not change the behavior of the final code kind:bug This is a clearly a bug kind:documentation
Projects
AIP-79: Remove Flask AppBuilder as Core dependency
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bbovenzi @vincbeck