Project

Profile

Help

Issue #4746

closed

Do not query saved content in QueryExistingContents stage

Added by gmbnomis almost 5 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix
Sprint:
Quarter:

Description

Motivation

Plugins may have multiple stages that query content. In
pulp_cookbook there is a custom stage that queries content and artifact from
the previous repo version. In immediate mode, the stages are:

...
QueryExistingRepoContentAndArtifacts(new_version=new_version),
ArtifactDownloader(),
ArtifactSaver(),
UpdateContentWithDownloadResult(),  # add digest to content
QueryExistingContents(),  # share content with known digest
...

After downloading missing artifacts (and thus, obtaining a digest), the
QueryExistingContents stage is used to identify content that already exists
outside of the repo. In this scenario, QueryExistingContents may encounter
already saved content which it should not query again for performance reasons.

Solution

Do as the current documentation states ('This stage inspects any "unsaved"
Content unit objects and searches for existing saved Content units inside
Pulp with the same unit key') and do not query for content that is already
saved.

Actions #2

Updated by gmbnomis almost 5 years ago

  • Subject changed from Do not query saved content QueryExistingContents stage to Do not query saved content in QueryExistingContents stage
Actions #3

Updated by gmbnomis almost 5 years ago

  • Status changed from NEW to MODIFIED

Applied in changeset commit:pulpcore-plugin|938d86953bd3d22c2c2e207cb9a86f98a6d7bbe2.

Actions #4

Updated by bmbouter over 4 years ago

  • Sprint/Milestone set to 3.0.0
Actions #5

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF