-
Notifications
You must be signed in to change notification settings - Fork 0
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 Helm chart to simplify deployment #45
Merged
Merged
Conversation
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
The osmcha-frontend image now includes a production-ready nginx.conf.
These values are now baked into the osmcha-django image.
Terraform Format and Style 🖌
|
This is now ready for review, but do not merge it yet. The other PRs linked above need to land first, and then this PR needs to be updated with the hashes of the new container images. |
Terraform Format and Style 🖌
|
Terraform Format and Style 🖌
|
I believe this is now ready to merge. Deploying shortly. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
I mentioned in OSMCha/osmcha-frontend#749 and OSMCha/osmcha-django#719 that some updates to the Helm chart would be necessary in order to safely deploy those changes. These are those updates.
I tested this today using minikube. Several additional modifications (not included in this PR) were necessary to make the "production" chart work on localhost:
enable_tls
to false, to disable Let's Encrypt automationWith these modifications, I was able to verify that:
/api/v1/changesets/157277981/
since I'm running the app with an empty database).Keeping this as a draft while I do a bit more testing, but this should all be ready to review pretty soon. Once it's reviewed, I'll merge the other two PRs (linked above) which will cause the new container images to get published to GHCR. After that I'll update the version hashes in this PR and then it should be ready to merge.