Project

General

Profile

Actions

Bug #35534

closed

Repo Deletion with no feed url causes a `ArgumentError`

Added by Partha Aji over 1 year ago. Updated over 1 year ago.

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

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.

Actions #1

Updated by Samir Jha over 1 year ago

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

Updated by Samir Jha over 1 year ago

  • Bugzilla link set to 2128864
Actions #3

Updated by The Foreman Bot over 1 year ago

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

Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases Katello 4.7.0 added
Actions #5

Updated by Samir Jha over 1 year ago

  • Status changed from Ready For Testing to Closed
Actions #6

Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/katello/pull/10375 added
Actions #7

Updated by Samir Jha over 1 year ago

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

Also available in: Atom PDF