org icon indicating copy to clipboard operation
org copied to clipboard

REQUEST: Create new repository for kube-network-policies

Open aojea opened this issue 10 months ago • 9 comments

New repo, staging repo, or migrate existing

New repo

Is it a staging repo?

no

Requested name for new repository

kube-network-policies

Which Organization should it reside

kubernetes-sigs

Who should have admin access?

aojea, danwinship, thockin

Who should have write access?

aojea, danwinship, thockin

Who should be listed as approvers in OWNERS?

aojea, danwinship, thockin

Who should be listed in SECURITY_CONTACTS?

aojea, danwinship, thockin

What should the repo description be?

Kubernetes network policies

What SIG and subproject does this fall under?

sig-network

Please provide references to appropriate approval for this new repo

There are only two jobs that test network policies and are using calico Calico is a third dependency we don't have control and jobs are failing for some time now https://github.com/kubernetes/kubernetes/issues/124130

It turns out that is easier to have our own implementation of network policies, that can also expanded to support admin network policies, than to debug third party network plugins, so I created one in https://github.com/aojea/kube-netpol that I'd like to donate to the kubernetes project

Additional context for request

No response

aojea avatar Apr 01 '24 10:04 aojea

/cc @danwinship @thockin /assign @mrbobbytables

aojea avatar Apr 01 '24 10:04 aojea

/cc @ameukam

aojea avatar Apr 01 '24 10:04 aojea

Requested name for new repository

kube-netpol

Should bikeshed the name a bit; we don't really use "netpol" as an abbreviation anywhere other than test/e2e/network/netpol/. Also, if it's going to be in kubernetes-sigs, does it really need "kube" in its name?

Should it have "reference" or "userspace" or something in the name to better distinguish it from other things?

danwinship avatar Apr 01 '24 12:04 danwinship

+1 FWIW.

we could just use an acronym for the name ? nprc (Network Policy Reference Controller) ?

ameukam avatar Apr 01 '24 15:04 ameukam

Should bikeshed the name a bit

yes please, I prefer something more sounding

👍 praesidium 😄 netshield 😅 nprc ...

aojea avatar Apr 01 '24 15:04 aojea

/hold

vote with the icons above

aojea avatar Apr 01 '24 15:04 aojea

If this is purely a "reference implementation" then it should have a boring, obvious name.

If we intend for it to actually be/become a viable option for use in production clusters, then it could have a more "exciting" name (though it doesn't need to, eg, "kube-proxy"). But that also implies more of a commitment from SIG Network and maybe this needs to be discussed somewhere?

FTR I was objecting to "kube-netpol" solely on the grounds that we don't normally use "netpol" as an abbreviation for "NetworkPolicy".

danwinship avatar Apr 04 '24 14:04 danwinship

If we intend for it to actually be/become a viable option for use in production cluster

I'd like to make this a kube-proxy like but for network policies, not a project in its own, having adoption on users should not be a goal of the project ... having something that helps us to get CI signal and validate APIs is a goal

kube-network-policies then?

aojea avatar Apr 04 '24 14:04 aojea

/retitle REQUEST: Create new repository for kube-network-policies

(fix typo)

danwinship avatar Apr 04 '24 17:04 danwinship

/hold cancel

aojea avatar Apr 17 '24 10:04 aojea

cc @kubernetes/owners 🙏🏾

ameukam avatar Apr 18 '24 13:04 ameukam

/unassign @mrbobbytables

I think he is out until next week, maybe someone else from @kubernetes/owners can take it?

aojea avatar Apr 19 '24 17:04 aojea

/assign

I'll go ahead with creating the new repo

@aojea @danwinship, the requested new repo will be part of which subproject of sig-network? network-policy?

Priyankasaggu11929 avatar Apr 20 '24 06:04 Priyankasaggu11929

New repo: https://github.com/kubernetes-sigs/kube-network-policies

PR to add GitHub teams for the new repo: https://github.com/kubernetes/org/pull/4892

will wait for input on the subproject name https://github.com/kubernetes/org/issues/4856#issuecomment-2067577275 and then create another PR to add documentation in k/community. Thanks!

Priyankasaggu11929 avatar Apr 20 '24 07:04 Priyankasaggu11929

@Priyankasaggu11929 subproject is https://github.com/kubernetes/community/blob/master/sig-network/README.md#network-policy

aojea avatar Apr 20 '24 10:04 aojea

PR to document new repo to k/community - https://github.com/kubernetes/community/pull/7832

Priyankasaggu11929 avatar Apr 20 '24 13:04 Priyankasaggu11929

All PRs are merged. New Repo is ready.

/close

Priyankasaggu11929 avatar Apr 24 '24 06:04 Priyankasaggu11929

@Priyankasaggu11929: Closing this issue.

In response to this:

All PRs are merged. New Repo is ready.

/close

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/test-infra repository.

k8s-ci-robot avatar Apr 24 '24 06:04 k8s-ci-robot