fix(chart): default codexNanocodexRolloutPercent to 0 (opt-in Nanocodex rollout) - #1433
Open
tiagofneto wants to merge 1 commit into
Open
fix(chart): default codexNanocodexRolloutPercent to 0 (opt-in Nanocodex rollout)#1433tiagofneto wants to merge 1 commit into
tiagofneto wants to merge 1 commit into
Conversation
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
1 participant
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.
Problem
The chart defaults
apiRs.codexNanocodexRolloutPercentto50, silently overriding the server's own default of0(args.rs,default_value_t = 0). A fresh production deployment therefore routes half of all Codex sessions to Nanocodex without the operator ever opting in.As an operator I was genuinely surprised to find
session_harness_rollout_resolvedin the logs and realize our Codex threads had been landing on Nanocodex — nothing in the deploy flow hints an A/B experiment is on by default. The subtleCodex*footer in Slack is easy to miss. This is extra sharp while Nanocodex is pinned to an outdated version (#1359).Change
contrib/chart/values.yaml:codexNanocodexRolloutPercent: 50→0, aligning the chart with the server default. The experiment becomes opt-in.docs/pages/reference/configuration.mdx: updated the default in the reference table.No behavior change for deployments that set the value explicitly.