mirror of
https://github.com/ngrok/ngrok-operator.git
synced 2026-05-17 16:50:44 +00:00
alex bezek
081d244901
* fix(driver): use pointer in listener status range loop — bug 2.6 The range loop created value copies of ListenerStatus structs, so SetStatusCondition modifications were silently discarded. Changed to index-based iteration with pointer to fix. Bug-ID: 2.6 Co-Authored-By: Claude <noreply@anthropic.com> * fix(driver): sort gateway addresses and ingress status for deterministic comparison — bugs 2.10, 2.11 Gateway status addresses and Ingress LoadBalancer status were built from map iteration with nondeterministic order. reflect.DeepEqual saw false diffs, causing spurious status updates every reconcile. Bug-ID: 2.10, 2.11 Co-Authored-By: Claude <noreply@anthropic.com> * fix(translate): avoid mutating cached gateway cert ref namespace — bug 2.16 Translation code mutated certRef.Namespace on cached Gateway objects from the store, corrupting the informer cache. Replaced with local variable computation. Bug-ID: 2.16 Co-Authored-By: Claude <noreply@anthropic.com> * fix(tests): apply go fix modernizations Applies automatic go fix changes: ptr.To → new for string pointers, range-over-int syntax for loop. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: alex-bezek <8029578+alex-bezek@users.noreply.github.com>