Project

General

Profile

Actions

Bug #21243

closed

Rails 5 - Replace alias_method_chain with Module prepend

Added by Ivan Necas over 6 years ago. Updated almost 6 years ago.

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

Description

After we replaced alias_method_chain in remote execution, we're hitting
this when katello around. We're releasing 1.3.5 rex with reverting
the original patch (for 1.15/1.16) but in nightlies, the katello
should get rid or alias_mathod_chain as well, as those two don't work nicely
together

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'

Related issues 4 (0 open4 closed)

Related to Foreman Remote Execution - Bug #21242: SystemStackError (stack level too deep) when calling smart proxy features refresh with Katello 3.4ClosedIvan Necas10/09/2017Actions
Related to Foreman Remote Execution - Refactor #21205: Replace alias_method_chain with Module prepend ClosedIvan Necas10/05/2017Actions
Related to Boot disk - Bug #14797: Medium validation wrong with Katello and foreman_bootdiskClosedLukas ZapletalActions
Blocks Foreman - Tracker #20948: Rails 5.1 upgrade tasksClosed09/16/2017

Actions
Actions #1

Updated by Ivan Necas over 6 years ago

  • Related to Bug #21242: SystemStackError (stack level too deep) when calling smart proxy features refresh with Katello 3.4 added
Actions #2

Updated by Ivan Necas over 6 years ago

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

Updated by Ivan Necas over 6 years ago

Actions #4

Updated by Justin Sherrill over 6 years ago

  • Target version set to 225
  • translation missing: en.field_release set to 284
Actions #5

Updated by Brad Buckingham over 6 years ago

  • Subject changed from Replace alias_method_chain with Module prepend to RAILS 5 - Replace alias_method_chain with Module prepend
Actions #6

Updated by Brad Buckingham over 6 years ago

  • Subject changed from RAILS 5 - Replace alias_method_chain with Module prepend to Rails 5 - Replace alias_method_chain with Module prepend
Actions #7

Updated by Justin Sherrill over 6 years ago

  • Status changed from New to Assigned
Actions #8

Updated by Justin Sherrill over 6 years ago

  • Assignee set to Justin Sherrill
Actions #9

Updated by The Foreman Bot over 6 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/7027 added
Actions #10

Updated by Brad Buckingham over 6 years ago

  • Target version changed from 225 to 232
Actions #11

Updated by Justin Sherrill over 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #12

Updated by Lukas Zapletal over 6 years ago

  • Related to Bug #14797: Medium validation wrong with Katello and foreman_bootdisk added
Actions

Also available in: Atom PDF