Skip to content

fix(observability): force Alloy config rollouts#12399

Merged
gregkonush merged 1 commit into
mainfrom
codex/observability-alloy-config-hash
Jul 13, 2026
Merged

fix(observability): force Alloy config rollouts#12399
gregkonush merged 1 commit into
mainfrom
codex/observability-alloy-config-hash

Conversation

@gregkonush

Copy link
Copy Markdown
Member

Summary

  • Keep the generated Alloy ConfigMap name stable for the Argo lovely plugin.
  • Pin the Alloy pod template to the exact SHA-256 of its River configuration so config changes force a rollout.
  • Enforce config/annotation parity in the existing observability contract test.

Related Issues

Follow-up to #12398.

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

Screenshots (if applicable)

N/A. Manifest-only change.

Breaking Changes

None. The stable ConfigMap is recreated and the SHA annotation rolls Alloy once.

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; the config hash contract is covered in tests.

@gregkonush
gregkonush merged commit 67abf61 into main Jul 13, 2026
8 checks passed
@gregkonush
gregkonush deleted the codex/observability-alloy-config-hash branch July 13, 2026 22:53
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