-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
feat: gcp sql database support #623
base: main
Are you sure you want to change the base?
feat: gcp sql database support #623
Conversation
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.
Changes that fix a gateway to service timeout mismatch have been reverted in the PR.
0fe4ca8
to
cff278f
Compare
67145b9
to
fef1554
Compare
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.
First deploy works well.
But next ones don't apply the migrations at the correct time, the cloud run builds are triggered before the new image is available. I used prisma to test.
Also might need to include similar dependencies for deploying services after migrations, like these: #627
7985ae4
to
3f49cb1
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #623 +/- ##
===========================================
+ Coverage 49.21% 77.88% +28.66%
===========================================
Files 23 5 -18
Lines 3233 217 -3016
===========================================
- Hits 1591 169 -1422
+ Misses 1537 40 -1497
+ Partials 105 8 -97
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
fd9c96d
to
60acdcf
Compare
89b6980
to
ddb015e
Compare
use pulumi gcp version with vpc network access fix rebase issue and format fix handle client close error rebase with source id fix add dependency to migration for each service deployed rebase fixes tidy ensure correct container image is always used add retry instead of using cloud build wait long running operation fix build resource remove unused build status code handle error on artifact registry unauthorised lint add license file add polling to retrieve latest metadata add auth mechanism for pkg.dev deployments correctly poll cloud build builds add cpu config add deletion policy config lint and generate generate
4c086c7
to
5ea9462
Compare
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.
Issue with deletion policy found, see comments.
Last issue is making sure services are created after the cloud build run ends. |
No description provided.