Org: Migrate packethost/cluster-api-provider-packet to kubernetes-sigs

Created on 15 Jun 2020 · 19Comments · Source: kubernetes/org

New Repo, Staging Repo, or migrate existing

Migrate existing packethost/cluster-api-provider-packet

Requested name for new repository

cluster-api-provider-packet

Which Organization should it reside

kubernetes-sigs

If not a staging repo, who should have admin access

@mrmrcoleman @deitch

If not a staging repo, who should have write access

@gianarb @deitch @jacobsmith928 @mrmrcoleman probably we need a team for it: cluster-api-packet-maintainers

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

This is the issue that tracks this work: https://github.com/packethost/cluster-api-provider-packet/issues/114

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

This is the issue that tracks this work: https://github.com/packethost/cluster-api-provider-packet/issues/121

What should the repo description be

"Packet Cloud Provider for Cluster API"

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

sig-cluster-lifecycle

Approvals

Wed 10 June during cluster-api Office Hour, we raised our desire to move the repo under kubernetes-sigs. The video will be linked as soon as available.

Additional context for request

16th June I will bring this topic to sig-cluster-lifecycle be-weekly meeting because for now I just had a chat in ClusterAPI meeting. Will update this PR with more information

aregithub-repo

Source

gianarb

Most helpful comment

Thanks a lot for your support!! We are happy and excited about all of this!! :D Now let's try to make @fejta-bot happy!

gianarb on 26 Jun 2020

❤2 🎉2 😄2

All 19 comments

/assign @justinsb @neolit123 @timothysc @fabianofranz
For sign off

After sign off from the leads, @deitch @jacobsmith928 @mrmrcoleman will need to apply for membership to kubernetes-sigs 👍

mrbobbytables on 15 Jun 2020

👍2

Aside from what @mrbobbytables has detailed in https://github.com/kubernetes/org/issues/1934#issuecomment-644100402, the repo will need some more updates before it is ready for migration.

As per the rules, all contributors must have signed the CNCF CLA. @rkoster has not signed the CLA yet (@rkoster -- you can find instructions for singing the CLA here).
Add all template files including code-of-conduct.md, CONTRIBUTING.md, OWNERS, etc as per https://github.com/kubernetes/kubernetes-template-project. Please also add a note in your README for https://github.com/kubernetes/kubernetes-template-project#community-discussion-contribution-and-support.
The LICENSE needs to be updated to Apache v2.
All files need the standard Kubernetes copyright header. You can find the boilerplate for different file extensions here. Please also update the boilerplate used by generator tools.
Once everything is ready, please add me and @mrbobbytables as admins to the repo and we can migrate it to @kubernetes-sigs. :)

nikhita on 15 Jun 2020

❤1

S&W had already signed: https://identity.linuxfoundation.org/content/cncf-stark-wayne, but apparently I did not yet connect my linux foundation account to github. It should all be set now.

rkoster on 15 Jun 2020

🎉1

@rkoster Thanks for the quick response! Can confirm that the CLA looks fine now :smile:

nikhita on 15 Jun 2020

🎉1

hello, please join the sig-cluster-lifecycle meeting Tomorrow, where we can give our thumbs up on the migration. the item is on the agenda.

neolit123 on 15 Jun 2020

👍1

/unassign @fabianofranz
/assign @fabriziopandini

neolit123 on 15 Jun 2020

/unassign @fabianofranz
/assign @fabriziopandini

Oh woops, sorry about that misassign >_> auto-complete got me.

mrbobbytables on 15 Jun 2020

👀1 ❤1

thank you for joining the meeting today @gianarb
we've +1ed for the migration from the SIG CL standpoint.
/lgtm

the subproject addition is in this PR, but we requested an amend to it:
https://github.com/kubernetes/community/pull/4861
(to add it as cluster-api-provider-packet under SIG CL and not SIG CP).
(EDIT: looks like that is done now)

neolit123 on 16 Jun 2020

@gianarb tiny nit: the copyright header needs to have newlines. example https://github.com/packethost/cluster-api-provider-packet/blob/master/controllers/packetcluster_controller.go#L3 and https://github.com/kubernetes/kubernetes/blob/master/hack/boilerplate/boilerplate.go.txt#L3 for comparison.

Can you ping here once all membership requests are processed as per https://github.com/kubernetes/org/issues/1934#issuecomment-644100402? Will migrate the repo after that. :+1:

nikhita on 17 Jun 2020

Some more small updates needed:

The repo has an OWNERS file that mentions aliases like cluster-api-packet-maintainers but doesn't have an OWNERS_ALIASES file at the root to define these aliases. You can find the format for the aliases file here.
Can you also add a note in the README for the "Community, discussion, contribution, and support" section as per https://github.com/kubernetes/kubernetes-template-project#community-discussion-contribution-and-support? This is to ensure that all repos have links to slack channels, mailing lists and a link to the CoC in the README.

nikhita on 17 Jun 2020

@nikhita thank you for your help.
I think we are all set! Do you agree? You and @mrbobbytables are admin for packethost/cluster-api-provider-packet

Thanks

gianarb on 22 Jun 2020

@gianarb There are a few things still outstanding :grimacing:

The link to the slack channels, CoC still needs to be added in the README as per https://github.com/kubernetes/org/issues/1934#issuecomment-645170323
There are still a few copyright headers that need newlines. Example - https://github.com/packethost/cluster-api-provider-packet/blob/master/main.go#L2
@deitch @jacobsmith928 @mrmrcoleman will need to apply for membership to kubernetes-sigs (https://github.com/kubernetes/org/issues/1934#issuecomment-644100402)

nikhita on 22 Jun 2020

For the record -- I discussed with @gianarb, we are sorting out some org level settings around who can transfer the repo. Everything looks good otherwise, I'll migrate the repo once the settings have been fixed. :+1:

@gianarb also thank you for taking care of all the prerequisites and being so responsive with everything! :heart:

nikhita on 25 Jun 2020

🚀1 👍1

the up to date list for this:
https://github.com/packethost/cluster-api-provider-packet/blob/master/OWNERS_ALIASES#L4-L7

is here:
https://github.com/kubernetes-sigs/cluster-api/blob/master/OWNERS_ALIASES#L4-L8

but we can update it later too.

neolit123 on 25 Jun 2020

Repo has been migrated - https://github.com/kubernetes-sigs/cluster-api-provider-packet :tada:

Teams have been granted access and sigs.yaml has been updated.

@neolit123 -- also, OWNERS_ALIASES has been updated in https://github.com/kubernetes-sigs/cluster-api-provider-packet/pull/138 :smile:

/close

nikhita on 26 Jun 2020

@nikhita: Closing this issue.

In response to this:

Repo has been migrated - https://github.com/kubernetes-sigs/cluster-api-provider-packet :tada:

Teams have been granted access and sigs.yaml has been updated.

@neolit123 -- also, OWNERS_ALIASES has been updated in https://github.com/kubernetes-sigs/cluster-api-provider-packet/pull/138 :smile:

/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.