From 8a9cdf35fdc9348bebfa96e6679abd2b3ccb6a06 Mon Sep 17 00:00:00 2001
From: Ben Vincent <ben@unkin.net>
Date: Sun, 12 Jul 2026 23:22:04 +1000
Subject: [PATCH] Bump bind-operator to v0.2.3 (deterministic render) (#255)

## Why

v0.2.2's config-hash rolling restart exposed a latent non-determinism: `client.List` returns cache-ordered results, so the resolver's forward zones reshuffled every reconcile, flipping the config hash and rolling `bind-resolvers-2` endlessly. v0.2.3 (bind-operator #11) sorts every rendered list so `named.conf` is byte-identical across reconciles and the hash is stable.

## Changes

- Bump the operator image (`bind-system/deployment.yaml`) and the pulled CRD bundle URL (`bind-system/kustomization.yaml`) to `v0.2.3`.
- Bump the `bind-tsig-api` image (`bind-internal/tsig-api/tsig-api.yaml`) to `v0.2.3`.

CRDs are unchanged from v0.2.2 (controller-only change), so the generated kubeconform schemas need no update.

## Validation

- `bind-system` renders with the v0.2.3 CRD bundle; `bind-internal` passes `kubeconform` (56/56); pre-commit clean.

## Deploy note

On deploy the deterministic operator stops churning the ConfigMap; the config hash stabilizes and the stuck resolver rolling update completes, leaving all three pods Ready on one revision.

Reviewed-on: https://git.unkin.net/unkin/argocd-apps/pulls/255
Co-authored-by: Ben Vincent <ben@unkin.net>
Co-committed-by: Ben Vincent <ben@unkin.net>
---
 apps/base/bind-internal/tsig-api/tsig-api.yaml | 2 +-
 apps/base/bind-system/deployment.yaml          | 2 +-
 apps/base/bind-system/kustomization.yaml       | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/apps/base/bind-internal/tsig-api/tsig-api.yaml b/apps/base/bind-internal/tsig-api/tsig-api.yaml
index 0cdf3d4..d40d2c6 100644
--- a/apps/base/bind-internal/tsig-api/tsig-api.yaml
+++ b/apps/base/bind-internal/tsig-api/tsig-api.yaml
@@ -13,7 +13,7 @@ metadata:
   name: bind-tsig-api
   namespace: bind-internal
 spec:
-  image: git.unkin.net/unkin/bind-tsig-api:v0.2.2
+  image: git.unkin.net/unkin/bind-tsig-api:v0.2.3
   replicas: 1
   port: 8443
   # targetNamespace defaults to this resource's namespace (bind-internal), where
diff --git a/apps/base/bind-system/deployment.yaml b/apps/base/bind-system/deployment.yaml
index c68486c..aaf72ac 100644
--- a/apps/base/bind-system/deployment.yaml
+++ b/apps/base/bind-system/deployment.yaml
@@ -21,7 +21,7 @@ spec:
         runAsNonRoot: true
       containers:
         - name: operator
-          image: git.unkin.net/unkin/bind-operator:v0.2.2
+          image: git.unkin.net/unkin/bind-operator:v0.2.3
           args:
             - --metrics-bind-address=:8080
             - --health-probe-bind-address=:8081
diff --git a/apps/base/bind-system/kustomization.yaml b/apps/base/bind-system/kustomization.yaml
index e18accb..3b57528 100644
--- a/apps/base/bind-system/kustomization.yaml
+++ b/apps/base/bind-system/kustomization.yaml
@@ -6,6 +6,6 @@ resources:
   - namespace.yaml
   # CRDs are pulled from the bind-operator repo at the matching tag rather than
   # vendored here, so they never drift from the operator.
-  - https://git.unkin.net/unkin/bind-operator/raw/tag/v0.2.2/config/crd/install.yaml
+  - https://git.unkin.net/unkin/bind-operator/raw/tag/v0.2.3/config/crd/install.yaml
   - rbac.yaml
   - deployment.yaml