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

Update the SDK to use PATCH /deployments/{id} endpoint for existing deployments #17050

Merged
merged 22 commits into from
Feb 8, 2025

Conversation

desertaxle
Copy link
Member

@desertaxle desertaxle commented Feb 7, 2025

Updates flow.serve, flow.deploy, prefect deploy to use the PATCH /deployments/{id} endpoint when updating an existing deployment. This allows for more targeted updates that won't overwrite changes made via other clients or the UI.

Copy link

codspeed-hq bot commented Feb 7, 2025

CodSpeed Performance Report

Merging #17050 will not alter performance

Comparing use-patch-endpoint-in-client (7daad2c) with main (0794427)

Summary

✅ 2 untouched benchmarks

@desertaxle desertaxle force-pushed the use-patch-endpoint-in-client branch from 71a04c7 to 983b82e Compare February 7, 2025 20:20
@desertaxle desertaxle changed the base branch from use-patch-for-dot-deploy to main February 7, 2025 20:21
@desertaxle desertaxle changed the title Update prefect deploy to use PATCH /deployments/{id} endpoint for existing deployments Update the SDK to use PATCH /deployments/{id} endpoint for existing deployments Feb 7, 2025
@desertaxle desertaxle marked this pull request as ready for review February 7, 2025 21:03
@cicdw cicdw merged commit c8986ed into main Feb 8, 2025
45 of 46 checks passed
@cicdw cicdw deleted the use-patch-endpoint-in-client branch February 8, 2025 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants