HelmRepository is immediately deleted without any errors #4996
-
|
Hello. So I created a flux installation cycle where I'm installing cert-manager and some config using kustomize and I'm using a script to install this like this: #!/usr/bin/env bash
# setup flux cluster reconilication
flux create source git flux-system \
--url=https://github.com/open-component-model/ocm-controller \
--branch=${BRANCH} \
--username=${GITHUB_USER} \
--password=${GITHUB_TOKEN} \
--ignore-paths="clusters/**/flux-system/"
flux create kustomization flux-system \
--source=flux-system \
--path=./deploy/flux/infra
What I'm observing is that cert-manager and the config is correctly installed but as soon as the ocm-controller HelmRepository gets reconciled and what I assume is a success because of the logs herE: it IMMEDIATELY gets deleted. Like, puff, it vanishes. The interesting part is that the HelmRelease still reconciles~ And doesn't error only after 5m it says, nope no luck. helm-controller doesn't show any errors either: I'm not sure what's going on... Please help? :D Flux version |
Beta Was this translation helpful? Give feedback.
Answered by
stefanprodan
Sep 23, 2024
Replies: 4 comments 10 replies
-
|
Check the kustomize-controller logs for that HelmRepository, it will tell if it gets garbage collected. |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
There is nothing in the kustomization logs either. But why does it get garbage collected before it's even used? And the cert-manager is the same so on that basis it should have garbage collected that as well, didn't it? 🤔 |
Beta Was this translation helpful? Give feedback.
-
|
If I apply this HelmRepository: apiVersion: source.toolkit.fluxcd.io/v1beta2
kind: HelmRepository
metadata:
name: ocm-controller
namespace: ocm-system
spec:
interval: 15m
type: "oci"
url: oci://ghcr.io/open-component-model/helmIt's reconciled and then immediately deleted... WHY? |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Same happens if I try it with an OCIRepository. It says, all good, reconciled, and then just removes it. |
Beta Was this translation helpful? Give feedback.
2 replies
-
|
Can you post here the OCIRepo and HelmRelease, I will try it on cluster. |
Beta Was this translation helpful? Give feedback.
-
apiVersion: helm.toolkit.fluxcd.io/v2
kind: HelmRelease
metadata:
name: ocm-controller
namespace: ocm-system
spec:
interval: 30m
chart:
spec:
chart: ocm-controller
version: "v0.24.0"
sourceRef:
kind: HelmRepository
name: ocm-controller
namespace: ocm-system
interval: 12h
values:
cert-manager:
enabled: falseHelm Release OCIRepositroy i tried with previously but switched to HelmRepository later: apiVersion: source.toolkit.fluxcd.io/v1beta2
kind: OCIRepository
metadata:
name: ocm-controller
namespace: ocm-system
spec:
interval: 15m
layerSelector:
mediaType: "application/vnd.cncf.helm.chart.content.v1.tar+gzip"
operation: copy
url: oci://ghcr.io/open-component-model/helm/ocm-controller
ref:
semver: "v0.24.0"And the HelmRelease: apiVersion: helm.toolkit.fluxcd.io/v2
kind: HelmRelease
metadata:
name: ocm
namespace: ocm-system
spec:
interval: 30m
chartRef:
kind: OCIRepository
name: ocm-controller
releaseName: ocm-controller
values:
cert-manager:
enabled: false |
Beta Was this translation helpful? Give feedback.
-
|
Looks like this chart messes up with the namespace and at install all things are GCed... |
Beta Was this translation helpful? Give feedback.
7 replies
-
|
It's really bad practice to include namespaces in a chart, a HelmRelease is a namespaced object FFS, on uninstall you wipe the whole thing. |
Beta Was this translation helpful? Give feedback.
-
|
Yes. 🙇 I actually have another fix on the repo that will remove it. |
Beta Was this translation helpful? Give feedback.
-
|
Here is how you can do it now, you need a different namespace for the Flux objects: |
Beta Was this translation helpful? Give feedback.
-
|
Anyway this chart needs a major rewrite, instead of |
Beta Was this translation helpful? Give feedback.
-
|
Oh yeah, I know there are a LOT of things that I need to fix there.. I wanted something that's working at least at that point |
Beta Was this translation helpful? Give feedback.
Answer selected by
Skarlso
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Looks like this chart messes up with the namespace and at install all things are GCed...