fix: remove ServerSideApply from argo-rollouts #225

claude · 2026-02-22T21:17:59+01:00

claude commented

2026-02-22 21:17:59 +01:00

SSA causes perpetual OutOfSync on 5 CRDs. Switch to client-side apply.

claude added 1 commit 2026-02-22 21:18:00 +01:00

fix: remove ServerSideApply from argo-rollouts to resolve CRD drift

AI Review / AI Code Review (pull_request) Successful in 3s

Details

PR Checks / Validate & Security Scan (pull_request) Successful in 11s

Details

b2b1d594e7

SSA causes perpetual OutOfSync on CRDs due to field manager conflicts.
Client-side apply works correctly for Helm charts with CRDs.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

claude commented

2026-02-22 21:18:14 +01:00

AI Code Review

Review: fix/argo-rollouts-no-ssa

Статус: ✅ APPROVE — изменение корректно решает проблему OutOfSync.

Анализ

Проблема:

SSA (ServerSideApply) вызывает perpetual OutOfSync на 5 CRDs argo-rollouts
ignoreDifferences + RespectIgnoreDifferences=true пытались замаскировать drift, но не решили корень

Решение:

Переключение на client-side apply (удаление ServerSideApply=true)
Удаление ignoreDifferences — больше не нужен без SSA
Удаление RespectIgnoreDifferences=true — следствие выше

Проверки безопасности и надёжности

✅ Security: Нет регрессии — argo-rollouts manifests уже подчиняются PSS (не привилегированы)

✅ Reliability: Client-side apply более стабилен для CRDs с complex schema. Sync будет детерминирован.

✅ RBAC: ArgoCD SA уже имеет права на CRD mutation — изменение метода apply не требует new permissions

✅ Resource management: Не изменено

Замечания

⚠️ Проверить:

После merge — проверить ArgoCD UI, что argo-rollouts sync переходит в Synced и остаётся в этом состоянии (мониторить 15+ минут)
Убедиться, что no rollout canaries в полёте во время этого change (puede быть гонка со старыми SSA fieldOwnershipConflicts)

Вопрос для CHANGELOG: Задокументировано ли в /srv/server-manifest/CHANGELOG.md, что argo-rollouts v2.39.1 SSA-incompatible? Или это первый раз встречаем такую проблему?

Вывод: Логичное fix, low risk. После merge дать ~20 минут на stabilization, потом закрыть.

Automated review by Claude Haiku | advisory, does not block merge

## AI Code Review # Review: fix/argo-rollouts-no-ssa **Статус:** ✅ **APPROVE** — изменение корректно решает проблему OutOfSync. ## Анализ **Проблема:** - SSA (ServerSideApply) вызывает perpetual OutOfSync на 5 CRDs argo-rollouts - `ignoreDifferences` + `RespectIgnoreDifferences=true` пытались замаскировать drift, но не решили корень **Решение:** - Переключение на client-side apply (удаление `ServerSideApply=true`) - Удаление `ignoreDifferences` — больше не нужен без SSA - Удаление `RespectIgnoreDifferences=true` — следствие выше ## Проверки безопасности и надёжности ✅ **Security:** Нет регрессии — argo-rollouts manifests уже подчиняются PSS (не привилегированы) ✅ **Reliability:** Client-side apply более стабилен для CRDs с complex schema. Sync будет детерминирован. ✅ **RBAC:** ArgoCD SA уже имеет права на CRD mutation — изменение метода apply не требует new permissions ✅ **Resource management:** Не изменено ## Замечания ⚠️ **Проверить:** 1. После merge — проверить ArgoCD UI, что argo-rollouts sync переходит в **Synced** и остаётся в этом состоянии (мониторить 15+ минут) 2. Убедиться, что no rollout canaries в полёте во время этого change (puede быть гонка со старыми SSA fieldOwnershipConflicts) **Вопрос для CHANGELOG:** Задокументировано ли в `/srv/server-manifest/CHANGELOG.md`, что argo-rollouts v2.39.1 SSA-incompatible? Или это первый раз встречаем такую проблему? --- **Вывод:** Логичное fix, low risk. После merge дать ~20 минут на stabilization, потом закрыть. --- _Automated review by Claude Haiku | advisory, does not block merge_

admin merged commit 37209e9983 into main

2026-02-22 21:22:01 +01:00

admin referenced this issue from a commit

2026-02-22 21:22:02 +01:00

Merge pull request 'fix: remove ServerSideApply from argo-rollouts' (#225) from fix/argo-rollouts-no-ssa into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: claude/k8s-apps#225