helm icon indicating copy to clipboard operation
helm copied to clipboard

Portworx: operator helm chart

Open cabrinha opened this issue 1 year ago • 2 comments

What this PR does / why we need it: Update the helm chart to reflect industry standards, allow more customization and reduce complexity

Which issue(s) this PR fixes (optional) Closes #

Special notes for your reviewer:

cabrinha avatar May 04 '23 17:05 cabrinha

Can you also add the following to the PR helm pacakge helm index and create it in repo/beta directory. Steps:

  • cd helm/charts/portworx
  • helm package . # there is . at the end
  • mkdir -p helm/repo/beta
  • mv helm/charts/portworx/portworx-2.13.3.tgz helm/repo/beta
  • cd helm/repo/beta
  • helm repo index . --url https://raw.githubusercontent.com/portworx/helm/<your_branch_name>/repo/stable Add the resultant files to the PR, that way we can consume this as a helm chart for testing.

sharma-tapas avatar May 12 '23 15:05 sharma-tapas

Hi everyone, sorry to burst into this - I just wanted to encourage you to go ahead with this. I found this PR while I am in the process of using kustomize to achieve some core changes you already have build in here. These changes would definitely be much appreciated. Like I need to get rid of namespace creation ( which is easy ) but the namespace is hard coded e. g. in a batch job - which needs an ugly replacement procedure ... Anyway, this definitely makes it harder and more time consuming to implement a POC in my/our case. Please make this PR happen if you can :)

quercus-carsten avatar Nov 30 '23 12:11 quercus-carsten