Project

General

Profile

Bug #35534

Repo Deletion with no feed url causes a `ArgumentError`

Added by Partha Aji 2 months ago. Updated 2 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

Deleting repositories that have no feed url causes the following error.


ArgumentError
bad argument (expected URI object or URI string)
- "/usr/share/ruby/uri/common.rb:739:in `URI'" 
- "/home/vagrant/katello/app/services/katello/pulp3/repository/yum.rb:15:in `remote_options'" 
- "/home/vagrant/katello/app/services/katello/pulp3/repository.rb:102:in `block in
  delete_remote'" 
- "/home/vagrant/katello/app/services/katello/pulp3/service_common.rb:51:in `ignore_404_exception'" 
- "/home/vagrant/katello/app/services/katello/pulp3/repository.rb:102:in `delete_remote'" 
- "/home/vagrant/katello/app/lib/actions/pulp3/repository/delete_remote.rb:11:in `invoke_external_task'" 

Steps to repro
  • Create a custom repo with feed url
  • Now remove feed url and update the repo
  • Delete the repo

Expected:
Successful deletion

Actual:
Above error

Additional Notes:
When a repo has a feed url a repo remote is created in pulp. Possibly on repo update the remotes are not getting cleaned properly.

Associated revisions

Revision ab0cce83 (diff)
Added by Samir Jha 2 months ago

Fixes #35534 - Repo Deletion with no feed url causes a (#10283)

History

#1 Updated by Samir Jha 2 months ago

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

#2 Updated by Samir Jha 2 months ago

  • Bugzilla link set to 2128864

#3 Updated by The Foreman Bot 2 months ago

  • Assignee set to Samir Jha
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/10283 added

#4 Updated by The Foreman Bot 2 months ago

  • Fixed in Releases Katello 4.7.0 added

#5 Updated by Samir Jha 2 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF