Project

General

Profile

Bug #33370

Could not update repository with URL (ISE 500 in logs) when url was not previously set

Added by Justin Sherrill 9 months ago. Updated 8 months ago.

Status:
Closed
Priority:
Normal
Category:
Repositories
Target version:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

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

Description of problem:

Could not update repository with URL

Version-Release number of selected component (if applicable):
6.10 snap 15

How reproducible:
same error on different Satellite

Steps to Reproduce:
1. hammer product create --name 'TestProduct' --organization-id 1
2. hammer repository create --content-type yum --name CustomRepo --product 'TestProduct' --organization-id 1
3. hammer repository update --url 'https://stephenw.fedorapeople.org/fakerepos/multiple_errata' --name CustomRepo --product 'TestProduct' --organization-id 1

Actual results:

Could not update the repository:
Task 72f3b063-111c-4665-9304-a90e31d2b452: NoMethodError: undefined method `[]' for true:TrueClass

Expected results:

Repo is updated with URL


Related issues

Related to Katello - Bug #33251: usability issues with updating a repository with invalid dataClosed

Associated revisions

Revision 0b288def (diff)
Added by Justin Sherrill 9 months ago

Fixes #33370 - remote create does not generate tasks

on repository update, if the repo did not have a url
previously

History

#1 Updated by The Foreman Bot 9 months ago

  • Assignee set to Justin Sherrill
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9589 added

#2 Updated by Justin Sherrill 9 months ago

  • Target version set to Katello 4.2.0

#3 Updated by Justin Sherrill 9 months ago

  • Related to Bug #33251: usability issues with updating a repository with invalid data added

#4 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases Katello 4.3.0 added

#5 Updated by Justin Sherrill 9 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Justin Sherrill 9 months ago

  • Triaged changed from No to Yes

#7 Updated by The Foreman Bot 8 months ago

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

Also available in: Atom PDF