Project

General

Profile

Bug #33534

Update of the Registry Name Pattern fails

Added by Ian Ballou 8 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Repositories
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
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 8 months ago

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

History

#1 Updated by Ian Ballou 8 months 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.

#2 Updated by The Foreman Bot 8 months ago

  • Assignee set to Ian Ballou
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9663 added

#3 Updated by Samir Jha 8 months ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.2.0

#4 Updated by The Foreman Bot 8 months ago

  • Fixed in Releases Katello 4.3.0 added

#5 Updated by Anonymous 8 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by The Foreman Bot 7 months ago

  • Pull request https://github.com/Katello/katello/pull/9696 added

#7 Updated by Justin Sherrill 7 months ago

  • Pull request deleted (https://github.com/Katello/katello/pull/9696)

Also available in: Atom PDF