Project

General

Profile

Bug #32590

20210119162528_delete_puppet_and_ostree_repos.rb fails to remove puppet repos in CVs, leading to upgrade errors

Added by James Jeffers about 1 year ago. Updated 12 months ago.

Status:
Closed
Priority:
Normal
Category:
Upgrades
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

foreman-rake katello:correct_puppet_environments
All operations will be skipped.  Re-run with COMMIT=true to perform corrections.
Processing Puppet Environment 1/4: 3-8-default-v20_0-puppet-ff4590c3-68ae-4507-8c76-8c0e305e30bf (37)
rake aborted!
Cannot find repository type puppet, is it enabled?
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-4.0.0/app/models/katello/concerns/smart_proxy_extensions.rb:226:in `pulp3_repository_type_support?'
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-4.0.0/app/models/katello/concerns/smart_proxy_extensions.rb:197:in `pulp3_support?'
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-4.0.0/lib/katello/tasks/repository.rake:140:in `repo_exists?'
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-4.0.0/lib/katello/tasks/repository.rake:81:in `block (3 levels) in <top (required)>'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:70:in `block (2 levels) in find_each'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:70:in `each'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:70:in `block in find_each'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:136:in `block in find_in_batches'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:238:in `block in in_batches'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:222:in `loop'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:222:in `in_batches'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:135:in `find_in_batches'
/opt/theforeman/tfm/root/usr/share/gems/gems/activerecord-6.0.3.4/lib/active_record/relation/batches.rb:69:in `find_each'
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-4.0.0/lib/katello/tasks/repository.rake:79:in `with_index'
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-4.0.0/lib/katello/tasks/repository.rake:79:in `block (2 levels) in <top (required)>'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rake-12.3.0/exe/rake:27:in `<top (required)>'
Tasks: TOP => katello:correct_puppet_environments
(See full trace by running task with --trace)

The query in question is at: https://github.com/Katello/katello/blob/master/db/migrate/20210119162528_delete_puppet_and_ostree_repos.rb#L35

Associated revisions

Revision 49b68610 (diff)
Added by Justin Sherrill 12 months ago

Fixes #32590 - always remove puppet and ostree repos

the previous detection of puppet repos did not account
for the situation where the user had a content view puppet
environment, but no longer had any puppet repos. This now
will run the cleanup code regardless

History

#2 Updated by Justin Sherrill about 1 year ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.0.1
  • Category set to Upgrades

#3 Updated by The Foreman Bot 12 months ago

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

#4 Updated by James Jeffers 12 months ago

  • Target version changed from Katello 4.0.1 to Katello 4.0.2

#5 Updated by The Foreman Bot 12 months ago

  • Fixed in Releases Katello 4.2.0 added

#6 Updated by Justin Sherrill 12 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF