-
Notifications
You must be signed in to change notification settings - Fork 229
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
Reconcile Broad permissions for eallred #91
Comments
I've created a temporary "team" for Eric and Paul under Knative Admins here: https://github.com/orgs/knative/teams/tbd-admins |
Are there any reasons for y'all to still have this level of access? I'd like to make headway on this old issue and this feels like this is no longer necessary. Could you please lmk if this is the case? |
And actually I hit send a little bit too soon. The group only contains @eallred-google. |
Feel free to remove me.
With gratitude,
Eric
…On Fri, Jan 29, 2021 at 2:07 AM Ville Aikas ***@***.***> wrote:
And actually I hit send a little bit too soon. The group only contains
@eallred-google <https://github.com/eallred-google>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#91 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALR6L2Z6VJYNYNGU2ZVP4IDS4KCE5ANCNFSM4KTGWWYQ>
.
--
*Eric Allred*
eallred@google.com
Technical Program Manager
Cloud Programs - Serverless
|
Thanks, @eallred-google |
I sent a (linked) PR for @vaikas to stamp dropping the group and the |
Thanks @eallred-google PR has been stamped. |
Is something not working correctly or am I just holding things wrong? And @eallred-google is still a member and tbd-admins is still a member of knative-admin: @mattmoor hold up, wait a minute, something ain't right? Or is it? |
Maybe Peribolos doesn't delete teams? 🤔 /assign @chizhg @evankanderson |
Ok, so looks like this might be an issue. Went looking for the peribolos jobs logs here:
|
Ok, after digging at this, peribolos was silently failing, created an issue for that in test-infra. |
Currently Eric and Paul are members of Knative Admin, but don't hold an official role on the Project.
We are trying to align levels of access with project roles, and this is an instance where we've likely uncovered folks implicitly performing important functions: https://docs.google.com/document/d/1pCLzTfNP5-_Z9taNK0W5hntyKbP4E5DI4-6n16GpyeU/edit#
The text was updated successfully, but these errors were encountered: