Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

addon: implement flux helm operator add-on #103

Merged
merged 1 commit into from
Oct 25, 2019
Merged

Conversation

chanwit
Copy link
Member

@chanwit chanwit commented Oct 22, 2019

This PR implements an add-on for Flux Helm operator.
The current instructions here: https://github.com/fluxcd/helm-operator-get-started assume that we start Flux in a normal way (without using wksctl).
This is a tweaked version of helm operator yaml to work with wksctl's Flux service account.

Fixes #100

Copy link
Contributor

@jrryjcksn jrryjcksn left a comment

Choose a reason for hiding this comment

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

Full disclosure: I'm only peripherally familiar with flux deployment. That said, this LGTM.

addons/flux-helm-op/addon.json Outdated Show resolved Hide resolved
addons/flux-helm-op/flux-helm-op.yaml Outdated Show resolved Hide resolved
@chanwit chanwit force-pushed the flux_helm_op_addon branch 4 times, most recently from ec0effc to 191b29f Compare October 24, 2019 17:04
@chanwit chanwit merged commit 07f3d20 into master Oct 25, 2019
@chanwit
Copy link
Member Author

chanwit commented Oct 25, 2019

thank you @jrryjcksn for reviewing this!

@chanwit chanwit deleted the flux_helm_op_addon branch October 25, 2019 04:59
D3nn pushed a commit that referenced this pull request May 8, 2020
addon: implement flux helm operator add-on
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposal: Flux Helm Operator as an Add-on
2 participants