From 3daecdc3bf80bc2cb67134157fd47ba6b64c2956 Mon Sep 17 00:00:00 2001 From: Steve Loeppky Date: Thu, 22 May 2025 12:23:15 -0700 Subject: [PATCH 1/6] Adjusting "proofs" repos to rely on team solely and archived some repos This is handling the "archive the repos", "remove direct collaborators", and "reduce permissions generally" steps of https://github.com/filecoin-project/github-mgmt/issues/132 --- github/filecoin-project.yml | 204 ++++-------------------------------- 1 file changed, 23 insertions(+), 181 deletions(-) diff --git a/github/filecoin-project.yml b/github/filecoin-project.yml index 3e1affc..cd52f87 100644 --- a/github/filecoin-project.yml +++ b/github/filecoin-project.yml @@ -341,9 +341,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -392,11 +389,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - push: - - ognots has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -407,12 +399,8 @@ repositories: teams: admin: - proofs-crate-owners - pull: - - ci push: - - dev-misc - proofs - - research visibility: public blst: archived: true @@ -420,9 +408,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -977,10 +962,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - - porcuquine has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -1236,10 +1217,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - - vmx files: .github/dependabot.yml: content: .github/dependabot.yml @@ -1288,9 +1265,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - vmx has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -1308,9 +1282,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - vmx has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -1510,6 +1481,7 @@ repositories: teams: admin: - lotus-maintainers + push: - proofs visibility: private filecoin-ffi: @@ -1526,7 +1498,7 @@ repositories: teams: admin: - lotus-maintainers - - proofs-crate-owners + push: - proofs visibility: public filecoin-fvm-localnet: @@ -1569,11 +1541,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - push: - - DrPeterVanNostrand has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -2847,26 +2814,7 @@ repositories: go-storage-market: archived: true go-storage-miner: - advanced_security: false - allow_update_branch: false - archived: false - collaborators: - admin: - - magik6k - - whyrusleeping - maintain: - - Kubuxu - has_discussions: false - merge_commit_message: PR_TITLE - merge_commit_title: MERGE_MESSAGE - secret_scanning_push_protection: true - secret_scanning: true - squash_merge_commit_message: COMMIT_MESSAGES - squash_merge_commit_title: COMMIT_OR_PR_TITLE - teams: - maintain: - - proofs - visibility: public + archived: true go-storedcounter: # KEEP: this is a lotus dependency advanced_security: false @@ -2949,9 +2897,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - vmx files: .github/dependabot.yml: content: | @@ -3199,31 +3144,7 @@ repositories: lotus-badges: archived: true lotus-dcops: - advanced_security: false - allow_update_branch: false - archived: false - collaborators: - admin: - - cryptonemo - - schreck23 - pull: - - vvkio - push: - - ognots - - Zorlin - has_discussions: false - merge_commit_message: PR_TITLE - merge_commit_title: MERGE_MESSAGE - secret_scanning_push_protection: false - secret_scanning: false - squash_merge_commit_message: COMMIT_MESSAGES - squash_merge_commit_title: COMMIT_OR_PR_TITLE - teams: - pull: - - proofs - push: - - infra - visibility: private + archived: true lotus-docs: advanced_security: false allow_update_branch: false @@ -3403,11 +3324,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - push: - - arajasek has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -3420,8 +3336,6 @@ repositories: - proofs-crate-owners maintain: - proofs - pull: - - ci visibility: public meta-aggregator-frontend: advanced_security: false @@ -3590,9 +3504,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - vmx has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -3600,6 +3511,9 @@ repositories: secret_scanning: true squash_merge_commit_message: COMMIT_MESSAGES squash_merge_commit_title: COMMIT_OR_PR_TITLE + teams: + push: + - proofs visibility: public PebblingAndDepthReductionAttacks: archived: true @@ -3610,16 +3524,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - porcuquine - pull: - - ribasushi - push: - - cryptonemo - - DrPeterVanNostrand - - jnthnvctr - - vmx has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -3627,6 +3531,9 @@ repositories: secret_scanning: true squash_merge_commit_message: COMMIT_MESSAGES squash_merge_commit_title: COMMIT_OR_PR_TITLE + teams: + push: + - proofs visibility: public phase2: archived: true @@ -3929,11 +3836,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - vmx - pull: - - ognots has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -3946,37 +3848,15 @@ repositories: - proofs-crate-owners maintain: - proofs - pull: - - dev-misc visibility: public rust-fil-nse-gpu: - advanced_security: false - allow_update_branch: false - archived: false - collaborators: - admin: - - porcuquine - has_discussions: false - merge_commit_message: PR_TITLE - merge_commit_title: MERGE_MESSAGE - secret_scanning_push_protection: true - secret_scanning: true - squash_merge_commit_message: COMMIT_MESSAGES - squash_merge_commit_title: COMMIT_OR_PR_TITLE - teams: - admin: - - proofs-crate-owners - visibility: public + archived: true rust-fil-proofs-ffi: archived: true - # NOTE: We don't add create-owners team here because this repo is never published rust-fil-proofs-private: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -3984,21 +3864,16 @@ repositories: secret_scanning: false squash_merge_commit_message: COMMIT_MESSAGES squash_merge_commit_title: COMMIT_OR_PR_TITLE + teams: + admin: + # NOTE: This repo is never published, but we still use this group for admin. + + - proofs-crate-owners visibility: private rust-fil-proofs: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - - jennijuju - pull: - - BlocksOnAChain - - DignifiedquireSyncBot - - ognots - push: - - porcuquine has_discussions: true merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -4009,14 +3884,8 @@ repositories: teams: admin: - proofs-crate-owners - pull: - - ci - - dev-misc - - infra push: - proofs - triage: - - research visibility: public rust-fil-secp256k1: archived: true @@ -4024,14 +3893,10 @@ repositories: archived: true rust-filbase: archived: true - # NOTE: We don't add create-owners team here because this repo is never published rust-filecoin-proofs-api-private: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -4039,14 +3904,17 @@ repositories: secret_scanning: false squash_merge_commit_message: COMMIT_MESSAGES squash_merge_commit_title: COMMIT_OR_PR_TITLE + teams: + admin: + # NOTE: This repo is never published but we use this team for admin. + + - proofs-crate-owners visibility: private rust-filecoin-proofs-api: advanced_security: false allow_update_branch: false archived: false collaborators: - admin: - - cryptonemo has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -4059,18 +3927,11 @@ repositories: - proofs-crate-owners maintain: - proofs - pull: - - ci visibility: public rust-gpu-tools: advanced_security: false allow_update_branch: false archived: false - collaborators: - admin: - - cryptonemo - - porcuquine - - vmx has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -4089,26 +3950,7 @@ repositories: rust-proofs-infra: archived: true rust-sha2ni: - advanced_security: false - allow_update_branch: false - archived: false - collaborators: - maintain: - - vmx - files: - .github/dependabot.yml: - content: .github/dependabot.yml - has_discussions: false - merge_commit_message: PR_TITLE - merge_commit_title: MERGE_MESSAGE - secret_scanning_push_protection: true - secret_scanning: true - squash_merge_commit_message: COMMIT_MESSAGES - squash_merge_commit_title: COMMIT_OR_PR_TITLE - teams: - admin: - - proofs-crate-owners - visibility: public + archived: true sample-data: archived: true sapling-crypto: @@ -4586,7 +4428,6 @@ repositories: - proofs-crate-owners pull: - proofs - - research visibility: public terraform-google-lily-starter: archived: true @@ -5394,6 +5235,7 @@ teams: - rjan90 - rvagg - ZenGround0 + # TODO: rename this to proofs-contributors as part of https://github.com/filecoin-project/github-mgmt/issues/132 proofs: members: maintainer: From a17aac63206274b705c567b810467cf9efe47bf8 Mon Sep 17 00:00:00 2001 From: Steve Loeppky Date: Thu, 22 May 2025 12:29:19 -0700 Subject: [PATCH 2/6] Fix syntax error --- github/filecoin-project.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/github/filecoin-project.yml b/github/filecoin-project.yml index cd52f87..a71de58 100644 --- a/github/filecoin-project.yml +++ b/github/filecoin-project.yml @@ -3914,7 +3914,6 @@ repositories: advanced_security: false allow_update_branch: false archived: false - collaborators: has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE From 60471a62637017c45d2e06549f72605e6d423ba0 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Thu, 22 May 2025 19:31:02 +0000 Subject: [PATCH 3/6] fix@15195201997 [skip fix] --- github/filecoin-project.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/github/filecoin-project.yml b/github/filecoin-project.yml index a71de58..31607e4 100644 --- a/github/filecoin-project.yml +++ b/github/filecoin-project.yml @@ -3867,7 +3867,7 @@ repositories: teams: admin: # NOTE: This repo is never published, but we still use this group for admin. - + - proofs-crate-owners visibility: private rust-fil-proofs: @@ -3907,7 +3907,7 @@ repositories: teams: admin: # NOTE: This repo is never published but we use this team for admin. - + - proofs-crate-owners visibility: private rust-filecoin-proofs-api: From 81f2504b5d3add2eeffcfbec6d3e018a42ff178c Mon Sep 17 00:00:00 2001 From: Steve Loeppky Date: Thu, 22 May 2025 12:43:45 -0700 Subject: [PATCH 4/6] Update permissions of the "proofs" team from PL Inc / Elliptic Research days This is handling the "Adjust membership of the "proofs" team" and "Remove cases where..." steps of https://github.com/filecoin-project/github-mgmt/issues/132 --- github/filecoin-project.yml | 20 ++++++-------------- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git a/github/filecoin-project.yml b/github/filecoin-project.yml index 31607e4..5b7d386 100644 --- a/github/filecoin-project.yml +++ b/github/filecoin-project.yml @@ -2024,7 +2024,6 @@ repositories: - longfeiWan9 - snissn pull: - - DrPeterVanNostrand - WhiteFlamingo push: - jsoares @@ -3191,13 +3190,10 @@ repositories: pull: - turinglabsorg push: - - cryptonemo - - DrPeterVanNostrand - irenegia - LexLuthr - ribasushi - scotthconner - - vmx has_discussions: false merge_commit_message: PR_TITLE merge_commit_title: MERGE_MESSAGE @@ -4906,14 +4902,11 @@ teams: - anorth - magik6k member: - - DrPeterVanNostrand - eshon - frrist - jbenet - jnthnvctr - Kubuxu - - porcuquine - - vmx - whyrusleeping - ZenGround0 - zixuanzh @@ -5219,7 +5212,6 @@ teams: - frrist - jnthnvctr - mishmosh - - porcuquine - protocolin - raulk - whyrusleeping @@ -5238,19 +5230,20 @@ teams: proofs: members: maintainer: - - cryptonemo - member: - - DrPeterVanNostrand - Kubuxu + member: + # 202506: not active, but here in case help is needed in 202506+. + - cryptonemo - nicola - - porcuquine + - rvagg + # 202506: not active, but here in case help is needed in 202506+. - vmx + - ZenGround0 research: members: maintainer: - nicola member: - - DrPeterVanNostrand - irenegia - jbenet - jsoares @@ -5316,7 +5309,6 @@ teams: - mishmosh - momack2 - nicola - - porcuquine - whyrusleeping - yiannisbot - zixuanzh From 31a7f30ea54f144f90d8a22b2a0d3343ffda156b Mon Sep 17 00:00:00 2001 From: Steve Loeppky Date: Tue, 27 May 2025 14:23:43 -0700 Subject: [PATCH 5/6] Incorporating feedback --- github/filecoin-project.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/github/filecoin-project.yml b/github/filecoin-project.yml index 5b7d386..a0d19f4 100644 --- a/github/filecoin-project.yml +++ b/github/filecoin-project.yml @@ -4751,6 +4751,7 @@ teams: - cryptonemo - davidd8 - DiegoRBaquero + - DrPeterVanNostrand - dkkapur - emilymvaughan - eshon @@ -5121,7 +5122,6 @@ teams: - BigLep - jennijuju member: - - cryptonemo - fridrik01 - GlacierWalrus - hannahhoward From 85c6a6b0e63f522a276f56a10ae21fdc7aa13723 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Thu, 29 May 2025 20:51:30 +0000 Subject: [PATCH 6/6] fix@15333314695 [skip fix] --- github/filecoin-project.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/github/filecoin-project.yml b/github/filecoin-project.yml index d27af89..09038c9 100644 --- a/github/filecoin-project.yml +++ b/github/filecoin-project.yml @@ -4619,8 +4619,8 @@ teams: - cryptonemo - davidd8 - DiegoRBaquero - - DrPeterVanNostrand - dkkapur + - DrPeterVanNostrand - emilymvaughan - eshon - figureouter