Skip to content

Migrate FAB POST /users to FastAPI #60523

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
Merged
vincbeck merged 2 commits into from
Jan 14, 2026
Merged

Migrate FAB POST /users to FastAPI #60523

vincbeck merged 2 commits into from
Jan 14, 2026

Conversation

@henry3260
Copy link
Contributor

@henry3260 henry3260 commented Jan 14, 2026 •
edited
Loading

Take over #57480
And I will do this #57480 (comment) in a follow-up pr.

Was generative AI tooling used to co-author this PR?

  • Yes (please specify the tool below)
  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@henry3260 henry3260 requested a review from vincbeck as a code owner January 14, 2026 14:35
vincbeck
vincbeck approved these changes Jan 14, 2026
View reviewed changes
Copy link
Contributor

@vincbeck vincbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@vincbeck
Copy link
Contributor

vincbeck commented Jan 14, 2026

Tests are failing

@henry3260
Copy link
Contributor Author

henry3260 commented Jan 14, 2026

Tests are failing

Yes, I found errors. I will fix it later

@vincbeck vincbeck merged commit eaa40bf into apache:main Jan 14, 2026
221 checks passed
@vincbeck vincbeck mentioned this pull request Jan 14, 2026
Closed
@ddovbii ddovbii mentioned this pull request Jan 15, 2026
Open
@Dev-iL
Copy link
Collaborator

Dev-iL commented Jan 15, 2026

(Creating the linkage to #56730)

jason810496 pushed a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
@henry3260 @jason810496
Migrate FAB POST /users to FastAPI (apache#60523)
7c7ee9c
@potiuk
Copy link
Member

potiuk commented Jan 22, 2026

Nice! Remaining things added as sub-issues in #56730

@vincbeck vincbeck mentioned this pull request Jan 28, 2026
Open
85 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vincbeck vincbeck vincbeck approved these changes

Assignees

No one assigned

Labels

area:providers provider:fab

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@henry3260 @vincbeck @Dev-iL @potiuk