Bug 1646748 - Upgrade to Satellite 6.4 is failning on script 20180516103339_update_idm_params.rb
Summary: Upgrade to Satellite 6.4 is failning on script 20180516103339_update_idm_para...
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Branding
Version: 6.4
Hardware: All
OS: All
Target Milestone: 6.4.1
Assignee: satellite6-bugs
QA Contact: Nikhil Kathole
Depends On:
TreeView+ depends on / blocked
Reported: 2018-11-05 22:51 UTC by Mike McCune
Modified: 2019-10-30 19:37 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1641017
Last Closed: 2018-12-06 22:32:53 UTC
Target Upstream Version:

Attachments (Terms of Use)
pre_upgrade screenshot (32.08 KB, image/png)
2018-11-16 11:05 UTC, Nikhil Kathole
no flags Details
post_upgrade screenshot (45.54 KB, image/png)
2018-11-16 11:06 UTC, Nikhil Kathole
no flags Details

System ID Private Priority Status Summary Last Updated
Red Hat Knowledge Base (Solution) 3660281 0 None None None 2018-11-05 22:52:01 UTC
Red Hat Product Errata RHBA-2018:3799 0 None None None 2018-12-06 22:33:26 UTC

Comment 2 Ivan Necas 2018-11-16 09:20:09 UTC
The easiest way to reproduce is via foreman-rake console in sat 63 before upgrade:

param = CommonParameter.new(name: 'idm_server invalid', value: 'example.com')
param.save(:validate => false)

This will simulate a presence of invalid parameter. The upgrade should succeed with this and its value should be 'freeipa_server invalid'/

Comment 3 Nikhil Kathole 2018-11-16 11:04:55 UTC

Version tested:
Satellite 6.3.5 -> 6.4.1 upgrade

# foreman-rake console
Loading production environment (Rails 4.2.6)
irb(main):001:0> param = CommonParameter.new(name: 'idm_server invalid', value: 'example.com')
=> #<CommonParameter id: nil, name: "idm_server invalid", value: "example.com", reference_id: nil, created_at: nil, updated_at: nil, type: "CommonParameter", priority: nil, hidden_value: false>
irb(main):002:0> param.save(:validate => false)
=> true

 Upgrade Step: migrate_foreman...
 /usr/share/foreman/lib/foreman.rb:8: warning: already initialized constant Foreman::UUID_REGEXP
 /usr/share/foreman/lib/foreman.rb:8: warning: previous definition of UUID_REGEXP was here
 /usr/share/foreman/lib/core_extensions.rb:182: warning: already initialized constant ActiveSupport::MessageEncryptor::DEFAULT_CIPHER
 /opt/theforeman/tfm-ror51/root/usr/share/gems/gems/activesupport-5.1.6/lib/active_support/message_encryptor.rb:22: warning: previous definition of DEFAULT_CIPHER was here
 /opt/theforeman/tfm/root/usr/share/gems/gems/katello- warning: already initialized constant Katello::Concerns::ContentFacetHostExtensions::ERRATA_STATUS_MAP
 /opt/theforeman/tfm/root/usr/share/gems/gems/katello- warning: previous definition of ERRATA_STATUS_MAP was here
 /opt/theforeman/tfm/root/usr/share/gems/gems/katello- warning: already initialized constant Katello::Concerns::ContentFacetHostExtensions::TRACE_STATUS_MAP
 /opt/theforeman/tfm/root/usr/share/gems/gems/katello- warning: previous definition of TRACE_STATUS_MAP was here
 /opt/theforeman/tfm/root/usr/share/gems/gems/katello- warning: already initialized constant Katello::Concerns::SubscriptionFacetHostExtensions::SUBSCRIPTION_STATUS_MAP
 /opt/theforeman/tfm/root/usr/share/gems/gems/katello- warning: previous definition of SUBSCRIPTION_STATUS_MAP was here
 foreman-rake db:migrate finished successfully!

Comment 4 Nikhil Kathole 2018-11-16 11:05:56 UTC
Created attachment 1506387 [details]
pre_upgrade screenshot

Comment 5 Nikhil Kathole 2018-11-16 11:06:25 UTC
Created attachment 1506388 [details]
post_upgrade screenshot

Comment 7 errata-xmlrpc 2018-12-06 22:32:53 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.