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

createResources flag in helm chart looks backwards #1723

Closed
ryandawsonuk opened this issue Apr 21, 2020 · 4 comments · Fixed by #1725
Closed

createResources flag in helm chart looks backwards #1723

ryandawsonuk opened this issue Apr 21, 2020 · 4 comments · Fixed by #1725
Labels
bug triage Needs to be triaged and prioritised accordingly

Comments

@ryandawsonuk
Copy link
Contributor

ryandawsonuk commented Apr 21, 2020

Only creates configmap if createResources is set to false.

https://github.com/SeldonIO/seldon-core/blob/master/helm-charts/seldon-core-operator/templates/configmap_seldon-config.yaml#L1

Passing this on after a customer hit trouble with it.

@ryandawsonuk ryandawsonuk added bug triage Needs to be triaged and prioritised accordingly labels Apr 21, 2020
@ukclivecox
Copy link
Contributor

createResources is for the Operator itself to create the core resources: configmap, webhook svc, webhook configuration rather than use yaml.

@ryandawsonuk
Copy link
Contributor Author

Eventually we realised it needs to be set to false so that the resources are created via helm rather than operator.

It is a bit confusing as a create flag is often used in helm charts to indicate whether helm should create the resource.

@ryandawsonuk
Copy link
Contributor Author

createResourcesOnStartup might be better

@ryandawsonuk
Copy link
Contributor Author

Actually, thinking back what caused confusion was less the name and more the comment in the values file that says # Whether to create the webhook service, and webhookconfigurations on startup so it wasn't realised that it affected the configmap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug triage Needs to be triaged and prioritised accordingly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants