Add blog enhancing-vpn-high-availability-with-round-robin-bonding-mode - #823
Conversation
gardener-prow
Bot
added
do-not-merge/work-in-progress
ghost
added
the
reviewed/ok-to-test
|
LGTM label has been added. DetailsGit tree hash: 103de8738963fb495ab5d4068fb4383ca774f1d0 |
gardener-prow
Bot
added
the
approved
|
ping @domdom82 |
|
The Gardener project currently lacks enough active contributors to adequately respond to all PRs.
You can:
/lifecycle stale |
gardener-prow
Bot
added
the
lifecycle/stale
|
@domdom82 Hi, could you please take a look at this PR? |
gardener-prow
Bot
added
cla: no
|
@domdom82 Could you review this PR, please? :) |
✅ Deploy Preview for gardener-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughChangesVPN round-robin documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
The base branch for this PR is extremely outdated. That's why the preview fails. |
gardener-prow
Bot
added
cla: no
|
The Gardener project currently lacks enough active contributors to adequately respond to all PRs.
You can:
/lifecycle rotten |
gardener-prow
Bot
added
lifecycle/rotten
BoHristova
added
lifecycle/frozen
gardener-prow
Bot
removed
the
do-not-merge/needs-kind
…h-round-robin-bonding-mode.md
Signed-off-by: Bozhidara Hristova <bozhidara.hristova@sap.com>
deab4f3 to
819115a
Compare
|
/lgtm |
|
@domdom82: adding LGTM is restricted to approvers and reviewers in OWNERS files. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
gardener-prow
Bot
removed
the
do-not-merge/work-in-progress
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BoHristova, n-boshnakov The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
LGTM label has been added. DetailsGit tree hash: 4927f2bdabb33b080ae6f6190a0a33e8746787f2 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md (2)
35-37: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winState the operational limits of the alpha feature.
VPNBondingModeRoundRobinis an alpha feature. State that alpha features may be buggy, may change incompatibly, and may be removed without notice. Link to the Feature Gates documentation before advising operators to enable it. (gardener.cloud)🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md` around lines 35 - 37, Update the `VPNBondingModeRoundRobin` introduction to state that alpha features may be buggy, change incompatibly, or be removed without notice. Add a link to the Feature Gates documentation before instructing operators how to enable the gate.Source: MCP tools
29-29: 🚀 Performance & Scalability | 🔵 Trivial | ⚡ Quick winDocument the round-robin trade-off.
If
balance-rrstripes packets across tunnels with different path latency, packets can arrive out of order and TCP can retransmit them. Add a short limitation and any Gardener-specific network prerequisites here. Otherwise, readers may interpret round-robin as an unconditional availability improvement. (docs.kernel.org)🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md` at line 29, Add a brief limitation to the round-robin bonding description noting that unequal tunnel latency can cause packet reordering and TCP retransmissions. Also document the required Gardener-specific network prerequisites for using this mode, while clarifying that it is not an unconditional availability improvement.Source: MCP tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md`:
- Line 23: Update the headings “The Challenge with Active-Backup Bonding” and
the other affected section headings in this article from level-three headings to
level-two headings, ensuring the document outline starts at ## and satisfies
markdownlint MD001.
---
Nitpick comments:
In
`@hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md`:
- Around line 35-37: Update the `VPNBondingModeRoundRobin` introduction to state
that alpha features may be buggy, change incompatibly, or be removed without
notice. Add a link to the Feature Gates documentation before instructing
operators how to enable the gate.
- Line 29: Add a brief limitation to the round-robin bonding description noting
that unequal tunnel latency can cause packet reordering and TCP retransmissions.
Also document the required Gardener-specific network prerequisites for using
this mode, while clarifying that it is not an unconditional availability
improvement.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro Plus
Run ID: 07adecc5-0991-43af-ad43-14bf32a33363
📒 Files selected for processing (1)
hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md
|
|
||
| To address this, Gardener now introduces an alternative bonding mode: `round-robin`. | ||
|
|
||
| ### The Challenge with Active-Backup Bonding |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Promote these section headings to ##.
Line 23 is the first content heading, but it uses ### without a preceding ##. The same issue appears at Lines 27, 33, and 40. This triggers markdownlint MD001 and creates an invalid document outline.
Proposed fix
-### The Challenge with Active-Backup Bonding
+## The Challenge with Active-Backup Bonding
...
-### A More Resilient Approach: Round-Robin Bonding
+## A More Resilient Approach: Round-Robin Bonding
...
-### How to Enable the Feature
+## How to Enable the Feature
...
-### Further Reading
+## Further ReadingAlso applies to: 27-27, 33-33, 40-40
🧰 Tools
🪛 markdownlint-cli2 (0.23.2)
[warning] 23-23: Heading levels should only increment by one level at a time
Expected: h2; Actual: h3
(MD001, heading-increment)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In
`@hugo/content/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md`
at line 23, Update the headings “The Challenge with Active-Backup Bonding” and
the other affected section headings in this article from level-three headings to
level-two headings, ensuring the document outline starts at ## and satisfies
markdownlint MD001.
Source: Linters/SAST tools
7 participants
Purpose
@domdom82 This is an automatically generated draft pull request proposing a new blog post based on your Gardener review meeting presentation you gave on 2026-01-28 titled:
The purpose of the blog post is to actively inform the community about new Gardener features or changes, as discussed during review meetings.
Notes to Reviewers
This draft was automatically generated by LLMs using the review meeting recording and referenced materials.
Please evaluate whether this topic is suitable for a blog post. If so, review and edit the content as needed.
If you decide the topic isn't appropriate for a blog post, feel free to close this PR and delete the branch.
Instructions for Reviewers
❌ If the draft isn't viable
✏️ If the draft is viable but requires editing
git clone https://github.com/gardener/documentation cd documentationgit fetch origin && git checkout blog/2026-01-28-enhancing-vpn-high-availability-with-round-robin-bonding-modewebsite/blog/2026/01/01-28-enhancing-vpn-high-availability-with-round-robin-bonding-mode.md.✅ If the draft is ready for review
/lgtmto approve (required step)The documentation team will review your PR, as required by branch protection.
They will merge it once you (and any additional reviewers) have approved it.
@domdom82 Thank you for helping us share valuable updates from the Gardener project with the community!
Summary by CodeRabbit