Project

General

Profile

Actions
Copy link

Bug #19317

closed

Plugin and dashboard widget registry auto-reloaded

Added by Dominic Cleal over 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Dominic Cleal
Category:
Rails
Target version:
1.16.0
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/4472, https://github.com/theforeman/foreman/pull/4507
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Since #18445, the plugin and related classes may be reloaded, losing registrations - probably for plugin registrations, default dashboard widgets, modified menu items, and permissions.

The change to use require_dependency will allow Rails to track the constants and reload them when the source files are modified, discarding the class variable. The use of require_dependency is however required to ensure eager loading doesn't load the file twice (#18445).

The source files may need to be moved to prevent double-loading and to be excluded from autoloading.

Related issues 5 (0 open — 5 closed)

Related to Foreman - Bug #18445: "Cannot define multiple 'included' blocks" for audit_extensions with eager_load enabledClosedDominic Cleal02/09/2017Actions
Related to Foreman - Bug #10927: Resetting dashboard to default widgets adds no widgetsClosedDominic Cleal06/25/2015Actions
Related to Katello - Bug #19351: transient test failureClosedJustin Sherrill04/21/2017Actions
Has duplicate Foreman - Bug #19318: Plugins infrastructure don't work reliably in developmentDuplicate04/19/2017Actions
Has duplicate Katello - Bug #19290: Unable to retrieve API bindings via hammerDuplicate04/17/2017Actions
Actions
Copy link
 #1

Updated by Dominic Cleal over 7 years ago

  • Related to Bug #18445: "Cannot define multiple 'included' blocks" for audit_extensions with eager_load enabled added
Actions
Copy link
 #2

Updated by Tomer Brisker over 7 years ago

  • Related to Bug #10927: Resetting dashboard to default widgets adds no widgets added
Actions
Copy link
 #3

Updated by Dominic Cleal over 7 years ago

  • Has duplicate Bug #19318: Plugins infrastructure don't work reliably in development added
Actions
Copy link
 #4

Updated by John Mitsch over 7 years ago

  • Has duplicate Bug #19290: Unable to retrieve API bindings via hammer added
Actions
Copy link
 #5

Updated by The Foreman Bot over 7 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/4472 added
Actions
Copy link
 #6

Updated by Justin Sherrill over 7 years ago

  • Related to Bug #19351: transient test failure added
Actions
Copy link
 #7

Updated by Dominic Cleal over 7 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions
Copy link
 #8

Updated by The Foreman Bot over 7 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4507 added
Actions
Copy link
 #9

Updated by Marek Hulán over 7 years ago

  • Translation missing: en.field_release set to 240
Actions
Copy link

Also available in: Atom PDF