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

triage & maintainers need permissions to transfer issues across repos #13

Closed
handrews opened this issue Feb 8, 2024 · 6 comments
Closed

Comments

@handrews
Copy link
Member

handrews commented Feb 8, 2024

@webron I'm guessing we need your admin powers on this as well as I'm not sure anyone else has access to all the repos needed.

In today's call we agreed to remove some issue labels from the spec repo, but this requires @OAI/triage and @OAI/openapi-maintainers being able to transfer some of the labeled issues to other repos, specifically:

  • community (this repo)
  • learn.openapis.org
  • Outreach
  • Tooling
  • oascomply
  • Overlay-Specification
  • sig-*

Could this ability be added for both of those groups (at the moment, I don't think adding to triage automatically empowers maintainers).

@webron
Copy link
Member

webron commented Feb 8, 2024

@handrews any concerns with limiting this to @OAI/openapi-maintainers only for now? The @OAI/triage team is set with github's triage permissions that do not allow manipulating labels or transferring issues, but also don't allow merging/modifying PRs. Feels like elevating the team's permissions would go against what triage means by Github's standards.

I believe the active members of the triage team are part of the maintainers team so it shouldn't hinder anyone's work. We can always look into adding more people to the maintainers team.

@handrews
Copy link
Member Author

handrews commented Feb 8, 2024

@webron maintainers only for now would be great and would unblock things - thanks for the quick reply!

I'm pretty vague on how they're configured and did not intend to propose adding new privileges. We do plan to revisit the roles once the new TSC is in place:

@webron
Copy link
Member

webron commented Feb 8, 2024

@handrews ok, I've added @OAI/openapi-maintainers to all the requested repos, with the Write permission, and not the Maintain permission, since that's how it was set for the OAS repo. Once we define the roles better, we can make the decision of changing the permissions set, it's easy enough to do.

We should also have a task to clean out some of the users/permissions that we currently have. I've noticed some inactive users with access, and we should probably remove those to not have any potential destructive actions (intentional or not).

Since you're a member of @OAI/openapi-maintainers, please run a test to see if you can access what's needed. Happy to look into any issues.

@handrews
Copy link
Member Author

handrews commented Feb 8, 2024

Thanks, @webron - the only one I don't see is Outreach. If you left that one off intentionally, there are a few issues that need moving, and @swaldron58 says he doesn't have permissions either. As long as someone can move them it will be fine with me.

But I can see the other repos as transfer options here, and this repo as a transfer option elsewhere. 🎉

@webron
Copy link
Member

webron commented Feb 8, 2024

No, I see maintainers in the Outreach repo, so it should be fine. Regardless, I'll transfer those issues quickly now, no problem. Please let me know if you run into any more issues.

@handrews
Copy link
Member Author

handrews commented Feb 8, 2024

Thanks again, Ron! I do seem to have maintainer privileges in the Outreach repo, but it's not showing up as a possible transfer destination from other repos. I seem to recall @earth2marsh and I had this problem with the sig-moonwalk repo at first? There might be some other setting that's also involved.

Anyway, I doubt we'll get many Outreach issues in the spec repo, so this is close enough to done for me :-)

@handrews handrews closed this as completed Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants