Project

General

Profile

Actions

Bug #28354

closed

Pulp 3 Docker repo distribution collision for mirrors when changing environment naming scheme

Added by Ian Ballou almost 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

If you publish a Docker content view, sync it with a mirror, change the environment naming scheme, and the sync again, it'll fail with an error like this: https://paste.fedoraproject.org/paste/s6ekvDoQGtbUqx-F~VY2pQ

The issue is that, in refresh_distributions for in repository_mirror.rb, we look up the current distributions by their path, when we should be looking them up by their path and their name.

Actions #1

Updated by Ian Ballou almost 5 years ago

Changing the if statement in refresh_distributions to

if (distro = repo_service.lookup_distributions(base_path: path).first) ||
   (distro = repo_service.lookup_distributions(name: "#{backend_object_name}").first)

should fix it.

Actions #2

Updated by Ian Ballou almost 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/8429 added
Actions #3

Updated by Samir Jha almost 5 years ago

  • Target version set to Katello 3.15.0
  • Triaged changed from No to Yes
Actions #4

Updated by Ian Ballou almost 5 years ago

  • Status changed from Ready For Testing to Closed
  • Fixed in Releases Katello 3.15.0 added
Actions #5

Updated by Jonathon Turel over 4 years ago

  • Category set to Repositories
Actions

Also available in: Atom PDF