Update explanation on Helm use case #1375
Conversation
Signed-off-by: Somtochi Onyekwere <somtochionyekwere@gmail.com>
|
@stefanprodan I don't know if we want to add an beginning section where we say we expect them to have flux bootstrapped |
|
@somtochiama yes, we need to tell people to bootstrap and link to the install doc. |
Signed-off-by: Somtochi Onyekwere <somtochionyekwere@gmail.com>
content/en/flux/use-cases/helm.md
Outdated
| flux create helmrelease my-traefik --chart traefik \ | ||
| --source HelmRepository/traefik \ | ||
| --chart-version 9.18.2 \ | ||
| --namespace traefik \ | ||
| --export > /flux/boot/traefik/helmrepo.yaml | ||
| --export > traefik/helmrepo.yaml |
There was a problem hiding this comment.
Can you please test this with Kind, I guess the HR will fail due to the lack of LB support.
There was a problem hiding this comment.
I propose we replace traefik with nginx, and then we can use the HR from here which is e2e tested on Kind. https://github.com/fluxcd/flux2-kustomize-helm-example/blob/main/infrastructure/controllers/ingress-nginx.yaml
There was a problem hiding this comment.
@stefanprodan you are right
Helm install failed: timed out waiting for the condition │
│
Last Helm logs: │
│
creating 1 resource(s) │
CRD traefikservices.traefik.containo.us is already present. Skipping. │
creating 5 resource(s) │
beginning wait for 5 resources with timeout of 5m0s │
Service does not have load balancer ingress IP address: flux-system/my-traefik
I will update it to use nginx
There was a problem hiding this comment.
Nginx also has the same problem. Should we use podinfo?
There was a problem hiding this comment.
Nginx does not have this issue, see the values used in the link I posted above
There was a problem hiding this comment.
In the usecase guide, values are introduced in a later section after the helm release is applied.
So I was thinking we could start with something that just works without tweaking the values first
Signed-off-by: Somtochi Onyekwere <somtochionyekwere@gmail.com>
The explanation after the
flux create source helmcommand seems to imply that the command will commit the files to git.This pull request tries to fix that.
Ref: https://cloud-native.slack.com/archives/CLAJ40HV3/p1676320569584019
Signed-off-by: Somtochi Onyekwere somtochionyekwere@gmail.com