docs: audit membership roles and update community_membership.md#4500
docs: audit membership roles and update community_membership.md#4500igooch merged 2 commits intoagones-dev:mainfrom
Conversation
markmandel
added
the
area/meta
Delete stale OWNERS file and update community_membership.md with confirmed approvers/releasers from the agones-dev#4479 membership audit. Adds Current Members section (Approvers, Releasers, Approver Emeritus) and updates role definitions to reference this document as the source of truth for membership, replacing the now-deleted OWNERS file. Signed-off-by: Mark Mandel <[email protected]>
markmandel
force-pushed
the
meta/approvers
branch
from
b17680b to
88eecba
Compare
| ## Current Members | ||
|
|
||
| ### @Approvers | ||
| _Members of Team @Approvers_ |
There was a problem hiding this comment.
Lemme know what you think of putting the team here.
There was a problem hiding this comment.
List looks good to me, that Owners file was several years stale :) Out of curiosity why the change from an Owners file? That seems to be the more standard format, at least for K8s.
There was a problem hiding this comment.
Owners file is (AFAIK)specific to Prow, which we don't use (although we did try!)
Also, CNCF is working on https://github.com/cncf/automation/tree/main/utilities/dot-project - which will end up being the ultimate source of truth for us (we'll make our own), but one thing at a time - we needed a list to get started, and we can always adjust over time.
|
Build Failed 😭 Build Id: adb51464-6d2d-466b-8508-84b556af5231 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
/gcbrun |
|
Build Failed 😭 Build Id: cb903391-6de6-4e67-b6c0-ae9271a72170 Status: FAILURE To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
|
Microsoft registry was 429ing. /Gcbrun |
|
/gcbrun |
|
Build Succeeded 🥳 Build Id: 5950f315-c15d-475c-ad79-8fb55964d177 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
lacroixthomas
left a comment
lacroixthomas
left a comment
There was a problem hiding this comment.
I guess we'll probably need to update the release flow at some point as well ?
Around the release process - needing to work at Google etc 🤔
|
Build Succeeded 🥳 Build Id: ab72aeb3-faed-4623-913d-7c51724cf221 The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version: |
4 participants
What type of PR is this?
/kind feature
What this PR does / Why we need it:
Delete stale OWNERS file and update community_membership.md with confirmed approvers/releasers from the #4479 membership audit.
Adds Current Members section (Approvers, Releasers, Approver Emeritus) and updates role definitions to reference this document as the source of truth for membership, replacing the now-deleted OWNERS file.
Which issue(s) this PR fixes:
Work on #4479 and #4421
Special notes for your reviewer:
Getting there!