<fix>[lb]: support tcp ipvs listener params#4403
Conversation
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 26 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. 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: Path: http://open.zstack.ai:20001/code-reviews/zstack-cloud.yaml (via .coderabbit.yaml) Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (8)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
28f2de4 to
5463261
Compare
Add dataPlane and forwardMode to TCP listener create and inventory paths so Cloud can select the IPVS data plane for L4 listeners. Reject forwardMode updates after listener creation and reject healthCheckTimeout for TCP IPVS listeners because the IPVS health daemon does not consume that HAProxy-style timeout. Regenerate the load balancer listener API doc templates so the API template check stays clean after docpremium generation. Test: mvn -pl plugin/loadBalancer -am -DskipTests install Test: ./runMavenProfile docpremium Resolves: ZSTAC-86152 Change-Id: I34d94c59651369e7de9d8ae9b33f7e6025300948
5463261 to
b84c1d4
Compare
Expose TCP IPVS balancerAlgorithm and maxConnection.
Return them from create, change, and query inventory paths.
Allow TCP IPVS NAT and DR forward modes in Cloud validation.
Update the create API docs for the supported forward modes.
Cover REST returns and /lb/refresh payloads after create and change.
Test: TcpIpvsLoadBalancerListenerApiCase
Test: mvn test -Dtest=ZStackTest
Test: -DsubCaseCollectionStrategy=Designated
Test: -DcaseFilePath=/tmp/sug2795-tcp-ipvs-case.txt
Test: -Djacoco.skip=true
Resolves: ZSTAC-86152
Change-Id: If8e04660f1c490b9556c35e192c2e39e2dbb66ca
sync from gitlab !10363