Project

General

Profile

Bug #18655

Do not index full rpm metadata if already present

Added by John Mitsch about 2 years ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1405162

Description of problem:

Currently during any repo sync, we:

1. check to see if there was any change (or mismatch of unit counts)
2. IF so we request all UUIDs of rpms in the repo
3. We then fetch all rpm units by their UUID

After chatting with mhrivnak, the unit metadata for an rpm should never change (for the fields we care about), so we could optimize this by getting the list of UUIDs and then only fetching (and updating) units that aren't in the repo.

This should speed up repo syncing quite a bit.

We may want to add a 'force' sync option to override this behavior as well.

Associated revisions

Revision 937ca0da (diff)
Added by John Mitsch about 2 years ago

Fixes #18655 - Only index new rpms on repository sync

We index all rpms after a repository sync, which is unnecessary
for the already present rpms (in katello db). We should only
index the rpms that are not present in katello's db. This
new logic will say "What rpms are in pulp that are not in
our db for this repository?" and index those.

History

#1 Updated by John Mitsch about 2 years ago

  • Subject changed from Do not index full rpm metadata if already present to Do not index full rpm metadata if already present
  • Target version set to 169

#2 Updated by The Foreman Bot about 2 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/6634 added

#3 Updated by Eric Helms about 2 years ago

  • Legacy Backlogs Release (now unused) set to 211

#4 Updated by John Mitsch about 2 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF