Bug #23658
closedForeman Salt Plugin 10.0 upload-salt-reports error with Foreman 1.17
Added by Nathan Grennan over 6 years ago. Updated over 6 years ago.
Description
I installed the Foreman Salt Plugin 10.0 with Foreman 1.17, but get the error below. I also tried 8.0.4 and 9.0.1, same error. I downgraded to Foreman 1.16 with Foreman Salt Plugin 9.0.1, and it works.
upload-salt-reports:
Unable to upload job - aborting report upload
{
"error": {"message":"unable to convert unpermitted parameters to hash"}
}
I lost the backtrace on reinstall, it pointed to line 25 in the code below.
app/controllers/foreman_salt/api/v2/jobs_controller.rb:
Rails.logger.info("Processing job #{params[:job][:job_id]} from Salt.")
case params[:job][:function]
when 'state.highstate'
--> task = ForemanTasks.async_task(::Actions::ForemanSalt::ReportImport, params[:job], detected_proxy.try(:id))
render :json => { :task_id => task.id }
else
render :json => { :message => 'Unsupported function' }, :status => :unprocessable_entity
Updated by Nathan Grennan over 6 years ago
- Subject changed from Foreman Salt Plugin 10.0 with Foreman 1.17 to Foreman Salt Plugin 10.0 upload-salt-reports error with Foreman 1.17
Updated by Nathan Grennan over 6 years ago
Found a backtrace I had saved in pastebin:
2018-05-18T23:02:04 0ea272ea [background] [E] unable to convert unpermitted parameters to hash (ActionController::UnfilteredParameters) | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/strong_parameters.rb:265:in `to_h' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/strong_parameters.rb:282:in `to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:50:in `recursive_to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `block in recursive_to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `each' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `inject' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `recursive_to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:561:in `check_serializable' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:503:in `block in execute_plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:444:in `block in with_error_handling' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:444:in `catch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:444:in `with_error_handling' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:483:in `execute_plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:277:in `execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan/steps/plan_step.rb:54:in `block in execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan/steps/abstract.rb:155:in `with_meta_calculation' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan/steps/plan_step.rb:53:in `execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:270:in `block (2 levels) in plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:368:in `switch_flow' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:358:in `with_planning_scope' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:269:in `block in plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:26:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:18:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:43:in `plan_phase' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:26:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:18:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:43:in `plan_phase' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:26:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:18:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/common/transaction.rb:16:in `block in rollback_on_error' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/database_statements.rb:235:in `block in transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/transaction.rb:194:in `block in within_new_transaction' | /usr/lib/ruby/2.3.0/monitor.rb:214:in `mon_synchronize' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/transaction.rb:191:in `within_new_transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/database_statements.rb:235:in `transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/transactions.rb:210:in `transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/transaction_adapters/active_record.rb:5:in `transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/common/transaction.rb:15:in `rollback_on_error' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/common/transaction.rb:5:in `plan_phase' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/world.rb:30:in `execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:268:in `plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:174:in `block in plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:172:in `tap' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:172:in `plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:148:in `trigger' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:19:in `trigger' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:25:in `block in trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:39:in `block in rails_safe_trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/dependencies/interlock.rb:46:in `block in permit_concurrent_loads' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/concurrency/share_lock.rb:185:in `yield_shares' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/dependencies/interlock.rb:45:in `permit_concurrent_loads' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:38:in `rails_safe_trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:23:in `trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:47:in `async_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman_salt-10.0.0/app/controllers/foreman_salt/api/v2/jobs_controller.rb:25:in `upload' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/basic_implicit_render.rb:4:in `send_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/base.rb:186:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/rendering.rb:30:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/callbacks.rb:20:in `block in process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:108:in `block in run_callbacks' | /usr/share/foreman/app/controllers/api/v2/base_controller.rb:152:in `disable_json_root' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/app/controllers/concerns/application_shared.rb:15:in `set_timezone' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/app/models/concerns/foreman/thread_session.rb:32:in `clear_thread' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/app/controllers/concerns/foreman/controller/topbar_sweeper.rb:12:in `set_topbar_sweeper_controller' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/audited-4.5.0/lib/audited/sweeper.rb:14:in `around' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/audited-4.5.0/lib/audited/sweeper.rb:14:in `around' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:135:in `run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/callbacks.rb:19:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/rescue.rb:20:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/instrumentation.rb:32:in `block in process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/notifications.rb:166:in `block in instrument' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/notifications/instrumenter.rb:21:in `instrument' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/notifications.rb:166:in `instrument' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/instrumentation.rb:30:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/params_wrapper.rb:252:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/railties/controller_runtime.rb:22:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/base.rb:124:in `process' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionview-5.1.4/lib/action_view/rendering.rb:30:in `process' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal.rb:189:in `dispatch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal.rb:253:in `dispatch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/route_set.rb:49:in `dispatch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/route_set.rb:31:in `serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/mapper.rb:16:in `block in <class:Constraints>' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/mapper.rb:46:in `serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/journey/router.rb:50:in `block in serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/journey/router.rb:33:in `each' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/journey/router.rb:33:in `serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/route_set.rb:834:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/apipie-rails-0.5.7/lib/apipie/static_dispatcher.rb:65:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/apipie-rails-0.5.7/lib/apipie/extractor/recorder.rb:136:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/apipie-rails-0.5.7/lib/apipie/middleware/checksum_in_headers.rb:27:in `call' | /usr/share/foreman/lib/middleware/catch_json_parse_errors.rb:8:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/etag.rb:25:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/conditional_get.rb:38:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/head.rb:12:in `call' | /usr/share/foreman/lib/middleware/session_safe_logging.rb:17:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/session/abstract/id.rb:232:in `context' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/session/abstract/id.rb:226:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/cookies.rb:613:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/callbacks.rb:26:in `block in call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:97:in `run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/callbacks.rb:24:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/debug_exceptions.rb:59:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/show_exceptions.rb:31:in `call' 2018-05-18T23:02:04 0ea272ea [background] [E] unable to convert unpermitted parameters to hash (ActionController::UnfilteredParameters) | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/strong_parameters.rb:265:in `to_h' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/strong_parameters.rb:282:in `to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:50:in `recursive_to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `block in recursive_to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `each' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `inject' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/serializable.rb:46:in `recursive_to_hash' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:561:in `check_serializable' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:503:in `block in execute_plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:444:in `block in with_error_handling' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:444:in `catch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:444:in `with_error_handling' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:483:in `execute_plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/action.rb:277:in `execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan/steps/plan_step.rb:54:in `block in execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan/steps/abstract.rb:155:in `with_meta_calculation' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan/steps/plan_step.rb:53:in `execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:270:in `block (2 levels) in plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:368:in `switch_flow' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:358:in `with_planning_scope' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:269:in `block in plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:26:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:18:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:43:in `plan_phase' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:26:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:18:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:43:in `plan_phase' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:26:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware.rb:18:in `pass' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/common/transaction.rb:16:in `block in rollback_on_error' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/database_statements.rb:235:in `block in transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/transaction.rb:194:in `block in within_new_transaction' | /usr/lib/ruby/2.3.0/monitor.rb:214:in `mon_synchronize' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/transaction.rb:191:in `within_new_transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/connection_adapters/abstract/database_statements.rb:235:in `transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/transactions.rb:210:in `transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/transaction_adapters/active_record.rb:5:in `transaction' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/common/transaction.rb:15:in `rollback_on_error' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/common/transaction.rb:5:in `plan_phase' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/stack.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/middleware/world.rb:30:in `execute' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/execution_plan.rb:268:in `plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:174:in `block in plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:172:in `tap' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:172:in `plan' | /usr/share/foreman/vendor/ruby/2.3.0/gems/dynflow-0.8.37/lib/dynflow/world.rb:148:in `trigger' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:19:in `trigger' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:25:in `block in trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:39:in `block in rails_safe_trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/dependencies/interlock.rb:46:in `block in permit_concurrent_loads' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/concurrency/share_lock.rb:185:in `yield_shares' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/dependencies/interlock.rb:45:in `permit_concurrent_loads' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:38:in `rails_safe_trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:23:in `trigger_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman-tasks-0.11.2/lib/foreman_tasks.rb:47:in `async_task' | /usr/share/foreman/vendor/ruby/2.3.0/gems/foreman_salt-10.0.0/app/controllers/foreman_salt/api/v2/jobs_controller.rb:25:in `upload' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/basic_implicit_render.rb:4:in `send_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/base.rb:186:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/rendering.rb:30:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/callbacks.rb:20:in `block in process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:108:in `block in run_callbacks' | /usr/share/foreman/app/controllers/api/v2/base_controller.rb:152:in `disable_json_root' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/app/controllers/concerns/application_shared.rb:15:in `set_timezone' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/app/models/concerns/foreman/thread_session.rb:32:in `clear_thread' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/app/controllers/concerns/foreman/controller/topbar_sweeper.rb:12:in `set_topbar_sweeper_controller' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/audited-4.5.0/lib/audited/sweeper.rb:14:in `around' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/audited-4.5.0/lib/audited/sweeper.rb:14:in `around' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:117:in `block in run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:135:in `run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/callbacks.rb:19:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/rescue.rb:20:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/instrumentation.rb:32:in `block in process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/notifications.rb:166:in `block in instrument' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/notifications/instrumenter.rb:21:in `instrument' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/notifications.rb:166:in `instrument' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/instrumentation.rb:30:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal/params_wrapper.rb:252:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activerecord-5.1.4/lib/active_record/railties/controller_runtime.rb:22:in `process_action' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/abstract_controller/base.rb:124:in `process' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionview-5.1.4/lib/action_view/rendering.rb:30:in `process' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal.rb:189:in `dispatch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_controller/metal.rb:253:in `dispatch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/route_set.rb:49:in `dispatch' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/route_set.rb:31:in `serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/mapper.rb:16:in `block in <class:Constraints>' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/mapper.rb:46:in `serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/journey/router.rb:50:in `block in serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/journey/router.rb:33:in `each' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/journey/router.rb:33:in `serve' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/routing/route_set.rb:834:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/apipie-rails-0.5.7/lib/apipie/static_dispatcher.rb:65:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/apipie-rails-0.5.7/lib/apipie/extractor/recorder.rb:136:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/apipie-rails-0.5.7/lib/apipie/middleware/checksum_in_headers.rb:27:in `call' | /usr/share/foreman/lib/middleware/catch_json_parse_errors.rb:8:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/etag.rb:25:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/conditional_get.rb:38:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/head.rb:12:in `call' | /usr/share/foreman/lib/middleware/session_safe_logging.rb:17:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/session/abstract/id.rb:232:in `context' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/session/abstract/id.rb:226:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/cookies.rb:613:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/callbacks.rb:26:in `block in call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/callbacks.rb:97:in `run_callbacks' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/callbacks.rb:24:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/debug_exceptions.rb:59:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/show_exceptions.rb:31:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/railties-5.1.4/lib/rails/rack/logger.rb:36:in `call_app' | /usr/share/foreman/vendor/ruby/2.3.0/gems/railties-5.1.4/lib/rails/rack/logger.rb:26:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/sprockets-rails-3.2.1/lib/sprockets/rails/quiet_assets.rb:13:in `call' | /usr/share/foreman/lib/middleware/tagged_logging.rb:18:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/remote_ip.rb:79:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/request_id.rb:25:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/method_override.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/runtime.rb:22:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/activesupport-5.1.4/lib/active_support/cache/strategy/local_cache_middleware.rb:27:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/executor.rb:12:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/actionpack-5.1.4/lib/action_dispatch/middleware/static.rb:125:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/sendfile.rb:111:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/secure_headers-3.7.3/lib/secure_headers/middleware.rb:12:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/railties-5.1.4/lib/rails/engine.rb:522:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/railties-5.1.4/lib/rails/railtie.rb:185:in `public_send' | /usr/share/foreman/vendor/ruby/2.3.0/gems/railties-5.1.4/lib/rails/railtie.rb:185:in `method_missing' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/urlmap.rb:68:in `block in call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/urlmap.rb:53:in `each' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/urlmap.rb:53:in `call' | /usr/share/foreman/vendor/ruby/2.3.0/gems/rack-2.0.4/lib/rack/handler/webrick.rb:86:in `service' | /usr/lib/ruby/2.3.0/webrick/httpserver.rb:140:in `service' | /usr/lib/ruby/2.3.0/webrick/httpserver.rb:96:in `run' | /usr/lib/ruby/2.3.0/webrick/server.rb:296:in `block in start_thread' | /usr/share/foreman/vendor/ruby/2.3.0/gems/logging-2.2.2/lib/logging/diagnostic_context.rb:474:in `block in create_with_logging_context'
Updated by Ondřej Pražák over 6 years ago
Hi, thanks for the backtrace, it will be useful. Unfortunately, foreman_salt needs a new maintainer so I cannot tell when this will be fixed. There is a Discourse thread about it.
Updated by Greg Sutcliffe over 6 years ago
- Category set to API
- Status changed from New to Assigned
- Assignee set to Greg Sutcliffe
- Priority changed from Normal to High
I'm digging into this for my own Salt setup. This should allow a single import:
--- app/controllers/foreman_salt/api/v2/jobs_controller.rb.orij 2018-06-25 21:23:50.425522920 +0000 +++ app/controllers/foreman_salt/api/v2/jobs_controller.rb 2018-06-25 21:23:48.711546489 +0000 @@ -19,7 +19,6 @@ param_group :job, :as => :upload def upload + params[:job].permit! Rails.logger.info("Processing job #{params[:job][:job_id]} from Salt.") Rails.logger.debug("GSS: #{params.inspect}") case params[:job][:function]
Unfortunately the upload-salt-reports script then bombs on a bad status return, still looking into that. Nathan, let me know if that helps, I can send a PR if so
Updated by Greg Sutcliffe over 6 years ago
- Status changed from Assigned to Closed
- Triaged set to No
- Pull request https://github.com/theforeman/foreman_salt/pull/79 added
Should be fixed in master, new release of Salt plugin will be out shortly
Updated by Greg Sutcliffe over 6 years ago
- Related to Bug #24133: Rails 5.1 error when submitting salt report: "unable to convert unpermitted parameters to hash (ActionController::UnfilteredParameters)" added
Updated by Greg Sutcliffe over 6 years ago
- Target version set to 10.1.0
- Found in Releases 10.0 (or older) added