gate hot-standby HA pending paired release - #373
Merged
Conversation
bpopadiuk
marked this pull request as ready for review
July 17, 2026 22:03
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
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.
Codex (GPT-5): Temporarily gates hot-standby HA off for the current Antfly release while preserving an explicit opt-in and manual verification path pending the paired Antfly and Colony changes.
Why
Hot-standby HA is present on current Antfly
main, but it is not release-ready until antflydb/antfly#347 and antflydb/colony#346 land together. Required Zig aggregates also include the HA suites today, so HA-specific flakes can block an otherwise unrelated release build.What changed
--enable-hot-standby-ha, defaulting tofalse, and pin the shipped operator Deployment manifest tofalse.spec.highAvailability.mode: HotStandbyresources at admission while disabled.ConfigurationValid=Falsestatus and backs off.-Dha-teststo Zig aggregate test selection. Required PR, push, merge-queue, and scheduled workflow runs pass-Dha-tests=false; manual workflow dispatch passes-Dha-tests=true.ha-test,ha-chaos-test, and HA compatibility targets available for explicit verification.Cloud boundary
Current Colony
maindoes not yet expose the HA admission/configuration surface, so it remains effectively off without adding an unused environment variable in this release. Colony PR #346 owns the Cloud-side default-offCOLONY_CLOUD_HA_ADMISSION_ENABLEDboundary. The intended rollout is to merge the paired Colony #346 and Antfly #347 changes, verify that exact pair, then explicitly enable both gates.Impact
Non-HA clusters are unchanged. Hot-standby HA requests fail closed with an actionable message until the operator is restarted with
--enable-hot-standby-ha=true. Required CI stops depending on the gated HA suites; manual and standalone HA validation remains available.Validation
GOCACHE=/private/tmp/antfly-ha-release-gate-full-gocache make testfromgo/pkg/operator— passed, including generate, fmt, vet, and all Go tests.go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.12 .github/workflows/zig-tests.yml— passed.zig build -Dha-tests=false --helpwith isolated caches — passed; aggregate option and standalone HA targets are registered.make -n zig-unit-test ZIG_BUILD_FLAGS=-Dha-tests=false— confirmed the root Makefile forwards the gate into the Zig build.git diff --check— passed.This intentionally does not address unrelated failures already present on
main.