Bug #33534: Update of the Registry Name Pattern fails - Katello - Foreman

Bug #33534

Update of the Registry Name Pattern fails

Added by Ian Ballou over 1 year ago. Updated over 1 year ago.

Status:

Closed

Priority:

Normal

Assignee:

Ian Ballou

Category:

Repositories

Target version:

Katello 4.2.0

Difficulty:

Triaged:

Yes

Bugzilla link:

2005137

Pull request:

https://github.com/Katello/katello/pull/9663

Fixed in Releases:

Katello 4.3.0

Found in Releases:

Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2005137

Description of problem:
Update of the Registry Name Pattern fails if a Container repo already exists.

Version-Release number of selected component (if applicable):
Katello 4.1

How reproducible:
always

Steps to Reproduce:
1. Sync a Container image repo (for example foreman/foreman from quay.io)
2. In UI > Content > Lifecycle Envs > Library try to change the Registry Name Pattern to one of the provided in examples

Actual results:
Fails with `An error occurred saving the Environment: There was an issue with the backend service pulp: 404 Not Found`

Expected results:
No error and successful update of existing repository

Associated revisions

Revision 7c2bb053 (diff)
Added by Ian Ballou over 1 year ago

Fixes #33534 - registry name pattern update fails (#9663)

History

#1 Updated by Ian Ballou over 1 year ago

Subject changed from Update of the Registry Name Pattern fails to Update of the Registry Name Pattern fails

I'm not seeing this error on master, but I am seeing a different one:

18:21:04 rails.1   | 2021-09-21T18:21:04 [E|app|5798cdd8] ArgumentError: wrong number of arguments (given 1, expected 2..3)
18:21:04 rails.1   |  5798cdd8 | /home/vagrant/katello/app/lib/actions/pulp3/orchestration/repository/refresh_if_needed.rb:6:in `plan'

The logic for refreshing distributions needs to be updated too. Currently, changing container registry naming patterns for lifecycle environments creates orphaned distributions.