Repository navigation
fix(machinery): drop the openebs Loki/MinIO/Alloy stack from machinery clusters - #1102
Merged
Merged
Conversation
…y clusters The openebs chart ships an observability stack enabled by default: a Loki StatefulSet, the MinIO StatefulSet backing it, an Alloy DaemonSet, and two extra StorageClasses. The localpv profile this play uses disabled the lvm/zfs/mayastor engines but left all of that on, so every machinery cluster ran seven pods of logging nobody asked for. On a 1+2 kind cluster — which is what every machinery cluster is — it cannot even run. Loki asks for three replicas with pod anti-affinity: the two workers take replicas 1 and 2, the control-plane node is tainted, and replica 0 sits Pending forever. u26-kind3 carried that permanently non-Running pod from the day it was built, and kind1 has the same one. MinIO has no toggle of its own — it is Loki's object store and a sub-dependency, so loki.enabled=false is what removes it. Alloy goes too, since a log shipper with no Loki ships logs at nothing. openebs-hostpath, the StorageClass everything here actually uses, is untouched. To pass the toggles, pre_crossplane_helmfiles entries become dicts with an optional state_values map rendered into --state-values-set. A plain string still works, so a caller overriding the list is unaffected. Requires stuttgart-things/helm#155, which adds the state values (defaulting to true there, so nothing else changes). Against an older helmfile the flags are simply ignored and the stack stays up. Verified on u26-kind3: after the run the openebs namespace holds only openebs-localpv-provisioner, the StorageClass list is down to openebs-hostpath, and the cluster has no non-Running pod left. The three MinIO StatefulSet PVCs survive a Helm removal and were deleted separately.
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The openebs chart ships an observability stack enabled by default: a Loki StatefulSet, the MinIO StatefulSet backing it, an Alloy DaemonSet, and two extra StorageClasses. The
localpvprofile this play uses disables the lvm/zfs/mayastor engines but left all of that on, so every machinery cluster ran seven pods of logging nobody asked for.On a 1+2 kind cluster — which is what every machinery cluster is — it cannot even run. Loki asks for three replicas with pod anti-affinity: the two workers take replicas 1 and 2, the control-plane node is tainted, and replica 0 sits
Pendingforever. u26-kind3 carried that permanently non-Running pod from the day it was built, and kind1 has the same one.Notes
loki.enabled=falseis what removes it.openebs-hostpathis untouched — that is the StorageClass everything here actually uses.Shape change
To pass the toggles,
pre_crossplane_helmfilesentries become dicts with an optionalstate_valuesmap rendered into--state-values-set. A plain string still works (item.location | default(item)), so a caller overriding the list is unaffected.Dependency
Requires stuttgart-things/helm#155, which adds the state values — defaulting to
truethere, so nothing outside this play changes. Against an older helmfile the flags are simply ignored and the stack stays up.Verification
On u26-kind3, after the run:
openebsopenebs-loki-0Pending)openebs-localpv-provisioner)openebs-hostpath,openebs-loki-localpv,openebs-minio-localpv,standardopenebs-hostpath,standardThe three MinIO StatefulSet PVCs survive a Helm removal (volumeClaimTemplates are not Helm-managed) and were deleted separately.
Follows #1098 and #1101.
🤖 Generated with Claude Code
https://claude.ai/code/session_01FSYX5dsdVdzLCuB6xYhaLK