/sig network

[sig-network 2021-06-10] Waiting for SIG Network to decide whether to do this as a built-in or a CRD. Tim wants to see if we can define the type here as a CRD and make the controller built-in first. If we can't figure this out in the next few weeks, we'll just try to make it a built-in.

@rahulkjoshi - to target 1.23, you need PRR approved By Thursday - feel free to assign to me once you open a PR

/milestone v1.23

/stage alpha

Hi @rahulkjoshi! 1.23 Enhancements team here. Just checking in as we approach enhancements freeze at 11:59pm PST on Thursday 09/09. Here's where this enhancement currently stands:

• Updated KEP file using the latest template has been merged into the k/enhancements repo.
• KEP status is marked as implementable
• KEP has a test plan section filled out.
• KEP has up to date graduation criteria.
• KEP has a production readiness review that has been completed and merged into k/enhancements.

I see there's an open PR that will complete those last 2 items, so once that merges we're all good to go!

Thanks!

Hi @rahulkjoshi, 1.23 Enhancements Shadow here 👋🏽
This issue is currently marked as atRisk and I am following up as we are approaching the enhancements freeze deadline at 23:59:59 PDT on Thursday, September 9th.

Looks like the merged KEP number doesn't match the enhancements issue #2593. It is currently pointing to the PR #. Could you please update that?

@supriya-premkumar @salaxander please take a look and let me know if this PR needs any other changes.

Thank you @rahulkjoshi. Looks like this enhancement is all set for the freeze party 🎉
I will update the tracking sheet.

Hi @rahulkjoshi👋🏽 1.23 Enhancements Shadow here
Just checking on the PR status for this issue as we are approaching code freeze deadline on Tuesday, November 16 at 6:00 pm PST

I see that the linked PRs are merged. Are there any other associated PRs for this issue?
Also, please update the respective PRs in the description

Marking this issue as tracked. Please let me know if there are any open PRs.

I think the code will not be merged before the 1.23 deadline. We'll probably merge into 1.24

@sarveshr7

@rahulkjoshi Are there any open PRs? If yes, Could you please link them in the description?
Marking this issue as atRisk for now

To my knowledge, there aren't any open PRs. I'll switch this to be targeting alpha for 1.24 since it seems we're missing the deadline.

Thank you @rahulkjoshi for confirming that this enhancement will target 1.24
/milestone v1.24

Milestoning for 1.28, but @aojea is driving a discussion about overlap with #1880

The perNodeHostBits should be per family.

To have just one for both ipv4/ipv6 enforces a limit on the number of PODs on a node only based on the shortcomings of ipv4. It should be possible to define for instance "perNodeHostBits4: 7" and "perNodeHostBits6: 32"

Some CNI-plugins, like Calico, already allows to configure ipv4 to some (not all) PODs on a node. My own kube-ipam can assign ipv4 addresses only to selected namespaces. I expect this feature, in some form, to be common practice in large cluster in the future.

It is possible to assign large segments to ipv6-only nodes in a cluster, and have a mix of dual-stack and ipv6-only nodes with the current implementation, but that is not enough.

Are we still shooting for Beta in 29?

Are we still shooting for Beta in 29?

There is concerns about moving this API inside the core group as it can cause confusion on users and the networking implementation already have their own mechanisms , see #4174

Hello @rahulkjoshi 👋, v1.29 Enhancements team here.

Just checking in as we approach enhancements freeze on 01:00 UTC, Friday, 6th October, 2023.

This enhancement is targeting for stage beta for v1.29 (correct me, if otherwise)

Here's where this enhancement currently stands:

• KEP readme using the latest template has been merged into the k/enhancements repo.
• KEP status is marked as implementable for latest-milestone: 1.29.
• KEP readme has a updated detailed test plan section filled out
• KEP readme has up to date graduation criteria
• KEP has a production readiness review that has been completed and merged into k/enhancements.

For this KEP, we would just need to update the following:

• Update the latest-milestone and graduation criteria (if any).

The status of this enhancement is marked as at risk for enhancement freeze. Please keep the issue description up-to-date with appropriate stages as well. Thank you!

I believe this should be removed from the milestone following #4253?

Are we still shooting for Beta in 29?

There is concerns about moving this API inside the core group as it can cause confusion on users and the networking implementation already have their own mechanisms , see #4174

/close

@aojea: Closing this issue.

This is now implemented out of tree https://github.com/kubernetes-sigs/node-ipam-controller