Project

Profile

Help

Issue #5359

closed

treeinfo files in deferred-download repos don't see feed updates

Added by ggainey over 4 years ago. Updated over 4 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
2.20.0
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 61
Quarter:

Description

Set up and sync a deferred-download repo. Update its feed and re-sync. Access a kickstart/treeinfo-related file. Look in the logs, and note that said file is downloaded from the original feed-url, not the newly-updated one.

Update: forcing a full-sync fixes the issue.

From comment in BZ#1744225 , investigation of which led to this issue:

"Start with a repo pulling on-demand from a Centos7 mirror, switch feeds, and then ask for an RPM - RPM is pulled from new-location as expected:

#### CREATE, SYNC, REQUEST, CHECK
# pulp-admin rpm repo create --repo-id centos-7-x86_64 --feed http://mirror.fileplanet.com/centos/7/os/x86_64/ --download-policy on_demand
# pulp-admin rpm repo sync run --repo-id centos-7-x86_64
# wget --no-check-certificate https://<pulp-machine>/pulp/repos/centos/7/os/x86_64/Packages/e/ElectricFence-2.2.2-39.el7.x86_64.rpm
# pjournal | grep Electric
Aug 26 19:00:47 pulp2.dev pulp_streamer[27677]: pulp.streamer.server:INFO: Trying URL: http://mirror.fileplanet.com/centos/7/os/x86_64/Packages/ElectricFence-2.2.2-39.el7.x86_64.rpm
#
### UPDATE, SYNC, REQUEST, CHECK
# pulp-admin rpm repo update --repo-id centos-7-x86_64 --feed http://mirror.linux.duke.edu/pub/centos/7/os/x86_64/
# pulp-admin rpm repo sync run --repo-id centos-7-x86_64
# wget --no-check-certificate https://<pulp-machine>/pulp/repos/centos/7/os/x86_64/e/elfutils-0.172-2.el7.x86_64.rpm
# pjournal | grep elfutils
Aug 26 20:14:53 pulp2.dev pulp_streamer[27677]: pulp.streamer.server:INFO: Trying URL: http://mirror.linux.duke.edu/pub/centos/7/os/x86_64/Packages/elfutils-0.172-2.el7.x86_64.rpm

Ask for a treeinfo-related file, note that it continues to pull from old-repo:
# wget --no-check-certificate https://<pulp-machine>/pulp/repos/centos/7/os/x86_64/images/pxeboot/initrd.img
# pjournal | grep  initrd
Aug 26 20:15:41 pulp2.dev pulp_streamer[27677]: pulp.streamer.server:INFO: Trying URL: http://mirror.fileplanet.com/centos/7/os/x86_64/images/pxeboot/initrd.img

The PR submitted for issue #4265 does NOT appear to fix this behavior - feels like 'update' doesn't touch treeinfo-related files, just repomd-related ones.

Update from a final test - "--force-full" resync appears to work around the issue:

### Full sync and pull another treeinfo file
# pulp-admin rpm repo sync run --repo-id centos-7-x86_64 --force-full
# wget --no-check-certificate https://192.168.122.124/pulp/repos/centos/7/os/x86_64/images/pxeboot/vmlinuz
# pjournal | grep vmlinuz
Aug 27 12:48:17 pulp2.dev pulp_streamer[22861]: pulp.streamer.server:INFO: Trying URL: http://mirror.linux.duke.edu/pub/centos/7/os/x86_64/images/pxeboot/vmlinuz
Actions #1

Updated by ggainey over 4 years ago

RE "The PR submitted for issue #4265 " - should be #4524 , sorry

Actions #2

Updated by amacdona@redhat.com over 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 58
Actions #3

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 58 to Sprint 59
Actions #4

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 59 to Sprint 60
Actions #5

Updated by ggainey over 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ggainey
Actions #6

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 60 to Sprint 61
Actions #7

Updated by ggainey over 4 years ago

  • Status changed from ASSIGNED to CLOSED - WONTFIX

--force-full on the sync fixes this issue since commit b30b5e2914c5d40f760fac2e77e2d40407902aaf was applied to fix #4267, as part of pulp-2.19

While the behavior between content-units and treeinfo-files should be consistent, with an available workaround and the low priority of the problem, I'm closing this as WONTFIX.

Also available in: Atom PDF