#9927 Add commit privilege for https://pagure.io/fedora-comps/ to proven packagers
Opened 11 months ago by zbyszek. Modified 4 months ago

A few years ago comps were writable by any packager. With a move to pagure, the list of people who can make changes was severely curtailed, currently just a few people + releng. This was done without any discussion, implicitly as part of the move.

As with other similar setups, this creates a choke point whenever @releng is swamped with other things. Pull requests to comps are not being reviewed and merged in time (https://pagure.io/fedora-comps/pull-request/509, https://pagure.io/fedora-comps/pull-request/543, https://pagure.io/fedora-comps/pull-request/567, others).

I'm sure plenty of people would like to help with this, but are blocked by the centralization of write privileges. I think allowing all proven packagers to write to the repo would help resolve the issue. There is really no need to restrict this to @releng, since @releng doesn't really know too much about KDE spin or i3 desktop. Groups should be managed by people who work on those groups. Restricting this to proven packagers is a reasonable compromise IMHO.

  • When do you need this? (YYYY/MM/DD)

Weeks rather than months.

  • When is this no longer needed or useful? (YYYY/MM/DD)

Never.

  • If we cannot complete your request, what is the impact?

See description.


The provenpackager and packager groups do not exist on pagure.io. That said, virtually all major SIGs have corresponding pagure.io groups, so we could request to grant those write permissions.

At least offhand, all working groups and the KDE SIG have membership managed through Pagure groups on pagure.io.

+1 to the idea. If this is technically not possible, we could make a "comps sig" group and populate it with any provenpackager and/or package group maintainer who expresses interest.

I'm gonna dispute your premise...

As with other similar setups, this creates a choke point whenever @releng is swamped with other things. Pull requests to comps are not being reviewed and merged in time (https://pagure.io/fedora-comps/pull-request/509, https://pagure.io/fedora-comps/pull-request/543, https://pagure.io/fedora-comps/pull-request/567, others).

509 -> as you asked in comments, the status is unknown. Really to merge this we need input from the kde sig... and the pull request isn't something they are pushing, just something someone else decided to do.

543 -> was just missed, a ping anytime would have gotten someone to merge it. It's merged now.

567 -> filed the day after xmas... I didn't expect anyone to see it until after holidays, but it's simple enough, so I just merged it.

Everything else is still under discussion/deveopment...

I don't think the problem is lack of people to merge things, I think it's more a lack of subject matter experts/sigs/working groups providing timely review.

Or perhaps it's that we don't close PR's very well. IMHO 509 and 488 could be dropped with "please figure it out in kde land and resubmit when everyone is happy with it"

Anyhow, I have no objection to adding more people to merge things, but I don't think thats the problem.

was just missed, a ping anytime would have gotten someone to merge it
filed the day after xmas... I

Yeah, but I don't want to ping you (or other releng members) during the holidays. That's essentially the whole point: I (and I assume others) want to help, without interrupting your well-deserved time off, but we're blocked by overly restrictive policy.

Anyhow, I have no objection to adding more people to merge things

Cool.

We need to look at how to sync fas groups into pagure.io, pinging @pingou for more insight.

Metadata Update from @mohanboddu:
- Issue tagged with: high-trouble, medium-gain, ops

11 months ago

From the releng meeting today

[11:14:12] <mboddu> #info pingou is going to look at how we can sync the fas groups into pagure.io

Metadata Update from @mohanboddu:
- Issue untagged with: high-trouble, ops
- Issue tagged with: dev, medium-trouble

11 months ago

This would be useful to package-maintainer-docs as well,
that repo it a good candidate to be managed by the fas packager group.

Login to comment on this ticket.

Metadata
Boards 1
Dev Status: Backlog