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

store helm values into a Kubernetes secret #782

Merged
merged 1 commit into from
Apr 12, 2022

Conversation

aanm
Copy link
Member

@aanm aanm commented Apr 9, 2022

Similarly to helm behavior, Cilium-cli will store its generated yaml
values file in a secret.

Signed-off-by: André Martins [email protected]

@aanm aanm requested a review from a team April 9, 2022 01:30
@aanm aanm requested a review from a team as a code owner April 9, 2022 01:30
@aanm aanm requested review from a team, sayboras, twpayne and jrfastab April 9, 2022 01:30
@aanm aanm temporarily deployed to ci April 9, 2022 01:30 Inactive
@aanm aanm force-pushed the pr/store-information-in-cilium-secret branch from 4d2c502 to 18fc2ff Compare April 9, 2022 02:48
@aanm aanm temporarily deployed to ci April 9, 2022 02:48 Inactive
@aanm aanm force-pushed the pr/store-information-in-cilium-secret branch from 18fc2ff to 93c64e5 Compare April 9, 2022 21:09
@aanm aanm temporarily deployed to ci April 9, 2022 21:09 Inactive
Similarly to helm behavior, Cilium-cli will store its generated yaml
values file in a secret.

Signed-off-by: André Martins <[email protected]>
@aanm aanm force-pushed the pr/store-information-in-cilium-secret branch from 93c64e5 to 4239e9d Compare April 10, 2022 21:12
@aanm aanm temporarily deployed to ci April 10, 2022 21:12 Inactive
Copy link
Contributor

@michi-covalent michi-covalent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to do anything in this pr, but i do wonder if we could just run helm install and let helm keep track of the installation state.

@aanm
Copy link
Member Author

aanm commented Apr 12, 2022

no need to do anything in this pr, but i do wonder if we could just run helm install and let helm keep track of the installation state.

@michi-covalent it depends on what we decide to use in the future. Right now, cilium-cli has the control over which manifests we want to install. Also perhaps in the future we will have the ability to control how we perform the upgrade of each manifests and which configmap options should be enabled or not. We kind of lose control of that by doing helm install

Copy link
Member

@sayboras sayboras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

@tklauser tklauser merged commit 77cc2f4 into master Apr 12, 2022
@tklauser tklauser deleted the pr/store-information-in-cilium-secret branch April 12, 2022 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants