packit / packit-service

Packit provided as a service
https://packit.dev
MIT License
34 stars 46 forks source link

Research Packit reverse-dependency checks #2361

Open lachmanfrantisek opened 4 months ago

lachmanfrantisek commented 4 months ago

As part of the cross-team discussion about how to help with #2343 , we've come up with an idea how to approach it:

So, let's try to think about what would it take to do this:

frenzymadness commented 3 months ago

I'm thinking about it and have a question: isn't „Fedora on Gitlab“ too big change to start with? I mean, there are many questions around this and I'm afraid they might block the reverse dependency builds. For example: if a package is onboarded to Gitlab, is pagure disabled for it? If not, what will happen to PRs open via Pagure? What will be UX for the maintainers?

lachmanfrantisek commented 3 months ago

@frenzymadness Good point, Lumír.

A lot is still unclear and that's why I've created this spike card...;)

I don't know many details about the OSCI tooling, but I would say it takes care only of the sync. At least for the start, it would probably be on you, as a maintainer, to disable pull requests on Pagure site if wanted: image

For us, the difference between GitLab/Pagure from the implementation point of view is not the hardest part of the implementation. If the FedoraOnGitLab won't work, we can do the Pagure support. Maybe the GitLab is just more future-proof. The crucial part is to handle these PRs as dist-git content and the reverse-dep logic itself.

lachmanfrantisek commented 3 months ago

Maybe just one concern -- how the user rights can be handled on GitLab. If we force people to create a GitLab account and also if the maintainer permissions can be synced.