Bug #21523: "stack level too deep" when viewing host with monitoring plugin installed. - Salt - Foreman

Actions

Copy link

Bug #21523

closed

"stack level too deep" when viewing host with monitoring plugin installed.

Added by Aaron Glenn over 7 years ago. Updated over 7 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Timo Goebel

Category:

Foreman Integration

Target version:

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman_salt/pull/71

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

Installed new foreman server v1.15.6 today with monitoring plugin installed. Stacktrace is attached.

Files

Untitled.txt

1.41 MB

stacktrace

Aaron Glenn, 10/31/2017 07:39 PM

Actions

Copy link

Updated by Aaron Glenn over 7 years ago

Removed monitoring plugin and restarted to resolve the issue.

Actions

Copy link

Updated by Greg Sutcliffe over 7 years ago

Category set to Plugin integration

I forgot that foreman_monitoring uses GitHub issues. Opened https://github.com/theforeman/foreman_monitoring/issues/30 to track this.

We should keep this open for now in case it becomes obvious we need to make changes in the Foreman core.

Actions

Copy link

Updated by Timo Goebel over 7 years ago

Thanks for reporting. This can easily be avoided by replacing alias_method_chain with Module.prepend. Expect a new release of the monitoring plugin by the end of this week. This will address this issue.

Actions

Copy link

Updated by Timo Goebel over 7 years ago

Project changed from Foreman to Salt
Category changed from Plugin integration to Foreman Integration
Assignee set to Timo Goebel

I believe this should be fixed in foreman_salt by changing alias_method_chain to Module.prepend. The monitoring plugin isn't doing anything wrong in my opinion.

Actions

Copy link