Skip to content

fix(ci): recover stuck dev Elasticsearch rollout - #100

Merged
rhanka merged 1 commit into
mainfrom
fix/dev-elasticsearch-statefulset-recovery
Sep 13, 2026
Merged

rhanka merged 1 commit into
mainfrom
fix/dev-elasticsearch-statefulset-recovery

Conversation

@rhanka

@rhanka rhanka commented Sep 13, 2026

Copy link
Copy Markdown
Member

The corrected dev Elasticsearch template could not roll out because the existing single-replica OrderedReady StatefulSet pod was non-ready on the previous controller revision. The deployment now detects that exact state after applying the manifest and recreates only the non-ready, outdated pod, allowing Kubernetes to start the desired revision with the retained PVC.

The recovery runs only from the dev GitHub Actions deployment paths. It does not delete a pod that is ready or already on the desired revision.

Validation:

  • bash deploy/k8s/tests/recover-stuck-statefulset.sh
  • bash -n deploy/k8s/scripts/recover-stuck-statefulset.sh deploy/k8s/tests/recover-stuck-statefulset.sh
  • git diff --check
  • harness check scope
  • harness check branch

External review note: AGY Gemini 3.8 was launched repeatedly with the complete diff but exited after initialization without returning a verdict; the interactive fallback stopped at its local trust prompt before submitting the brief.

@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: ac5256e0-941a-41dc-a4c0-93d13c0499cc


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@rhanka
rhanka merged commit f05e38d into main Sep 13, 2026
6 checks passed
@rhanka
rhanka deleted the fix/dev-elasticsearch-statefulset-recovery branch September 13, 2026 13:33
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.

2 participants