Project

Profile

Help

Issue #2056

closed

Distribution storage path migration re-links .treeinfo files incorrectly.

Added by jortel@redhat.com over 7 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
2.8.5
Platform Release:
2.8.6
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 4
Quarter:

Description

After testing with rpms including the fix from issue 1972, I still see several files with broken sym links (all .treeinfo files). Is this expected?

Note: prior to the Satellite/Pulp upgrade, there were no files with broken sym links.

E.g.

[root@sat61-rhel7 pulp]# find -L /var/lib/pulp/published/yum/http/repos/ -type l -! -exec test -e {} \; -print
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/custom/capsule/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/custom/satellite_tools/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/custom/satellite_tools/el6_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/tools_el6/custom/satellite_tools/el6_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/tools_el7/custom/satellite_tools/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/composite_app_el6/custom/satellite_tools/el6_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/Library/composite_app_el7/custom/satellite_tools/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/content_views/tools_el6/1.0/custom/satellite_tools/el6_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/content_views/tools_el7/1.0/custom/satellite_tools/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/content_views/composite_app_el6/1.0/custom/satellite_tools/el6_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/content_views/composite_app_el7/1.0/custom/satellite_tools/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/dev/composite_app_el7/custom/satellite_tools/el7_x86_64/.treeinfo
/var/lib/pulp/published/yum/http/repos/Default_Organization/dev/composite_app_el6/custom/satellite_tools/el6_x86_64/.treeinfo

Looks like the migration did re-link the "<path>/.treeinfo" file but to a non-existent "<new-path>/treeinfo". Looks like a 1 line code change upstream to do a stricter comparison.


Related issues

Related to Packaging - Task #2083: Issues common to 2.9.1 and 2.8 streamCLOSED - NOTABUGsemyers

Actions

Added by jortel@redhat.com over 7 years ago

Revision 9e580cb5 | View on GitHub

Fix distribution storage path migration incorrect re-linking of .treeinfo files. closes #2056

Added by jortel@redhat.com over 7 years ago

Revision 9e580cb5 | View on GitHub

Fix distribution storage path migration incorrect re-linking of .treeinfo files. closes #2056

Actions #1

Updated by jortel@redhat.com over 7 years ago

  • Status changed from ASSIGNED to POST
  • Triaged changed from No to Yes
Actions #2

Updated by jortel@redhat.com over 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #4

Updated by semyers over 7 years ago

  • Platform Release set to 2.8.6
Actions #5

Updated by semyers over 7 years ago

  • Status changed from MODIFIED to 5
Actions #6

Updated by semyers over 7 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #7

Updated by semyers over 7 years ago

  • Related to Task #2083: Issues common to 2.9.1 and 2.8 stream added
Actions #8

Updated by bmbouter about 6 years ago

  • Sprint set to Sprint 4
Actions #9

Updated by bmbouter about 6 years ago

  • Sprint/Milestone deleted (22)
Actions #10

Updated by bmbouter almost 5 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF