Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete sqlite metadata support #2457

Closed
dralley opened this issue Mar 30, 2022 · 0 comments · Fixed by #3328
Closed

Delete sqlite metadata support #2457

dralley opened this issue Mar 30, 2022 · 0 comments · Fixed by #3328
Assignees
Labels

Comments

@dralley
Copy link
Contributor

dralley commented Mar 30, 2022

Background:

It seems the only thing that uses sqlite metadata anymore, is generating repoview files. Except repoview isn't really used anymore and hasn't built since Fedora 28 - it can't be built on EL8 at all without packaging changes. And also, if pulp isn't generating the repoview metadata, then it won't be part of the hosted repo.

The only repo I know of that uses it is RPMFusion, and RPMFusion apparently uses repoview on Fedora 26 just to build these pages. It uses Python 2 and some other ancient dependencies including yum (as a library), which means it can't easily be built on Fedora 28+ / EL8+. And since pulp isn't going to support this anyway, you'd need to be using Pulp just to create the repo, then download the repo and serve it completely separately. You don't really need Pulp to do this, at that point you could just generate the sqlite metadata yourself too.

Rationale: Even though we currently "support" this, it's pretty much untested, untestable and useless, and we would like to reduce unneeded complexity anywhere we can.

Proposal: We announce that this feature will go away in a future release of pulp_rpm and mark the API parameter deprecated. In some future release we would then remove the backing code, and make a decision about whether the API / CLI parameters should just do nothing or be removed entirely.

dralley added a commit to dralley/pulp_rpm that referenced this issue Apr 6, 2022
dralley added a commit to dralley/pulp_rpm that referenced this issue Apr 6, 2022
dralley added a commit to dralley/pulp_rpm that referenced this issue Apr 6, 2022
dralley added a commit to dralley/pulp_rpm that referenced this issue Apr 6, 2022
dralley added a commit that referenced this issue Apr 7, 2022
@dralley dralley reopened this Sep 12, 2022
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 22, 2023
@pedro-psb pedro-psb self-assigned this Nov 22, 2023
pedro-psb added a commit to pedro-psb/pulp-cli that referenced this issue Nov 23, 2023
Removed from "pulp rpm repository update" command.
The reason is that it was phased out from pulp_rpm plugin.
pulp/pulp_rpm#2457

fixes: pulp#831
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 23, 2023
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 23, 2023
pedro-psb added a commit to pedro-psb/pulp-cli that referenced this issue Nov 23, 2023
Removed sqlite-metadata from "pulp rpm repository update" command.
It was phased out from pulp_rpm plugin (pulp/pulp_rpm#2457)

closes pulp#831
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 24, 2023
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 27, 2023
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 28, 2023
pedro-psb added a commit to pedro-psb/pulp_rpm that referenced this issue Nov 28, 2023
pedro-psb added a commit that referenced this issue Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants