Learn more about these different git repos.
Other Git URLs
It looks like there's some kind of sync issue with requesting dist-git branches.
These requests were denied:
However, @music is co-maintainer of the package through the rust-sig group.
A request made by me (the main admin) succeeded, so it's not this specific package that's weird:
https://pagure.io/releng/fedora-scm-requests/issue/66210
Myself, I was able to request an epel10 branch in the exact same situation a few days ago:
https://pagure.io/releng/fedora-scm-requests/issue/65678
I am not a direct co-maintainer of rust-autocfg, but only co-maintainer through the rust-sig group.
I'm not sure if this is specific to @music, but he's the most recent member of the rust-sig group, so I'm suspecting it's some kind of sync issue - as far as I know, requests for epel9 and epel10 branches are not handled differently, are they?
Soon would be good. I need help with branching things for epel10, and granting individual co-maintainer access to potentially hundreds of packages for no good reason does not scale.
N/A
More work for me when dealing with branching for epel10, and an ACL issue that remains lurking somewhere.
Metadata Update from @phsmoura: - Issue tagged with: investigation, medium-gain, medium-trouble, ops
This keeps coming up in different situations, e.g. https://bugzilla.redhat.com/show_bug.cgi?id=2312849.
So, is this only happening to music?
I tried to trace this in logs, but... sadly toddlers scm processor doesn't actually print anything around what groups it found, etc.
@zlopez can we make the scm_commits toddler more verbose when it says someone isn't in a correct group?
worker04.ocp.iad2.fedoraproject.org/2024/09/02/messages.log.xz:Sep 2 13:27:18 worker04.ocp.ia d2.fedoraproject.org : namespace_name=toddlers, container_name=toddlers, pod_name=toddlers-98- xjk87, message=2024-09-02 13:27:18,898 - [INFO toddlers.plugins.scm_request_processor] - Check ing if music is one of the maintainers of the package
We definitely can add something more to either log or the comment that is being sent to the ticket.
I opened a PR for it. Feel free to check if that could be helpful.
I have a theory, that will either be confirmed or debunked by the additional debug logging.
https://pagure.io/fedora-infra/toddlers/blob/main/f/toddlers/utils/fedora_account.py#_148
Running this manually, I get a list of dictionaries. @music is in that list, but is not the first result.
[ins] In [1]: from fasjson_client import Client [ins] In [2]: fasjson = Client(url='https://fasjson.fedoraproject.org/') [ins] In [3]: fasjson.search(username='music').result[0]['username'] Out[3]: 'musicalbox'
So while the message in the ticket is music is not a maintainer of the python-immutables package, it's not actually checking his permissions, it's checking the permissions of musicalbox.
music is not a maintainer of the python-immutables package
musicalbox
https://pagure.io/fedora-infra/toddlers/pull-request/262
@carlwgeorge Good work finding that, I will still add my PR, so we have more info if something similar will happen in the future.
The fix is now live. Closing this as fixed. Feel free to reopen if the issue happens again.
Metadata Update from @zlopez: - Issue close_status updated to: Fixed - Issue status updated to: Closed (was: Open)
Thank you! Unfortunately, I still see the same problem: https://pagure.io/releng/fedora-scm-requests/issue/67751
Could someone who has permission to do so (maybe @zlopez who closed it or @decathorpe who submitted it) please reopen the issue?
Metadata Update from @decathorpe: - Issue status updated to: Open (was: Closed)
@music That's strange this is not the new response the bot should make.
The issue was on my side, I didn't realized that toddlers don't have automatic deployment on new container build. So the changes weren't actually deployed.
I deployed them and processed the ticket without issue after that.
Metadata Update from @zlopez: - Issue close_status updated to: Fixed with Explanation - Issue status updated to: Closed (was: Open)
Log in to comment on this ticket.