docs: update deploying cilium docs for v1.8 #9034
Conversation
Signed-off-by: ChickenIQ <[email protected]>
| - install | ||
| - cilium |
There was a problem hiding this comment.
is it a typo or should it be two times?
I would want some input from @leppeK on this change, as he brought this change initially
There was a problem hiding this comment.
I believe it is correct.
helm install [NAME] [CHART] [flags]
There was a problem hiding this comment.
For helm it is correct, unfortunately helm leaves traces of the release which you might not want.
I'll see if I can convince Cilium to provide an official container image for all platforms
There was a problem hiding this comment.
I think the comments already indicate that you can use this method to do anything, if you want to use this to install something else using helm or whatever it works as well.
A use case could be installing your CNI and/or ArgoCD at bootstrap time
There was a problem hiding this comment.
Isn't it the same as manually installing using helm after bootstrap?
There was a problem hiding this comment.
Doing stuff at bootstrap means you can do it completely declaratively and in one go and at the right time... Great for reproducibilty
Signed-off-by: ChickenIQ <[email protected]>
|
Added the namespace. Previously it was trying to install to the "default" namespace instead of "kube-system". I somehow managed to not include it in the commit. |
|
I've asked over at Cilium to provide an official image so we'd have to wait for that |
|
Let's wait for this to be implemented and update the docs accordingly |
@ChickenIQ |
Pull Request
What? (description)
Replace quay.io/cilium/cilium-cli-ci:latest with alpine/helm:latest and modify the command accordingly in "Method 5: Using a job".
Why? (reasoning)
The current method does not support arm64 as there are no available images for this architecture. This solves that problem and is a drop-in replacement.
Acceptance
Please use the following checklist:
Most of the tasks in this checklist have generated unnecessary commits by removing newlines from existing code, so I decided not to include them.