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

New repo kubernetes-sigs/ingress-controller-conformance #1328

Closed
alexgervais opened this issue Oct 21, 2019 · 14 comments
Closed

New repo kubernetes-sigs/ingress-controller-conformance #1328

alexgervais opened this issue Oct 21, 2019 · 14 comments
Assignees
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository sig/network Categorizes an issue or PR as relevant to SIG Network.

Comments

@alexgervais
Copy link
Contributor

alexgervais commented Oct 21, 2019

New Repo, Staging Repo, or migrate existing

migrate existing (https://github.com/datawire/ingress-conformance)

Requested name for new repository

ingress-controller-conformance

Which Organization should it reside

kubernetes-sigs

If not a staging repo, who should have admin access

bowei, thockin

If not a staging repo, who should have write access

bowei, thockin, rhs, alexgervais

If not a staging repo, who should be listed as approvers in OWNERS

bowei, thockin

If not a staging repo, who should be listed in SECURITY_CONTACTS

bowei, thockin, rhs, alexgervais

What should the repo description be

Repository for a compliance specification of ingress-controllers.

What SIG and subproject does this fall under in sigs.yaml

sig-network

Approvals

thockin
https://groups.google.com/forum/#!topic/kubernetes-sig-network/dAt78O-JUCo

Additional context for request

The goal of this project is to act as an executable specification in the form of a test suite, implementing a standard, actively maintained ingress-controller conformance specification.

@alexgervais alexgervais added the area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository label Oct 21, 2019
@mrbobbytables
Copy link
Member

@alexgervais just for clarification, are you looking for a bare repository? or to migrate datawire/ingress-performance to kubernetes-sigs?

If it's a migration or if the code is copied -- there are a few more steps involved to donate it to the Kubernetes project. For the specifics on that, please see the docs for it here:
https://git.k8s.io/community//github-management/kubernetes-repositories.md#rules-for-donated-repositories

@alexgervais
Copy link
Contributor Author

@mrbobbytables We definitely want to contribute the code from datawire/ingress-performance. My thought was to create a bare repository and open a PR with the existing code, thus enforcing copywrite, licenses and other boilerplate.
All contributors to datawire/ingress-performance already signed the CNCF CLA.

I'm open to suggestions to make the copy/move as easy and transparent as possible.

@mrbobbytables
Copy link
Member

I think that should be fine -- I don't see any dependencies that are outside the norm either. @nikhita what do you think?

@nikhita
Copy link
Member

nikhita commented Oct 21, 2019

I'd suggest migrating/transfering the repo to the kubernetes-sigs org instead of going with a bare repo because it means different things with regards to copyright (IANAL though...). The When I donate my project, am I transferring my copyrights? part here has more details: https://github.com/kubernetes/community/blob/master/github-management/kubernetes-repositories.md#faq.

Like @mrbobbytables mentioned, there are additional things required for migrating repos. Could you add the required files (LICENSE, code of conduct, etc) as mentioned here: https://git.k8s.io/community//github-management/kubernetes-repositories.md#rules-for-donated-repositories? Thanks!

@alexgervais
Copy link
Contributor Author

The LICENSE and code-of-conduct.md files were added to the source repository.
The standard Kubernetes header to code can occur post-transfer.
@nikhita , @mrbobbytables please advise if you see any gaps!

@thockin
Copy link
Member

thockin commented Oct 29, 2019

/lgtm
/approve

@nikhita
Copy link
Member

nikhita commented Oct 30, 2019

@alexgervais Two more things need to be done pre-transfer, but looks good otherwise. 👍

/assign

@rhs
Copy link

rhs commented Oct 30, 2019

@nikhita @alexgervais Hi, I changed the one merge that was rhs@alum.mit.edu to rhs@datawire.io. I can see a corporate CLA in my account already, and I just signed an individual one. Please let me know if there is anything else you need from me. Thanks!

@nikhita
Copy link
Member

nikhita commented Oct 31, 2019

@rhs Thanks for handling that so quick!

Everything looks good. 👍 Please add me as an admin to the repo so that I can transfer it. :)

@nikhita
Copy link
Member

nikhita commented Oct 31, 2019

Repo has been migrated https://github.com/kubernetes-sigs/ingress-controller-conformance 🎉

Also created #1369 to create GitHub teams to grant access, and kubernetes/community#4208 to add the repo to sigs.yaml.

@alexgervais @rhs to provide write access to you, you'll need to be in the @kubernetes-sigs github org. Can you apply for membership to the org? Once you are added as members, feel free to PR yourselves to the team created in #1369. :)

@alexgervais
Copy link
Contributor Author

Thank you @nikhita & @mrbobbytables

@nikhita
Copy link
Member

nikhita commented Nov 4, 2019

I'll keep this open until kubernetes/community#4208 is merged, so that it doesn't get lost. :)

@nikhita nikhita reopened this Nov 4, 2019
@nikhita
Copy link
Member

nikhita commented Nov 15, 2019

kubernetes/community#4208 has merged
/close

@k8s-ci-robot
Copy link
Contributor

@nikhita: Closing this issue.

In response to this:

kubernetes/community#4208 has merged
/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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository sig/network Categorizes an issue or PR as relevant to SIG Network.
Projects
None yet
Development

No branches or pull requests

6 participants