Remove the is_modular field from the Package model #3525
Conversation
pulp_rpm/app/models/repository.py
Outdated
| .only("pk") | ||
| ).exclude( | ||
| pk__in=pulp_rpm.app.models.modulemd.Modulemd_packages.values_list("package", flat=True), | ||
| ).only("pk") |
There was a problem hiding this comment.
This on the other hand might be tricky to do in a performant way. The through table is implicit so this doesn't actually work.
There was a problem hiding this comment.
is it worth filtering modulemd within the current repo version?
|
TODO: online migration implications? Do we need to drop the reliance separately from the PR that removes the field from the DB? |
dralley
force-pushed
the
remove-is-modular
branch
from
aae984e
to
7c801c9
Compare
| modular_packages.update(module.packages.all().values_list("pk", flat=True)) | ||
|
|
||
| nonmodular_packages = set(packages.values_list("pk", flat=True)) - modular_packages | ||
| nonmodular_packages = Package.objects.filter(pk__in=nonmodular_packages) |
There was a problem hiding this comment.
@pedro-psb @ipanova Do either of you see a way to make this more efficient without table changes?
Making the ModulemdPackage table explicit rather than implicit would probably allow us to reduce the number of queries. I'm unsure if it's worth it though.
There was a problem hiding this comment.
I did some research to see if I could come up with something but no luck.
There is prefetch_related, but I couldn't figure if this could fit in here.
Also, ideas of caching the context of the last repover crossed my mind, but I'm not aware of a suitable method for doing so.
dralley
force-pushed
the
remove-is-modular
branch
2 times, most recently
from
f109047
to
0557824
Compare
github-actions
bot
added
the
multi-commit
|
The failures aren't related to the PR, it's all about the certificate. |
dralley
force-pushed
the
remove-is-modular
branch
from
0557824
to
2657323
Compare
|
Removing the field from the model needs to be done in a later PR |
dralley
force-pushed
the
remove-is-modular
branch
from
2657323
to
0089215
Compare
github-actions
bot
removed
the
multi-commit
|
Back to draft. Unfortunately it seems like Katello uses this field :( |
closes #3524