Bug #33240: Facts import after code change in DEV environment don't work - Foreman

Actions

Copy link

Bug #33240

closed

Facts import after code change in DEV environment don't work

Added by Leos Stejskal over 2 years ago. Updated over 2 years ago.

Status:

Resolved

Priority:

Normal

Assignee:

Adi Abramovitch

Category:

Facts

Target version:

Difficulty:

Triaged:

Yes

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman/pull/8027

Fixed in Releases:

Found in Releases:

Nightly

Red Hat JIRA:

Description

Change in the code while in DEV environment cause error during fact import, the only fix is restart.

Steps to reproduce

Generate registration command (Hosts > Register Host) and register CentOS machine
Do some change in the code (for example add comment somewhere in controller classes)
Try to register host again (with --force parameter)

Error

2021-08-11T10:53:05 [I|app|80d9e34e] Import facts for 'stream.example.com' completed. Added: 156, Updated: 0, Deleted 0 facts
2021-08-11T10:53:05 [E|kat|80d9e34e] NoMethodError: undefined method `new' for nil:NilClass
 80d9e34e | /home/vagrant/foreman/app/services/host_fact_importer.rb:47:in `block in parse_facts'
 80d9e34e | /home/vagrant/foreman/app/services/host_fact_importer.rb:90:in `block in skipping_orchestration'
 80d9e34e | /home/vagrant/foreman/app/models/concerns/orchestration.rb:124:in `without_orchestration'
 80d9e34e | /home/vagrant/foreman/app/services/host_fact_importer.rb:89:in `skipping_orchestration'
 80d9e34e | /home/vagrant/foreman/app/services/host_fact_importer.rb:45:in `parse_facts'
 80d9e34e | /home/vagrant/foreman/app/services/host_fact_importer.rb:34:in `import_facts'
 80d9e34e | /home/vagrant/foreman/app/models/host/base.rb:143:in `import_facts'
 80d9e34e | /home/vagrant/katello/app/models/katello/host/subscription_facet.rb:220:in `update_facts'
 80d9e34e | /home/vagrant/katello/app/services/katello/registration_manager.rb:217:in `create_in_candlepin'
 80d9e34e | /home/vagrant/katello/app/services/katello/registration_manager.rb:171:in `block in register_host'
 80d9e34e | /home/vagrant/foreman/app/models/concerns/foreman/thread_session.rb:108:in `as'
 80d9e34e | /home/vagrant/foreman/app/models/concerns/foreman/thread_session.rb:114:in `as_anonymous_admin'
 80d9e34e | /home/vagrant/katello/app/services/katello/registration_manager.rb:169:in `register_host'
 80d9e34e | /home/vagrant/katello/app/services/katello/registration_manager.rb:40:in `process_registration'
 80d9e34e | /home/vagrant/katello/app/controllers/katello/api/rhsm/candlepin_proxies_controller.rb:234:in `consumer_activate'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/abstract_controller/base.rb:195:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal/rendering.rb:30:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/abstract_controller/callbacks.rb:42:in `block in process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:112:in `block in run_callbacks'
 80d9e34e | /home/vagrant/katello/app/controllers/katello/api/rhsm/candlepin_proxies_controller.rb:35:in `repackage_message'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:121:in `block in run_callbacks'
 80d9e34e | /home/vagrant/foreman/app/controllers/concerns/foreman/controller/timezone.rb:10:in `set_timezone'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:121:in `block in run_callbacks'
 80d9e34e | /home/vagrant/foreman/app/models/concerns/foreman/thread_session.rb:32:in `clear_thread'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:121:in `block in run_callbacks'
 80d9e34e | /home/vagrant/foreman/app/controllers/concerns/foreman/controller/topbar_sweeper.rb:12:in `set_topbar_sweeper_controller'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:121:in `block in run_callbacks'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/audited-4.10.0/lib/audited/sweeper.rb:14:in `around'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:121:in `block in run_callbacks'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/audited-4.10.0/lib/audited/sweeper.rb:14:in `around'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:121:in `block in run_callbacks'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:139:in `run_callbacks'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/abstract_controller/callbacks.rb:41:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal/rescue.rb:22:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal/instrumentation.rb:33:in `block in process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/notifications.rb:180:in `block in instrument'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/notifications/instrumenter.rb:24:in `instrument'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/notifications.rb:180:in `instrument'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal/instrumentation.rb:32:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal/params_wrapper.rb:245:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activerecord-6.0.3.7/lib/active_record/railties/controller_runtime.rb:27:in `process_action'
 80d9e34e | /home/vagrant/katello/app/controllers/katello/concerns/api/api_controller.rb:50:in `process_action'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/abstract_controller/base.rb:136:in `process'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionview-6.0.3.7/lib/action_view/rendering.rb:39:in `process'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal.rb:190:in `dispatch'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_controller/metal.rb:254:in `dispatch'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/route_set.rb:50:in `dispatch'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/route_set.rb:33:in `serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/mapper.rb:18:in `block in <class:Constraints>'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/mapper.rb:48:in `serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/journey/router.rb:49:in `block in serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/journey/router.rb:32:in `each'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/journey/router.rb:32:in `serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/route_set.rb:834:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/engine.rb:527:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/railtie.rb:190:in `public_send'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/railtie.rb:190:in `method_missing'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/mapper.rb:19:in `block in <class:Constraints>'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/mapper.rb:48:in `serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/journey/router.rb:49:in `block in serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/journey/router.rb:32:in `each'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/journey/router.rb:32:in `serve'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/routing/route_set.rb:834:in `call'
 80d9e34e | /home/vagrant/katello/lib/katello/middleware/organization_created_enforcer.rb:18:in `call'
 80d9e34e | /home/vagrant/katello/lib/katello/middleware/event_daemon.rb:10:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/static.rb:126:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/static.rb:126:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/apipie-dsl-2.4.0/lib/apipie_dsl/static_dispatcher.rb:67:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/apipie-rails-0.5.19/lib/apipie/static_dispatcher.rb:66:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/bullet-6.1.4/lib/bullet/rack.rb:15:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/apipie-rails-0.5.19/lib/apipie/extractor/recorder.rb:137:in `call'
 80d9e34e | /home/vagrant/foreman/lib/foreman/middleware/telemetry.rb:10:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/apipie-rails-0.5.19/lib/apipie/middleware/checksum_in_headers.rb:27:in `call'
 80d9e34e | /home/vagrant/foreman/lib/foreman/middleware/catch_json_parse_errors.rb:9:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-openid-1.4.2/lib/rack/openid.rb:98:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/tempfile_reaper.rb:15:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/etag.rb:27:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/conditional_get.rb:40:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/head.rb:12:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/http/content_security_policy.rb:18:in `call'
 80d9e34e | /home/vagrant/foreman/lib/foreman/middleware/logging_context_session.rb:22:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/session/abstract/id.rb:266:in `context'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/session/abstract/id.rb:260:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/cookies.rb:648:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activerecord-6.0.3.7/lib/active_record/migration.rb:567:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/callbacks.rb:27:in `block in call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/callbacks.rb:101:in `run_callbacks'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/callbacks.rb:26:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/executor.rb:14:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/actionable_exceptions.rb:18:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/debug_exceptions.rb:32:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/show_exceptions.rb:33:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/rack/logger.rb:37:in `call_app'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/rack/logger.rb:28:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/sprockets-rails-3.2.2/lib/sprockets/rails/quiet_assets.rb:13:in `call'
 80d9e34e | /home/vagrant/foreman/lib/foreman/middleware/logging_context_request.rb:11:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/remote_ip.rb:81:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/request_id.rb:27:in `call'
 80d9e34e | /home/vagrant/katello/lib/katello/prevent_json_parsing.rb:12:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/method_override.rb:24:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/runtime.rb:22:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/activesupport-6.0.3.7/lib/active_support/cache/strategy/local_cache_middleware.rb:29:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/executor.rb:14:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/static.rb:126:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/sendfile.rb:110:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/actionpack-6.0.3.7/lib/action_dispatch/middleware/host_authorization.rb:82:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/secure_headers-6.3.2/lib/secure_headers/middleware.rb:11:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/engine.rb:527:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/railtie.rb:190:in `public_send'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/railties-6.0.3.7/lib/rails/railtie.rb:190:in `method_missing'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/urlmap.rb:74:in `block in call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/urlmap.rb:58:in `each'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/rack-2.2.3/lib/rack/urlmap.rb:58:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/puma-5.4.0/lib/puma/configuration.rb:249:in `call'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/puma-5.4.0/lib/puma/request.rb:77:in `block in handle_request'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/puma-5.4.0/lib/puma/thread_pool.rb:340:in `with_force_shutdown'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/puma-5.4.0/lib/puma/request.rb:76:in `handle_request'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/puma-5.4.0/lib/puma/server.rb:440:in `process_client'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/puma-5.4.0/lib/puma/thread_pool.rb:147:in `block in spawn_thread'
 80d9e34e | /home/vagrant/foreman/.vendor/ruby/2.7.0/gems/logging-2.3.0/lib/logging/diagnostic_context.rb:474:in `block in create_with_logging_context'
2021-08-11T10:53:05 [I|app|80d9e34e] Completed 500 Internal Server Error in 4647ms (Views: 0.3ms | ActiveRecord: 439.8ms | Allocations: 512389)
 80d9e34e | 
 80d9e34e | 
2021-08-11T10:53:05 [D|app|80d9e34e] With body: {"displayMessage":"undefined method `new' for nil:NilClass","errors":["undefined method `new' for nil:NilClass"]}
 80d9e34e |