Skip to content

feat: go-live mctl-agents-worker on 1.22.0 - #715

Merged
mashkovd merged 1 commit into
mainfrom
gitops/mctl-agents-worker-go-live
Aug 5, 2026
Merged

mashkovd merged 1 commit into
mainfrom
gitops/mctl-agents-worker-go-live

Conversation

@mashkovd

@mashkovd mashkovd commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

Test plan

  • helm lint clean.
  • helm template renders replicas: 1, correct image:, no livenessProbe/readinessProbe blocks.
  • Live: pod reaches Running, connects to Temporal, worker registers task queue mctl-dev-loop.

orchestrator/temporal/ (mctl-agents PR #105) shipped in release 1.22.0.
Bumps replicaCount 0->1 and image.tag 1.21.0->1.22.0 together, per the
crash-loop warning this file has carried since its creation (#713).

Verified: helm lint clean, helm template renders replicas: 1 and the
correct image ref with no probe blocks.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@claude

claude Bot commented Aug 5, 2026 •

Copy link
Copy Markdown

Claude finished @mashkovd's task in 40s —— View job


Review Analysis

Status: ✅ Trivial diff — approved

Changes reviewed:

  • Image tag bump: 1.21.0 → 1.22.0 ✓
  • Replica count: 0 → 1 ✓
  • Both changes bundled in same commit (avoids crash-loop) ✓

Verification:

  • helm lint: passed ✓
  • helm template: renders replicas: 1, correct image tag, no inherited probes ✓
  • Probes correctly nulled (no httpGet on non-HTTP worker) ✓
  • Resources reasonable for polling-only workload ✓
  • ExternalSecret properly configured for MCTL_TOKEN ✓
  • No security or operational issues ✓

This is a pure config change to enable the mctl-agents-worker on the orchestrator/temporal module, which first shipped in mctl-agents 1.22.0. No logic changes, no Helm modifications. Ready to merge.


@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trivial config change: image tag 1.21.0→1.22.0 and replica count 0→1 bundled together to enable orchestrator/temporal worker. Helm lint and template validation passed. Good to merge.

@mashkovd
mashkovd merged commit e1d2088 into main Aug 5, 2026
8 checks passed
@mashkovd
mashkovd deleted the gitops/mctl-agents-worker-go-live branch September 2, 2026 21:24
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