Project

General

Profile

Actions

Bug #24525

closed

Importing roles via UI broken

Added by Ondřej Pražák over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Target version:
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

UI controllers use the 'notice' method that was previously deprecated and removed once 1.20 was branched.

Steps to reproduce:
1) import roles via ui
2) observe the error

2018-08-02T13:24:30 [F|app|] ArgumentError (wrong number of arguments (given 1, expected 0)):
2018-08-02T13:24:30 [F|app|]   
2018-08-02T13:24:30 [F|app|] /home/vagrant/projects/foreman_ansible/app/controllers/ansible_roles_controller.rb:29:in `import'
 | app/controllers/concerns/application_shared.rb:15:in `set_timezone'
 | app/models/concerns/foreman/thread_session.rb:32:in `clear_thread'
 | app/controllers/concerns/foreman/controller/topbar_sweeper.rb:12:in `set_topbar_sweeper_controller'
 | lib/middleware/telemetry.rb:9:in `call'
 | lib/middleware/catch_json_parse_errors.rb:8:in `call'
 | lib/middleware/logging_context.rb:23:in `call'
 | /home/vagrant/projects/katello/lib/katello/prevent_json_parsing.rb:12:in `call'


Related issues 1 (0 open1 closed)

Related to Ansible - Bug #24979: Importing roles can cause 500ClosedMarek HulánActions
Actions #1

Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_ansible/pull/197 added
Actions #2

Updated by Ondřej Pražák over 5 years ago

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

Updated by Marek Hulán over 5 years ago

  • Related to Bug #24979: Importing roles can cause 500 added
Actions #4

Updated by Marek Hulán over 5 years ago

  • Target version set to foreman-ansible 2.3.0
  • Fixed in Releases foreman-ansible 2.3.0 added
Actions #5

Updated by Marek Hulán over 5 years ago

  • Fixed in Releases foreman-ansible 2.2.11 added
  • Fixed in Releases deleted (foreman-ansible 2.3.0)
Actions

Also available in: Atom PDF