Skip to content

Core-LB-G12: Add consistent hashing invariant tests - #345

Merged
RicheyWorks merged 1 commit into
mainfrom
codex/core-lb-g12-consistent-hashing-invariants
May 26, 2026
Merged

Core-LB-G12: Add consistent hashing invariant tests#345
RicheyWorks merged 1 commit into
mainfrom
codex/core-lb-g12-consistent-hashing-invariants

Conversation

@RicheyWorks

Copy link
Copy Markdown
Owner

Overall Classification: WARN
Campaign: Core LoadBalancer Engineering Campaign
Goal: Core-LB-G12
Scope: test-hardening plus narrow contract documentation for consistent hashing invariants

Summary:

  • Adds focused consistent hashing invariant tests for fixed-input deterministic routing.
  • Covers invalid key-count rejection and fail-closed empty/all-unhealthy server sets.
  • Verifies server removal and duplicate replacement do not leave reviewer-visible stale hash allocations.
  • Characterizes zero-load hashing as selected zero-valued buckets.
  • Updates the core load-balancer feature contract with G12 evidence.

Tests added/updated:

  • Added CoreLoadBalancerConsistentHashingInvariantTest.
  • Updated docs/agent/CORE_LOADBALANCER_FEATURE_CONTRACT.md evidence references.

Verification:

  • mvn test "-Dtest=CoreLoadBalancerConsistentHashingInvariantTest" passed after correcting a test sequencing assertion: 6 tests, 0 failures, 0 errors, 0 skipped.
  • mvn test "-Dtest=CoreLoadBalancerConsistentHashingInvariantTest,ConsistentHashRingTest,LoadBalancerTest,CoreLoadBalancerCrossStrategyEdgeInvariantTest,CoreLoadBalancerDeterministicOrderingTieTest,CoreLoadBalancerServerLifecycleInvariantTest,CoreLoadBalancerOverloadRecoveryScenarioTest,AgentCoreLoadBalancerFeatureContractDocumentationTest,AgentCoreLoadBalancerReviewerEvidenceMapDocumentationTest,AgentCoreLoadBalancerEvidenceConsolidationDocumentationTest" passed: 122 tests, 0 failures, 0 errors, 0 skipped.
  • mvn -q test passed.
  • mvn -q "-DskipTests" package passed.
  • mvn -B package passed: 2,542 tests, 0 failures, 0 errors, 0 skipped.
  • git diff --check passed.
  • git diff --cached --check passed.
  • git diff --check origin/main...HEAD passed.
  • .\scripts\smoke\enterprise-lab-workflow.ps1 -Package passed.

Speculative safety gate:

Safety:

  • No production code changed.
  • No Maven config changed.
  • No CI/workflow changed.
  • No Dockerfile/Compose changes.
  • No scripts/runtime resources/endpoints/secrets/external targets/deployment/automation changed.
  • PR Add next-level architecture skeleton #334 was not touched.
  • Core-LB-G13 was not started in this PR.

Not-proven boundaries:

  • No production readiness/certification.
  • No live-cloud or real-tenant validation.
  • No benchmark/load/stress proof.
  • No throughput/p95/p99 proof.
  • No real-world latency improvement claim.
  • No replay/export proof.
  • No broader automation.

@RicheyWorks
RicheyWorks merged commit 7d5c490 into main May 26, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant