Skip to content

Update the known limitation of global ip pool selection - #1101

Merged
w13915984028 merged 2 commits into
harvester:mainfrom
w13915984028:update10592
Aug 12, 2026
Merged

Update the known limitation of global ip pool selection#1101
w13915984028 merged 2 commits into
harvester:mainfrom
w13915984028:update10592

Conversation

@w13915984028

@w13915984028 w13915984028 commented Aug 11, 2026

Copy link
Copy Markdown
Member

Problem:

Global IP pool might be mis-selected.

Solution:

Update it as a known limitation.

Related Issue(s):

harvester/harvester#10592

Test plan:

Additional documentation or context

Original doc PR #1056 was reverted. We re-update the document with this PR #1101 to book the known limitation and workaround.

@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown
Name Link
🔨 Latest commit 8e6dd6a
😎 Deploy Preview https://6a7c63692a74feb238c89baa--harvester-preview.netlify.app

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Harvester documentation to describe and surface a known limitation where guest-cluster LoadBalancer IP allocation may fall back to a global IP pool even when the requested network differs, and links the limitation from the IP pool selection policy docs.

Changes:

  • Adds a troubleshooting entry describing the “unexpected global IP pool selection” behavior and suggested workarounds.
  • Adds notes in IP pool documentation explaining global pool behavior and linking to the troubleshooting entry.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
docs/troubleshooting/rancher.md Adds a troubleshooting section documenting the global IP pool fallback limitation and workarounds.
docs/networking/ippool.md Adds notes about global IP pools and links to the new troubleshooting section.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/networking/ippool.md Outdated
Comment thread docs/troubleshooting/rancher.md Outdated
Comment thread docs/troubleshooting/rancher.md Outdated
Comment thread docs/networking/ippool.md

@jillian-maroket jillian-maroket left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review done

Comment thread docs/troubleshooting/rancher.md Outdated
Comment thread docs/troubleshooting/rancher.md Outdated
Comment thread docs/troubleshooting/rancher.md Outdated
Comment thread docs/networking/ippool.md Outdated
Comment thread docs/networking/ippool.md Outdated
Signed-off-by: Jian Wang <jian.wang@suse.com>
Signed-off-by: Jian Wang <jian.wang@suse.com>
@w13915984028

Copy link
Copy Markdown
Member Author

All comments were addressed and synced to versioned_v1.*, the PR was also rebased with main head, thanks.

@martindekov martindekov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Jian, only 1 question regarding Issue reference, thanks.

Comment thread docs/troubleshooting/rancher.md
@mergify

mergify Bot commented Aug 12, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@w13915984028

Copy link
Copy Markdown
Member Author

I will merge this PR first, and continue to work on PR #1078;

for any additional comments, they will be addressed on #1078, thanks.

@w13915984028
w13915984028 merged commit 97ca6ec into harvester:main Aug 12, 2026
4 checks passed
@w13915984028
w13915984028 deleted the update10592 branch August 12, 2026 14:00
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.

4 participants