This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 26
WorkerV2 - Cloud Run V2 API #220
Merged
zzstoatzz
merged 27 commits into
PrefectHQ:main
from
japerry911:skylord/new/cloud_run_v2_and_worker_v2-active
Nov 28, 2023
Merged
Changes from 20 commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
dd522d0
👌 IMPROVE: started on cloud run v2 block, added foundation for all, b…
japerry911 3a82098
👌 IMPROVE: rough draft, still missing kill method, but ready to start…
japerry911 5d39edc
🤖 TEST: testing
japerry911 63f3687
‼️ BREAKING: removing memory string test
japerry911 c3dae79
👌 IMPROVE: creates successfully
japerry911 8ae0bcf
🐛 FIX: removed client
japerry911 b4bcc03
🐛 FIX: fixed validation bug
japerry911 9e43161
🐛 FIX: fix mispelling for var
japerry911 667bd70
🐛 FIX: fixed and simplified job name generation
japerry911 eb19816
🐛 FIX: fixed issue with observedGeneration keyerror
japerry911 6e7f9b9
🤖 TEST: test config job name as var
japerry911 8710d0e
🐛 FIX: made IDs use whole Prefect Name
japerry911 7f94031
👌 IMPROVE: working cancelation functionality
japerry911 caa4eb8
🤖 TEST: added cloud run v2 block unit tests
japerry911 aabb26f
📖 DOC: added some unit tests and docstrings
japerry911 0b72fb3
👌 IMPROVE: solved merge conflicts and tested serving mkdocs
japerry911 da5b404
🐛 FIX: applied some ci test fixes
japerry911 23f8935
🐛 FIX: fixed some failing unit tests
japerry911 459b183
🐛 FIX: fix missing optional and remove | None
japerry911 8d06dce
🐛 FIX: added the Pydantic Version conditional import logic to cr2 wor…
japerry911 6282c80
Merge branch 'PrefectHQ:main' into skylord/new/cloud_run_v2_and_worke…
japerry911 e64c356
🐛 FIX: PR feedback implemented, separate cloud run v2 block infra to …
japerry911 f7dbf5c
🐛 FIX: attempted fix on 3.8 test changing list to typing.List
japerry911 2842d8e
Merge branch 'main' into skylord/new/cloud_run_v2_and_worker_v2-active
japerry911 d88c158
👌 IMPROVE: added get_prefect_image_name as default factory to image c…
japerry911 3b74cd3
🐛 FIX: fix ci test and hardcode default to prefect 2 latest image
japerry911 4889acb
Merge branch 'main' into skylord/new/cloud_run_v2_and_worker_v2-active
zzstoatzz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,6 @@ | ||
# JetBrains | ||
.idea | ||
|
||
# Byte-compiled / optimized / DLL files | ||
__pycache__/ | ||
*.py[cod] | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
::: prefect_gcp.cloud_run_v2 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
::: prefect_gcp.workers.cloud_run_v2 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zzstoatzz It seems like there was a 404 in loading this that wasn't caught
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can work on that @serinamarie - I see another thing in the API docs that needs fixed, too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was fixed/merged! thanks though