Actions
Issue #3100
closedRemoval of existing iso units doesn't work if there are multiple iso files
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.14.3
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 28
Quarter:
Description
This is similar to #3047 and #2773. We added code to remove duplicate ISOs however, this code doesn't work after one iteration. Looks like a generator type was being used instead of a list:
Steps to reproduce:
export REPO1='iso-1'
export REPO2='iso-2'
mkdir /tmp/iso
cd /tmp/iso
echo "test1" > 1.iso
echo "test2" > 2.iso
echo 1.iso,`sha256sum 1.iso | awk '{ print $1 }'`,`stat -L -c '%s' 1.iso` > PULP_MANIFEST
echo 2.iso,`sha256sum 2.iso | awk '{ print $1 }'`,`stat -L -c '%s' 2.iso` >> PULP_MANIFEST
pulp-admin iso repo create --repo-id $REPO1 --feed file:///tmp/iso
pulp-admin iso repo sync run --repo-id $REPO1
pulp-admin iso repo create --repo-id $REPO2 --feed file:///tmp/iso
pulp-admin iso repo sync run --repo-id $REPO2
echo "blah1" > 1.iso
echo "blah2" > 2.iso
echo 1.iso,`sha256sum 1.iso | awk '{ print $1 }'`,`stat -L -c '%s' 1.iso` > PULP_MANIFEST
echo 2.iso,`sha256sum 2.iso | awk '{ print $1 }'`,`stat -L -c '%s' 2.iso` >> PULP_MANIFEST
pulp-admin iso repo sync run --repo-id $REPO1
pulp-admin iso repo sync run --repo-id $REPO2
pulp-admin iso repo list
You'll see that the repos have 3 units instead of 2. One of the existing dupe units is removed but the other is not.
Related issues
Added by daviddavis about 7 years ago
Updated by daviddavis about 7 years ago
- Status changed from NEW to POST
- Assignee set to daviddavis
Updated by daviddavis about 7 years ago
- Related to Issue #3047: ISO repo doesn't correctly handle updates to files for content already in Pulp added
Updated by dalley about 7 years ago
- Sprint/Milestone set to 46
- Triaged changed from No to Yes
Updated by daviddavis about 7 years ago
- Status changed from POST to MODIFIED
Applied in changeset 5d24a82494654ac77ae2ce9d378c165dc78ca95d.
Updated by pthomas@redhat.com about 7 years ago
See Issue #3047 for verification steps.
Updated by daviddavis over 5 years ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
Actions
Changing the type of repo_units to list from generator
The repo_units value was emptying after iterating through it because it was a generator and not a list.
fixes #3100 https://pulp.plan.io/issues/3100