#6976 Adopt and/or rename python-Fiona
Closed: Fixed 6 years ago Opened 6 years ago by qulogic.

The PkgDB migration page still points here, so...

I recently had the python-fiona package approved, but the New Repo request was denied due to an existing PDC branch. Contrarily, the branch request was denied because the repo didn't exist.

Instead, there exists a retired python-Fiona repo. So it seems that the PDC check is case-insensitive but the repo branch check is case-sensitive. These are the same package (except Python packages seem to be shifting to lower-case names everywhere), so I would like to either adopt the existing one with a rename or get some sort of manual intervention to add the lower case name. (Please also enable the f26 and f27 branches if that can be done at the same time.)


@ralph or @pingou can you help here? It seems to me that PDC should be case sensitive to allow the python-fiona package with python-Fiona being retired. Another option would be to allow renaming the global coponent (package) and repo but then ideally pagure would also be case insensitive regarding the repo.

I agree, it needs to be case sensitive.

@ralph @pingou @mprahl Can PDC be adjusted for this?

So the filter is currently case insensitive:
https://github.com/product-definition-center/product-definition-center/blob/master/pdc/apps/componentbranch/filters.py#L81

I can change this to case sensitive matches to solve this issue but it would require another release by @ralph.

I submitted a PR to make the filtering case-sensitive. This should resolve the issue once it gets merged/released:
https://github.com/product-definition-center/product-definition-center/pull/471

For the time being, we can either wait for a release, or manually create the entries in PDC since the restrictions were in the fedrepo-req-admin checking if the branch exists. Since these queries used the filters that are case-insensitive, fedrepo-req-admin thought the branch already exists.

Thanks @mprahl.

I'll try to proceed with creating python-fiona manually for now.

@qulogic - python-fiona should be created now.

https://apps.fedoraproject.org/datagrepper/raw?delta=127800&package=python-fiona&rows_per_page=1

Can you let us know if all is as it should be with the package?

@ralph Thanks; everything appears to have been built properly and the updates have been submitted too.

Okay, now the queries to the PDC API are case sensitive in production. This problem shouldn't arise again.

This ticket can be closed.

Metadata Update from @qulogic:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 years ago

Login to comment on this ticket.

Metadata