Project

Profile

Help

Issue #1086

pulp_distribution.xml sync should ignore repodata/*

Added by bcourt over 5 years ago. Updated over 1 year ago.

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

Description

Files that are synced via the pulp_distribution.xml file should ignore all files in the repodata/ directory of the repository.


Related issues

Related to RPM Support - Issue #1087: publishing of distribution units should ignore files in repodata directoryCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Related to RPM Support - Issue #1088: Incremental publish could use the wrong primary.xml during fast-forward publishCLOSED - WONTFIX<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 14a637bd View on GitHub
Added by bizhang almost 4 years ago

Ignore repodata/* when syncing from pulp_distribution.xml

closes #1086 https://pulp.plan.io/issues/1086

History

#1 Updated by bcourt over 5 years ago

  • Priority changed from Normal to High
  • Triaged changed from No to Yes

#3 Updated by mhrivnak about 5 years ago

  • Platform Release set to 2.8.0

#4 Updated by amacdona@redhat.com almost 5 years ago

  • Related to Issue #1087: publishing of distribution units should ignore files in repodata directory added

#5 Updated by amacdona@redhat.com almost 5 years ago

Before anyone takes this bug, it is worth investigating further. At first glance it seems like some of the files in `repodata/` should not be ignored.

#6 Updated by bmbouter almost 5 years ago

After talking some with bcourt, I think the pulp_distribution.xml repodata/ files should be ignored on sync. They are ignored on publish already[0]. I was told that Pulp generates all files in the repodata except the productid. productid is pulled in via the repomd.xml sync/parse/publish.

I was also reminded that the pulp_distribution.xml is a thing that pulp created to avoid modeling other file types as individual units.

[0]: https://github.com/pulp/pulp_rpm/commit/944a05f7c131893a3fd46f1a67a53e4a618d95d6#diff-4b7e211764045e0506c6ad263fddf8e0R808

#7 Updated by bmbouter almost 5 years ago

  • Related to Issue #1088: Incremental publish could use the wrong primary.xml during fast-forward publish added

#8 Updated by bmbouter almost 5 years ago

  • Tracker changed from Issue to Refactor
  • Groomed set to No
  • Sprint Candidate set to Yes

To be clear this is about keeping the Distribution unit files list clean/pristine. Because #1087 is fixed, keeping these repodata/* files tracked as part of a Distribution unit is now useless because those files will never be published. Since this is about cleanliness that is not user visible, I'm marking it as a Refactor.

#9 Updated by bmbouter almost 5 years ago

  • Priority changed from High to Normal
  • Platform Release deleted (2.8.0)

Also given ^, I'm setting to Normal priority and taking off the set Target Release.

#10 Updated by bmbouter almost 5 years ago

  • Tracker changed from Refactor to Issue
  • Severity set to 2. Medium
  • Triaged set to No

#12 Updated by mhrivnak almost 5 years ago

  • Triaged changed from No to Yes

#14 Updated by ipanova@redhat.com almost 4 years ago

  • Sprint/Milestone set to 31

#15 Updated by bizhang almost 4 years ago

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

#16 Updated by bizhang almost 4 years ago

  • Status changed from ASSIGNED to POST

#18 Updated by bizhang almost 4 years ago

  • Status changed from POST to MODIFIED

#19 Updated by semyers almost 4 years ago

  • Platform Release set to 2.11.2

#20 Updated by semyers over 3 years ago

  • Status changed from MODIFIED to 5

#21 Updated by semyers over 3 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

#22 Updated by bmbouter over 2 years ago

  • Sprint set to Sprint 13

#23 Updated by bmbouter over 2 years ago

  • Sprint/Milestone deleted (31)

#24 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF