Project

Profile

Help

Issue #2056

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

Added by jortel@redhat.com over 4 years ago. Updated over 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
High
Category:
-
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 - NOTABUG

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 9e580cb5 View on GitHub
Added by jortel@redhat.com over 4 years ago

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

Revision 9e580cb5 View on GitHub
Added by jortel@redhat.com over 4 years ago

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

History

#1 Updated by jortel@redhat.com over 4 years ago

  • Status changed from ASSIGNED to POST
  • Triaged changed from No to Yes

#2 Updated by jortel@redhat.com over 4 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#4 Updated by semyers over 4 years ago

  • Platform Release set to 2.8.6

#5 Updated by semyers over 4 years ago

  • Status changed from MODIFIED to 5

#6 Updated by semyers over 4 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#7 Updated by semyers over 4 years ago

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

#8 Updated by bmbouter over 2 years ago

  • Sprint set to Sprint 4

#9 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (22)

#10 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF