User requested a repo in: https://pagure.io/releng/fedora-scm-requests/issue/32706
https://bugzilla.redhat.com/show_bug.cgi?id=1917510
and it was setup: https://src.fedoraproject.org/rpms/airspyhf-1.6.8-1
but it should not have included the version there. I'm not sure how we can check that... I guess review name against project name should catch it, but it wouldn't have in this case...
Anyhow, we should clean up the repo and try and see if we can make fedscmadm block this kind of thing.
Metadata Update from @phsmoura: - Issue tagged with: low-gain, low-trouble, ops
From the releng weekly meeting:
[17:07:05] <jednorozec> maybe we can ask for confirmation if there are numbers in the name? [17:08:01] <nirik> yeah, I guess it might be impossible... [17:09:04] <nirik> could we check against the .spec in the bug? [17:09:11] <jednorozec> hmm [17:09:26] <jednorozec> that complicates things but it will work [17:09:40] <nirik> because the project should == the name in the spec file. [17:10:04] <jednorozec> right
@zlopez do you think it will be possible to parse the specfile to avoid creation of such repositories?
We can try to validate the name of the repository in scm_request_processor by NVR regex and fail if the name contains the version as well.
The validation against specfile is also possible, but it will be more complicated as the specfile is only available as attachment on bugzilla ticket.
Feel free to open ticket on toddlers repository what is better for you.
Metadata Update from @jnsamyak: - Issue untagged with: low-gain, low-trouble, ops - Issue tagged with: dev, high-trouble, medium-gain, meeting
The reviewer is supposed to check the bugzilla title matches the package.
The tool can then check if the repo name matches the bugzilla title, which it did in this case.
Since this is more of a toddler issue, I have opened the issue there: https://pagure.io/fedora-infra/toddlers/issue/194, let's triage this there, I'll copy-paste the replies from here to there!
Metadata Update from @jnsamyak: - Issue close_status updated to: upstream - Issue status updated to: Closed (was: Open)
Log in to comment on this ticket.