Skip to content

Commit 8b4432f

Browse files
hiddecostefanprodan
authored andcommitted
Add impact to Helm Operator
1 parent 0084e56 commit 8b4432f

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

docs/spec/README.md

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ of the components using them.
2626

2727
The controller implementation will watch for source objects in a cluster and act on them.
2828
The actions performed by the source controller could be:
29+
2930
* validate source definitions
3031
* authenticate to sources and validate authenticity
3132
* detect source changes based on update policies (semver)
@@ -39,6 +40,7 @@ The actions performed by the source controller could be:
3940
## Impact to Flux
4041

4142
Having a dedicated controller that manages Git repositories defined with Kubernetes custom resources would:
43+
4244
* simplify Flux configuration as fluxd could subscribe to Git sources in-cluster and pull the artifacts
4345
automatically without manual intervention from users to reconfigure and redeploy FLux
4446
* improve the installation experience as users will not have to patch fluxd's deployment to inject
@@ -51,4 +53,13 @@ the HTTPS basic auth credentials, change the source URL or other Git and PGP rel
5153

5254
## Impact to Helm Operator
5355

54-
TODO
56+
Having a dedicated controller that manages Helm repositories and charts defined with Kubernetes custom
57+
resources would:
58+
59+
* simplify the Helm Operator configuration as repository and chart definitions can be re-used across
60+
`HelmRelease` resources (see [fluxcd/helm-operator#142](https://github.com/fluxcd/helm-operator/issues/142))
61+
* improve the user experience as repositories requiring authentication will no longer require a
62+
`repositories.yaml` import / file mount
63+
* simplify the architecture of the Helm Operator as it allows the operator to work with a single
64+
source type (`HelmChart`) and way of preparing and executing installations and/or upgrades
65+
* allow the Helm Operator to run under a non-root user as it wouldn't need to shell out to git

0 commit comments

Comments
 (0)