Project

Profile

Help

Issue #4524

Broken symlinks for subrepos are created during on_demand sync of kickstart trees

Added by ttereshc 12 months ago. Updated 4 days ago.

Status:
NEW
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
Yes
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 67

Description

1. Sync kickstart tree with on_demand policy.
2. Switch to immediate policy and re-sync.
3. Note that subrepos (HighAvailability, Addons) have broken symlinklinks because Pulp doesn't support subrepos but it seems like symlinks are still created for those during on_demand sync.

That causes Pulp to Pulp sync to fail.
Pulp should not create symlinks for the unsupported parts of kickstart trees.


Related issues

Related to RPM Support - Test #4525: Test broken symlinks for subrepos are created during on_demand sync of kickstart trees NEW Actions

History

#1 Updated by kersom 12 months ago

  • Related to Test #4525: Test broken symlinks for subrepos are created during on_demand sync of kickstart trees added

#2 Updated by ttereshc 12 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 50

This issue needs at least some investigation to figure out if it's a regression or no

#3 Updated by ttereshc 12 months ago

Maybe worth checking if this fix helps to workaround/mitigate the problem https://pulp.plan.io/issues/4267, should be in master already.

#4 Updated by daviddavis 12 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis

#5 Updated by daviddavis 11 months ago

  • Status changed from ASSIGNED to NEW

#6 Updated by daviddavis 11 months ago

  • Status changed from NEW to ASSIGNED

#8 Updated by rchan 11 months ago

  • Sprint changed from Sprint 50 to Sprint 51

#9 Updated by daviddavis 11 months ago

  • Status changed from ASSIGNED to CLOSED - DUPLICATE

Talked with @dkliban who indicated that this is the same issue as #4603.

#10 Updated by daviddavis 11 months ago

  • Duplicates Issue #4603: pulp_streamer streams decodes responses, but sends the 'gzip' Content-Encoding header added

#11 Updated by bmbouter 11 months ago

  • Tags Pulp 2 added

#12 Updated by dkliban@redhat.com 7 months ago

  • Status changed from CLOSED - DUPLICATE to NEW
  • Assignee deleted (daviddavis)
  • Sprint changed from Sprint 51 to Sprint 56

I am reopening this issue because it is not a duplicate of https://pulp.plan.io/issues/4603 and is not fixed.

#13 Updated by dkliban@redhat.com 7 months ago

  • Duplicates deleted (Issue #4603: pulp_streamer streams decodes responses, but sends the 'gzip' Content-Encoding header)

#14 Updated by rchan 7 months ago

  • Sprint changed from Sprint 56 to Sprint 57

#15 Updated by ggainey 6 months ago

  • Status changed from NEW to POST
  • Assignee set to ggainey
  • Groomed changed from No to Yes

PR submitted:

https://github.com/pulp/pulp/pull/3951

A script that reproduces, and shows the fix working, is here :

https://github.com/ggainey/pulp_startup/blob/master/4524/reproducer.bsh

#16 Updated by rchan 6 months ago

  • Sprint changed from Sprint 57 to Sprint 58

#17 Updated by rchan 6 months ago

  • Sprint changed from Sprint 58 to Sprint 59

#18 Updated by rchan 5 months ago

  • Sprint changed from Sprint 59 to Sprint 60

#19 Updated by rchan 4 months ago

  • Sprint changed from Sprint 60 to Sprint 61

#20 Updated by rchan 4 months ago

  • Sprint changed from Sprint 61 to Sprint 62

#21 Updated by rchan 3 months ago

  • Sprint changed from Sprint 62 to Sprint 63

#22 Updated by rchan about 1 month ago

  • Sprint changed from Sprint 63 to Sprint 64

#23 Updated by rchan about 1 month ago

  • Sprint changed from Sprint 64 to Sprint 65

#24 Updated by rchan 22 days ago

  • Sprint changed from Sprint 65 to Sprint 66

#25 Updated by rchan 8 days ago

  • Sprint changed from Sprint 66 to Sprint 67

#26 Updated by ggainey 4 days ago

  • Status changed from POST to NEW

This appears to have fallen through a few cracks. The submitted PR 'fixes' the issue by fixing the links - the real problem is that subrepos aren't supported in pulp2, and the links shouldn't be there at all. Setting back to NEW, we need to address the actual problem.

Please register to edit this issue

Also available in: Atom PDF