Repository navigation
fix(opencode): use resource-scoped server-side apply for large config - #126
Merged
Merged
Conversation
Chart CI passedRepository hygiene and Helm validation passed. View the workflow run. |
6 of 7 tasks
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.
Summary
Corrective opencode-server 0.5.2 release after #125: the expanded ConfigMap exceeds the Kubernetes client-side last-applied annotation limit and blocks Argo reconciliation. Add
ServerSideApply=trueonly to the production ConfigMap; retain every primary protocol byte.Type of change
Validation
Adversarial/security/DevOps reviews ADVANCE; QA COVERED; release READY. Deployed Argo controller digest maps to Quay v2.14.15, whose versioned upstream documentation supports this resource-level option for large resources. No migration-disable, Replace, Force, or application-wide SSA option is added.
CI is authoritative; no local validation ran. Existing hygiene, Helm, static policy, source/archive and Python rendering checks must pass. Historical fixture applies exactly the annotation addition, preserving full rendered-resource equality including the derived Deployment checksum; nothing is ignored.
Impact and rollout
Owner approved implement, merge when ready, and rollout. Five files change: production ConfigMap metadata, Chart.yaml, Makefile version/archive assertions, README, existing Python rendering test. Image/appVersion 2.0.22, prompts, ConfigMap data, pilot configuration, models, workflows, PVCs, Secrets, permissions and networking remain unchanged.
Merge triggers main validation and immutable GHCR 0.5.2 publication, then the generated cluster version-pin PR. The parent must update that PR’s existing exact chart/protocol assertion to
(0.5.2, 2)before required checks can pass. Verify artifact publication before consuming merge, then root/child reconciliation and new pod/config delivery separately. Existing Recreate strategy entails brief downtime. No manual live patches or sync actions are needed or authorized here.Rollback selects a prior published chart and matching assertion through reviewed GitOps. 0.5.1 cannot be overwritten and remains known to hit the annotation limit; 0.5.0 is the previous working configuration. Runtime knowledge adherence remains unverified; waived synthetic diagnostic stays stopped.
Safety and secrets
Materially AI-authored. All Critical/High review findings are resolved.
PR CI 37284150966 passed test, detect and package at
b63f8e3f.