-
Notifications
You must be signed in to change notification settings - Fork 1
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
🔧(helm) rework chart based on jenny helm chart #252
base: main
Are you sure you want to change the base?
Conversation
|
71b3323
to
e174e19
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.
Good for me, just some questions to discuss about.
a8f8752
to
aa28caa
Compare
4f4f0e9
to
3ebc99b
Compare
bb89b4f
to
a9661b3
Compare
8f59a1c
to
01b80e6
Compare
7b8769c
to
d8bf4ef
Compare
Rework Warren Helm chart based on the work done on Jenny Helm chart in PR openfun/charts#5 Changes done in this commit: - Jobs for database migration or static collection are abstracted into a single job template - Env variables are now processed in the helper template - Removed unused HorizontalPodAutoscaler - Add startup probes for migration checks - Migration jobs are now executed post helm installation/upgrade - Removed unnecessary security context variables - Add a nginx container alongside warren-app to serve static files - Rework postgresql values to have a functional Helm chart on a local cluster - Update Helm chart README.md
d8bf4ef
to
bd35e66
Compare
Purpose
Rework Warren Helm chart based on the work done on Jenny Helm chart in PR openfun/charts#5
Proposal
job template