Skip to content
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

init chart helm #7

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

damienjacinto
Copy link

@damienjacinto damienjacinto commented Apr 9, 2021

Proposal !

Feel free to message for any change I'm going to use this for the dbaas
(english is not my mother tongue so feel free to change any wording)

still missing :

  • docs parameters (I'll update this at the end)
  • 1 tag for the appVersion (you are using latest ;p)
  • How you want to publish the chart, I'm used to create a branch gh-pages and use Githubpage but I can change
  • LivenessProbe

@Sh4d1
Copy link
Owner

Sh4d1 commented Apr 9, 2021

Nice 😄

How you want to publish the chart, I'm used to create a branch gh-pages and use Githubpage but I can change

Yep, either here, or I can create a helm-charts repo and move it here (for maybe other projects 😄)

1 tag for the appVersion (you are using latest ;p)

Yes I need to add versioning 😅 (especially since I want to do a rewrite using CRDs)

I won't have time to review it until next week but thanks, we'll get it merge soon 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants