| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
✅ Deploy Preview for docs-securecodebox canceled.
|
Sorry, something went wrong.
The operator hardcoded `.svc.cluster.local` when building the in-cluster Minio endpoint (RELEASE-minio.NAMESPACE.svc.cluster.local), so the operator can't reach Minio on clusters configured with a custom cluster domain (e.g. via kubelet's --cluster-domain). Adds a `clusterDomain` value (default `cluster.local`, preserving existing behavior) and uses it when building the endpoint. Fixes secureCodeBox#2895 Signed-off-by: Amit Mishra <amit.mishra.eee21@itbhu.ac.in>
There was a problem hiding this comment.
cool thanks :)
Sorry, something went wrong.
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Fixes #2895
Problem
The operator's Helm chart hardcodes .svc.cluster.local when building the in-cluster Minio endpoint:
operator/templates/manager/manager.yaml:78 value: "{{ .Release.Name }}-minio.{{ .Release.Namespace }}.svc.cluster.local"On clusters configured with a non-default cluster domain (e.g. kubelet started with --cluster-domain=mycorp.internal), this hostname doesn't resolve, and the operator can't reach its own bundled Minio.
Fix
Adds a clusterDomain value to operator/values.yaml, defaulting to cluster.local (no change to existing behavior), and uses it in the endpoint template instead of the hardcoded literal.
Note on prior attempt
I saw that #2921 previously tried to address this issue and was closed — that PR referenced a nonexistent image and changed the Minio image tag/version instead of actually touching the hardcoded endpoint, so it didn't fix the reported problem. This PR only touches the clusterDomain value and the one template line that builds the endpoint string; it doesn't change the Minio image, tag, or any chart dependency.
Testing
I wasn't able to verify this against a live cluster with a custom cluster domain, but the chart-level behavior (default preserved, override applied) is covered by the test above.