Project

General

Profile

Bug #21242

SystemStackError (stack level too deep) when calling smart proxy features refresh with Katello 3.4

Added by Ivan Necas almost 5 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Foreman
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

2017-10-09 07:12:07 8c03e5df [app] [I] Started PUT "/api/v2/smart_proxies/1/refresh" for 192.168.121.75 at 2017-10-09 07:12:07 +0000
2017-10-09 07:12:07 8c03e5df [app] [I] Processing by Api::V2::SmartProxiesController#refresh as JSON
2017-10-09 07:12:07 8c03e5df [app] [I]   Parameters: {"apiv"=>"v2", "id"=>"1", "smart_proxy"=>{}}

2017-10-09 07:12:07 8c03e5df [app] [I] Authorized user foreman_api_admin(API Admin)
2017-10-09 07:12:07 8c03e5df [app] [I] Current user: foreman_api_admin (administrator)
2017-10-09 07:12:07 8c03e5df [app] [I] Completed 500 Internal Server Error in 19ms (ActiveRecord: 1.8ms)
2017-10-09 07:12:08 8c03e5df [app] [F]
 | SystemStackError (stack level too deep):
 |   katello (3.4.5.6) app/models/katello/concerns/smart_proxy_extensions.rb:80:in `refresh_with_puppet_path'
 |   foreman_remote_execution (1.3.4) app/models/concerns/foreman_remote_execution/smart_proxy_extensions.rb:15:in `refresh'
 |   katello (3.4.5.6) app/models/katello/concerns/smart_proxy_extensions.rb:80:in `refresh_with_puppet_path'
 |   foreman_remote_execution (1.3.4) app/models/concerns/foreman_remote_execution/smart_proxy_extensions.rb:15:in `refresh'

This was initiated by #21205. Since this change was needed for the rex to work with nightlies,
the proper fix should go to the Katello side. On the other hand, since we aim for 1.3.4 to be compatible with 1.15 and 1.16,
we will start with 1.3.x branch and revert the changes in this branch. The proper fix should still go to the Katello
to remove the usage of alias_method_chain.


Related issues

Related to Foreman Remote Execution - Refactor #21205: Replace alias_method_chain with Module prepend Closed2017-10-05
Related to Katello - Bug #21243: Rails 5 - Replace alias_method_chain with Module prependClosed2017-10-09

History

#1 Updated by The Foreman Bot almost 5 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_remote_execution/pull/281 added

#2 Updated by Ivan Necas almost 5 years ago

  • Related to Refactor #21205: Replace alias_method_chain with Module prepend added

#3 Updated by Ivan Necas almost 5 years ago

  • Related to Bug #21243: Rails 5 - Replace alias_method_chain with Module prepend added

#4 Updated by Ivan Necas almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 317

#5 Updated by Ivan Necas almost 5 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF