Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automated backport of #2770: Improve podCIDR detection #2797

Conversation

sridhargaddam
Copy link
Member

Backport of #2770 on release-0.15.

#2770: Improve podCIDR detection

For details on the backport process, see the backport requests page.

In K8s, each node is typically assigned a unique PodCIDR range for the
pods that run on that node. Each node's PodCIDR is used to allocate IP
addresses to the pods scheduled on that node. Only if the cluster is a
single node deployment, we should rely on the node.Spec.PodCIDR as
podCIDR of the cluster.

Related to: submariner-io/submariner#2128
Signed-off-by: Sridhar Gaddam <[email protected]>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2797/sridhargaddam/automated-backport-of-#2770-origin-release-0.15
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@tpantelis tpantelis enabled auto-merge (rebase) September 14, 2023 13:29
@tpantelis tpantelis merged commit 1ecb650 into submariner-io:release-0.15 Sep 14, 2023
@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Sep 14, 2023
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2797/sridhargaddam/automated-backport-of-#2770-origin-release-0.15]

@dfarrell07 dfarrell07 added the release-note-needed Should be mentioned in the release notes label Sep 19, 2023
@sridhargaddam
Copy link
Member Author

Release note PR: submariner-io/submariner-website#1042

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated-backport ready-to-test When a PR is ready for full E2E testing release-note-handled release-note-needed Should be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants