-
Notifications
You must be signed in to change notification settings - Fork 706
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
REQUEST: Migrate aquasecurity/vuln-list-k8s #4873
Comments
/sig security |
if its just a subdir of the repo, I would opt to create a new repo and move the code over - there are only 5 commits that touch that dir. |
the primary contributor is going to be @chen-keinan, so please add him too. I (@itaysk) won't be contributing actively, so feel free to remove me, unless I'm needed for redundency. |
Thanks @itaysk I have updated the description now to include @chen-keinan @mrbobbytables we actually worked together to move all the code that we need to migrate into the new branch on this repo: https://github.com/aquasecurity/vuln-list-k8s/tree/migrate-k8s-org So this branch if made as main branch for the new repo with all the other kubernetes repo skeleton (license, etc), we would be golden! Let us know how I can help you and GitHub admin team to make this happen :) |
@PushkarJ if you can, stage all the things needed to be a k8s project first - https://github.com/kubernetes/community/blob/master/github-management/kubernetes-repositories.md#rules-for-donated-repositories the license headers etc (the stuff in the repo skeleton part you mentioned) |
@mrbobbytables Branch is now updated with template files and headers. Please let me know if we missed nothing :) 🙏 |
Hi there, is there anything else required to proceed? |
/assign |
@PushkarJ / @itaysk a couple of items for the first step of migration:
They will need to apply for membership before they can be given admin access to the migrated repo: https://github.com/kubernetes/community/blob/master/community-membership.md#requirements @ericsmalling since you're already part of the Kubernetes org, I can PR you in to the kubernetes-sigs org, you do not need to apply for membership anew. 🙂 However, this should be non-blocking for the migration itself, they can always be added in as and when the org membership requirements are met. Couple more things:
|
@MadhavJivrajani thank you for the info, I have signed the CLA. please do let me know if anything else is required |
@MadhavJivrajani It looks like I've already signed the CLA before. Please let me know if I'm missing something. |
@MadhavJivrajani Also, I have several questions regarding membership requirements.
|
That should be okay 👍🏼
Yes! If you go here: https://github.com/kubernetes/org/issues/new/choose and select "Org Membership Request", you should have the required template to fill out. |
@MadhavJivrajani which users should we put under |
You can put my name as one of the sponsors |
@MadhavJivrajani Can we put your name? Or should we look for someone else? |
@reylejano has offered to be second sponsor |
@PushkarJ @reylejano Thanks for sponsoring. I've opened an issue. |
@PushkarJ @reylejano also want to thank you for the sponsorship. |
Hey @PushkarJ, can you please take a look at the remaining items here: #4873 (comment)? |
@MadhavJivrajani This is probably not possible. Is it okay to keep the code in the same branch and then migrate that branch as main branch in the new repo? Let me know if anything else is missing. I read the whole comment twice to check if anything else is missing. |
@PushkarJ @MadhavJivrajani please put me as admin |
@MadhavJivrajani please let us know if anything else is pending. I summarized all the steps below. Hopefully we covered all the tasks and thank you again for being so descriptive in what was needed to make progress on this!
All are member of k8s Org now as per #5031 and #5032 and #5026
CLA is signed by both #4873 (comment) and #4873 (comment)
If the code to be migrated has to be on "main" branch then @itaysk suggested that he would be open to creating a temporary fork of the source repo where the fork's main branch is == migrate-k8s-org branch in terms of commit history. Let us know if you would prefer that and we can make it happen! |
@PushkarJ thanks for the rundown. Here's what I'm thinking I can do: I can do it in my AM tomorrow. |
The repo is live here: https://github.com/kubernetes-sigs/cve-feed-osv/ 🎉 After these 2 PRs merge, we can close this out: |
/close This is done now, thanks folks! |
@MadhavJivrajani: Closing this issue. In response to this:
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-sigs/prow repository. |
New repo, staging repo, or migrate existing
migrate existing: aquasecurity/vuln-list-k8s
Is it a staging repo?
no
Requested name for new repository
cve-feed-osv
Which Organization should it reside
kubernetes-sigs
Who should have admin access?
chen-keinan,itaysk, knqyf263, tabbysable, iancoldwater, pushkarj
Who should have write access?
pushkarj, ericsmalling
Who should be listed as approvers in OWNERS?
chen-keinan, itaysk, knqyf263, tabbysable, iancoldwater, pushkarj
Who should be listed in SECURITY_CONTACTS?
itaysk, knqyf263, tabbysable, iancoldwater, pushkarj
What should the repo description be?
OSV JSON format file generator for official Kubernetes CVE Feed
What SIG and subproject does this fall under?
sig-security
Please provide references to appropriate approval for this new repo
Meeting minutes: https://docs.google.com/document/d/1GgmmNYN88IZ2v2NBiO3gdU8Riomm0upge_XNVxEYXp0/edit#bookmark=id.kywwheybam91
Lazy Consensus announcement: https://groups.google.com/g/kubernetes-sig-security/c/FxXegIeO198
Additional context for request
We will be creating a separate branch where code from here: https://github.com/aquasecurity/vuln-list-update/tree/main/k8s will be migrated to https://github.com/aquasecurity/vuln-list-k8s
The text was updated successfully, but these errors were encountered: