copr icon indicating copy to clipboard operation
copr copied to clipboard

Implement support for Pulp manual createrepo

Open FrostyX opened this issue 1 year ago • 2 comments

Fix #3498

FrostyX avatar Dec 06 '24 16:12 FrostyX

I just wanted to push a proof of concept. This still needs to fix some quirks and refactor the code. Worth mentioning:

  • This approach of having two separate repositories and "copying packages from one to another" was recommended by Dennis
  • Currently we use fedora-rawhide-x86_64/devel/ directories on backend, but Pulp has a problem with that because the devel directory would encroach an existing repository. So that I used fedora-rawhide-x86_64_devel/ naming. I don't like it very much.

FrostyX avatar Dec 06 '24 16:12 FrostyX

Pull Request validation

Failed

🔴 Failed or pending checks:

  • RPM package copr-frontend[failure]
  • RPM package copr-backend[cancelled]
  • RPM package python-copr-common[cancelled]

🔴 Review - Missing review from a member (2 required)

Success

🟢 Review - Reviewed by undefined

github-actions[bot] avatar Feb 10 '25 13:02 github-actions[bot]

Have you considered to have two distributions, one for normal repo and one for the devel repo? https://pulpproject.org/pulpcore/docs/user/learn/lifecycle-promotion/

praiskup avatar Mar 11 '25 09:03 praiskup

What about fedora-rawhide-x86_64-staged/ instead?

Conan-Kudo avatar May 08 '25 15:05 Conan-Kudo

Sorry for the delay, I finally updated the PR. It now uses two distributions (aka the "Lifecyle Promotion Support") instead of two repositories, as you've all suggested to do.

PTAL

FrostyX avatar Jul 11 '25 11:07 FrostyX

I'd prefer having a review session :-) on (post) today's meeting

praiskup avatar Jul 21 '25 06:07 praiskup

Once again, this PR proves to be too complicated because I tested it once again while rebasing to main and found a bug.

FrostyX avatar Aug 19 '25 15:08 FrostyX

We should bump: https://github.com/fedora-copr/copr/blob/235810c90dfe47f59bde96d1cce2bb1cf32aa338/frontend/coprs_frontend/coprs/views/backend_ns/backend_general.py#L28

praiskup avatar Aug 20 '25 07:08 praiskup

:+1: for having the enum

praiskup avatar Aug 20 '25 07:08 praiskup