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

AIP-84 Get Event Log #43406

Merged

Conversation

jason810496
Copy link
Contributor

closes: #43327
related: #42370

@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Oct 27, 2024
@pierrejeambrun pierrejeambrun added the legacy api Whether legacy API changes should be allowed in PR label Oct 28, 2024
@pierrejeambrun
Copy link
Member

Thanks for the PR. A few suggestions before we can merge.

@jason810496 jason810496 force-pushed the feature/AIP-84/add-get-event-log branch from 8918cd9 to 5faa9be Compare October 29, 2024 03:42
@jason810496
Copy link
Contributor Author

Thanks for reviewing, @pierrejeambrun. I’ve just resolved the issue.

@pierrejeambrun pierrejeambrun merged commit b0a8605 into apache:main Oct 29, 2024
52 checks passed
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
* AIP-84 Get Event Log

* fix: add error docs, refactor test case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. legacy api Whether legacy API changes should be allowed in PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AIP-84 Migrate the public endpoint Get Event Log to FastAPI
2 participants