Skip to content

fix(observability): roll Alloy on config changes#12398

Merged
gregkonush merged 2 commits into
mainfrom
codex/observability-alloy-config-rollout
Jul 13, 2026
Merged

fix(observability): roll Alloy on config changes#12398
gregkonush merged 2 commits into
mainfrom
codex/observability-alloy-config-rollout

Conversation

@gregkonush

@gregkonush gregkonush commented Jul 13, 2026

Copy link
Copy Markdown
Member

Summary

  • Generate the cluster-metrics Alloy ConfigMap with Kustomize content hashing.
  • Move the Alloy River configuration into its native .river source file.
  • Let Kustomize rewrite the Deployment volume reference so every config change triggers a rolling restart.
  • Route .river changes through Argo validation and cover the generated-config contract in tests.

Related Issues

None.

Testing

  • nix develop -c kustomize build --enable-helm argocd/applications/observability
  • nix develop -c sh -lc 'kustomize build --enable-helm argocd/applications/observability | kubectl apply --dry-run=server -n observability -f -'
  • bun run lint:argocd
  • bun test packages/scripts/src (532 passing)
  • bunx oxfmt --check packages/scripts/src/shared/__tests__/flink-observability-contract.test.ts packages/scripts/src/ci/__tests__/impact-router.test.ts
  • Compared the extracted River payload with origin/main; no configuration content changed.
  • Verified the rendered Deployment references the generated hashed ConfigMap.

Screenshots (if applicable)

N/A. Manifest-only change.

Breaking Changes

None. Argo CD will prune the old unhashed ConfigMap after the Alloy rollout.

Checklist

  • Testing section documents the exact validation performed.
  • Screenshots and Breaking Changes sections are handled appropriately.
  • Documentation, release notes, and follow-ups are updated or tracked; no documentation change is required for this internal rollout mechanism.

@gregkonush
gregkonush merged commit f1890fe into main Jul 13, 2026
11 checks passed
@gregkonush
gregkonush deleted the codex/observability-alloy-config-rollout branch July 13, 2026 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant