fedpkg

#359 request-tests-repo creates invalid request
Closed: Fixed by onosek. Opened by hhorak.

Closed: Fixed
Reopen Issue

I've created a request using fedpkg request-tests-repo ... command and it resulted in https://pagure.io/releng/fedora-scm-requests/issue/19437, and this request was closed as invalid, because it does not included branch specification.

I believe we don't need a separate command for tests (request-tests-repo) -- there is a common command request-repo for creating requests for rpms, container, modules, etc., so why not just include the tests namespace into the list?

I created PR #368 which adds fixed branch 'master' into ticket body. Bud it doesn't have to work as we can see here:
https://pagure.io/releng/fedora-scm-requests/issue/19549
or here:
https://pagure.io/releng/fedora-scm-requests/issue/19845

It looks releng team has some automatic that is not prepared for 'tests' namespace.

On the other hand - this works:
ask somebody to do this manually
https://pagure.io/releng/fedora-scm-requests/issue/17124

As per unifying methods request-tests-repo and general request-repo, I think that the current state is a reasonable approach. There are specific sets of input arguments for namespaces and is easier to handle them separately in two methods.

Commit 37f8972c fixes this issue

Metadata Update from @onosek:
- Issue set to the milestone: 1.38
Metadata
None
None
None
None
None
Related Pull Requests
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.