Skip to content
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: Repository maintenance on opentelemetry-rust(-contrib) #1815

Closed
hdost opened this issue Nov 21, 2023 · 5 comments
Closed

REQUEST: Repository maintenance on opentelemetry-rust(-contrib) #1815

hdost opened this issue Nov 21, 2023 · 5 comments
Assignees
Labels
area/repo-maintenance Maintenance of repos in the open-telemetry org

Comments

@hdost
Copy link
Contributor

hdost commented Nov 21, 2023

Affected Repository

https://github.com/open-telemetry/opentelemetry-rust
https://github.com/open-telemetry/opentelemetry-rust-contrib

Requested changes

Give 'Admin' permissions @open-telemetry/rust-maintainers

We will abide by:

If requested, foo-maintainers will be granted Admin permissions, and in return they must document any changes they make to the repository settings in a file named .github/repository-settings.md in their repository (other than temporarily disabling "Do not allow bypassing the above settings", see branch protection rules below)
Source: https://github.com/open-telemetry/community/blob/main/docs/how-to-configure-new-repository.md#collaborators-and-teams

Purpose

We are looking to make a few modifications to our repo settings to match with our working style. Review process and open up the ability to

Expected Duration

permanently

Repository Maintainers

  • @open-telemetry/rust-maintainers
@hdost hdost added the area/repo-maintenance Maintenance of repos in the open-telemetry org label Nov 21, 2023
@bogdandrutu
Copy link
Member

We would prefer to not give permanent Admin power unless there is a strong argument. I can give temporal permission if you still need it.

@trask
Copy link
Member

trask commented Jan 2, 2024

We would prefer to not give permanent Admin power unless there is a strong argument

@open-telemetry/technical-committee we can definitely revisit the policy (especially as part of #1596), but I don't believe we have been requiring justification for admin rights since #1311

@jack-berg
Copy link
Member

@open-telemetry/technical-committee we can definitely revisit the policy (especially as part of #1596), but I don't believe we have been requiring justification for admin rights since #1311

Agreed - there's precedent for this request and a documented process.

Review process and open up the ability to

@hdost can you elaborate on the settings you intend to change? Perhaps open a draft PR of the .github/repository-settings.md the repo will ultimately need to maintain.

@jack-berg
Copy link
Member

@hdost just following up on this, and reiterating that repos with admin access need to document their settings in .github/repository-settings.md.

@hdost
Copy link
Contributor Author

hdost commented Apr 20, 2024

I think we can closed this as the thread was duplicated over here #2021

@hdost hdost closed this as not planned Won't fix, can't repro, duplicate, stale Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/repo-maintenance Maintenance of repos in the open-telemetry org
Projects
None yet
Development

No branches or pull requests

4 participants