HYPERFLEET-927 - fix: bump gRPC to v1.79.3 (CVE-2026-33186) + advisory lock race fix#105
Conversation
…actions Fixes race condition where advisory locks could be acquired within existing transactions, causing locks to be released before transactions commit. - Add fail-fast validation in NewAdvisoryLockContext to detect pre-existing transactions - Replace TestAdvisoryLocksWithTransactions with TestRowLevelConcurrency demonstrating correct SELECT FOR UPDATE pattern - Add TestAdvisoryLockFailsWithExistingTransaction to validate fail-fast behavior - Update documentation to clarify advisory locks are for pod coordination, not row-level concurrency update
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
b9d5825
into
openshift-hyperfleet:release-0.2
…2 configuration Enables revive linter and resolves all lint violations. Extracts operation constants to fix goconst. Aligns golangci.yml with the architecture standard from openshift-hyperfleet/architecture PR openshift-hyperfleet#105, enabling govet.enable-all and resolving all 178 resulting violations (147 fieldalignment, 31 shadow).
HYPERFLEET-857 - refactor: update Sentinel config from Ready to Reconciled
…-1023 HYPERFLEET-1023 - refactor: migrate AfterEach cleanup to DeferCleanup
Summary
Cherry-picks for z-stream release 0.2.1:
google.golang.org/grpcto v1.79.3 to fix CVE-2026-33186 (minimal dependency bump, no OpenTelemetry refactor)Test plan
make verify-allpasses (721 unit tests)