-
-
Notifications
You must be signed in to change notification settings - Fork 215
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
Deploy to GCP #10383
Comments
@PierreBesson created the project in GCP, I'll be able to work on it after holidays |
Still on vacations but I managed to work on this, this morning I'll update the cloudflare part too I'll finish the work maybe this weekend or next week |
It's really cool! Thanks!!! 👏👏👏 |
The version at https://lite.jhipster.tech/ is up-to-date ! After discussion with @murdos this morning, he would like to deploy the main branch instead of a release version So 3 solutions here:
I think the 3 is overkilled, and will consume too much resource at GCP About me, I don't have a strong opinion on this, as I don't use a lot the cloud version. |
Congratulations 🎊 and thank you, @pascalgrimaud! That's a really cool feature! I think I'll go for option [2] - publishing from the main branch. Here are my thoughts:
|
Lighter option 2, would be:
With this, after each new PRs from Renovate bot and other, there won't be new deployment... so no spam here |
I prefer that one! No overhead 👌👏 |
Do you fear some extraneous GCP cost because of that "spam"? Or that the service won't be available during deployment? |
I think specially about cost |
Cost shouldn't be an issue there is no extra costs associated with deployments on either AppEngine or CloudRun. I would worry more on maintenance with potentially having to troubleshoot failed deployment more frequently but as we have solid e2e tests that shouldn't be a problem. |
To be consistent with JHipster Online, I'm studying to deploy to GCP after each release.
So we'll have an official version at lite.jhipster.tech which is up-to-dated and not depend on @jdubois
Cc @murdos @renanfranca
The text was updated successfully, but these errors were encountered: