fix(deploy): accept collocated reference uploader upgrades - #808
fix(deploy): accept collocated reference uploader upgrades#808proerror77 wants to merge 1 commit into
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Warning Review limit reached
Next review available in: 34 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
baa9ec4 to
310f90a
Compare
310f90a to
3c6624e
Compare
Change contract
Allow a Binance USD-M Reference release produced by this cutover to pass the next upgrade preflight. The old uploader must be collocated with the previous collector release, and its release sidecar plus actual binary SHA-256 must match before any runtime transition starts. The receipt keeps the previous collector release SHA and actual uploader SHA; the obsolete standalone uploader-release field was removed after a full-repo search found no consumer.
Issue relationship
None
Out of scope
Dependencies and merge order
None
Focused validation
deployment/aliyun/test-binance-usdm-reference-release-contract.shdeployment/aliyun/test-binance-usdm-reference-shadow-gate.shdeployment/aliyun/test-binance-usdm-reference-host-gate.shbash -n deployment/aliyun/binance-usdm-reference-cutover.sh deployment/aliyun/test-binance-usdm-reference-host-gate.shshellcheck deployment/aliyun/binance-usdm-reference-cutover.sh deployment/aliyun/test-binance-usdm-reference-host-gate.shgit diff --checkprevious_uploader_release_sha256is absent andprevious_uploader_sha256remains bound to the actual old uploader bytes.The repository preflight script itself exits 141 with the current 190+ worktrees because its
git worktree list | awk ... exitpipeline tripspipefail; the same checks with the consumer draining the full worktree list returnedverdict=ok. This PR does not alter that unrelated script.Rollout and rollback
Code-only change; production was not touched. After merge, publish a new exact-head artifact before opening the separate #803 runtime rollout. Roll back this code by reverting the PR before any candidate staging.
Scope exception
None