Feature #29991: Enable Zeitwerk autoload mode for Rails 6+ - Foreman

Project

General

Profile

Home
Projects
Help

Search:

Foreman

All Projects

Overview
Activity
Roadmap
Issues
Wiki
Repository

Custom queries

3.17.3 release DONE
3.17.3 release TODO
3.18.2 release DONE
3.18.2 release TODO
3.19.0 release TODO
3.19.1 release DONE
5.0.0 release DONE
5.0.0 release TODO
All bugs by votes
All issues closed in the last 3 weeks
Easy and Trivial Issues
Easy Issues (only)
Foreman - new open issues in last 10 days
Good first issues - Foreman core
high prio open bugs
Host registration [open]
Mine
My issues closed in the last 3 weeks
old bugs
Open with target version set (aka blockers)
registration
taken-none
Team Atlas untriaged bugs
templates
Trivial Issues
Untriaged and opened in the past two weeks

Actions

Copy link

Feature #29991

closed

Enable Zeitwerk autoload mode for Rails 6+

Added by Lukas Zapletal about 6 years ago. Updated 8 months ago.

Status:

Closed

Priority:

Normal

Assignee:

Oleh Fedorenko

Category:

Rails

Target version:

Difficulty:

Triaged:

Bugzilla link:

Pull request:

Fixed in Releases:

3.13.0

Found in Releases:

Red Hat JIRA:

SAT-5755

Description

We should take advantage of the new and threadsafe autoloader: https://guides.rubyonrails.org/autoloading_and_reloading_constants.html#enabling-zeitwerk-mode

Subtasks 37 (0 open — 37 closed)

Refactor #33886: Fix early loads of constants	Closed	Ondřej Ezr	Actions
Refactor #33887: Postpone CacheManager recache after rails is initialized	Rejected	Ondřej Ezr	Actions
Refactor #33888: Postpone Setting load to after_initialization	Rejected		Actions
Refactor #33889: Postpone FogExtensions after initialization	Rejected		Actions
Refactor #33890: Require foreman/gettext directly	Closed		Actions
Refactor #33891: require Telemetry constants	Duplicate		Actions
Refactor #33892: Postpone setting validations to to_prepare	Closed	Ondřej Ezr	Actions
Refactor #33949: Fix early load of plugin fact parsers	Closed	Leos Stejskal	Actions
Refactor #33950: Fix early load of ForemanInternal in initializer	Closed	Leos Stejskal	Actions
Refactor #33957: Require Foreman::Util in initializers	Closed	Leos Stejskal	Actions
Refactor #33964: Load Menu::Loader in 'to_prepare'	Closed	Leos Stejskal	Actions
Refactor #33967: Move basic ActiveRecord::Base custom extensions to ApplicationRecord	Closed	Leos Stejskal	Actions
Refactor #33968: Fix loading of Foreman::Telemetry in initializers	Closed	Leos Stejskal	Actions
Refactor #33975: Move menu loader from registries to services	Rejected		Actions
Refactor #33989: Delay Initialization of ApiPie that depends on Model classes	Closed	Leos Stejskal	Actions
Refactor #33991: Require /lib dependencies in initializers/foreman.rb	Closed	Leos Stejskal	Actions
Refactor #34005: Require middlewares properly	Closed		Actions
Refactor #34006: Plugin and its rbac_support are using Role and Permission models too early	Closed	Ondřej Ezr	Actions
Refactor #34009: Add plugin after_initialize hook	Closed	Ondřej Ezr	Actions
Refactor #34010: Add initialization method to RbacRegistry	Closed	Ondřej Ezr	Actions
Refactor #34011: Move add_permissions_to_default_roles to rbac_registry	Rejected		Actions
Refactor #34012: Move add_all_permissions_to_default_roles to RbacRegistry	Rejected		Actions
Refactor #34013: Move add_resource_permissions_to_default_roles to RbacRegistry	Rejected		Actions
Refactor #34014: Move role creation into RbacRegistry	Rejected		Actions
Refactor #34015: Move permission creation in DB to RbacRegistry	Rejected		Actions
Refactor #34350: Require 'foreman/telemetry' in ldap initializer	Closed	Leos Stejskal	Actions
Refactor #34353: Fix early load of fog_extensions	Closed	Leos Stejskal	Actions
Refactor #33895: setup Zeitwerk inflector	Closed	Ewoud Kohl van Wijngaarden	Actions
Refactor #33896: Document how plugins can adjust Zeitwerk inflector	Rejected		Actions
Refactor #34007: Zeitwerk - remove 'foreman/provision' from init/foreman	Rejected		Actions
Refactor #34137: Auto-loading & requiring /lib code	Closed		Actions
Refactor #34138: Autoload ProxyAPI files from /app/services	Closed		Actions
Refactor #34139: Move Foreman::Renderer from lib/ to app/services	Closed		Actions
Refactor #34146: Move services from lib/foreman to app/services/foreman	Closed	Leos Stejskal	Actions
Refactor #34165: Require code from lib folder	Closed	Leos Stejskal	Actions
Refactor #34646: Move initialization from application.rb into initializer	Closed	Ondřej Ezr	Actions
Bug #35419: Remove empty webpack_assets.rb file	Closed	Ewoud Kohl van Wijngaarden	Actions

Related issues 5 (1 open — 4 closed)

Related to Foreman - Tracker #24837: Rails 6.0 Tracker

Closed

Actions

Related to Foreman - Tracker #28570: Rails 6.1 Tracker

Closed

Actions

Related to foreman-tasks - Bug #34917: Tasks are loading ApplicationRecord too soon and it derails Rails

Closed

Ondřej Ezr

Actions

Related to Foreman - Tracker #37480: Support Zeitwerk loader in plugins

Closed

Oleh Fedorenko

Actions

Blocks Foreman - Tracker #34647: Rails 7.0 Tracker

New

Ewoud Kohl van Wijngaarden

Actions

Issue # Delay: days Cancel Multiple values allowed (comma separated).

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Lukas Zapletal about 6 years ago

Related to Tracker #24837: Rails 6.0 Tracker added

Actions

Copy link

Updated by Lukas Zapletal about 6 years ago

Related to Tracker #28570: Rails 6.1 Tracker added

Actions

Copy link

Updated by Tomer Brisker over 4 years ago

https://edgeguides.rubyonrails.org/classic_to_zeitwerk_howto.html

Actions

Copy link

Updated by Ondřej Ezr over 4 years ago

Tracker changed from Bug to Feature

Actions

Copy link

Updated by Ondřej Ezr about 4 years ago

Blocks Tracker #34647: Rails 7.0 Tracker added

Actions

Copy link

Updated by Ondřej Ezr about 4 years ago

Related to Bug #34917: Tasks are loading ApplicationRecord too soon and it derails Rails added

Actions

Copy link

Updated by Oleh Fedorenko about 2 years ago

Status changed from New to Assigned
Assignee set to Oleh Fedorenko

Actions

Copy link

Updated by The Foreman Bot about 2 years ago

Status changed from Assigned to Ready For Testing
Pull request https://github.com/theforeman/foreman/pull/10131 added

Actions

Copy link

Updated by Oleh Fedorenko almost 2 years ago

Related to Tracker #37480: Support Zeitwerk loader in plugins added

Actions

Copy link

#10

Updated by The Foreman Bot almost 2 years ago

Pull request https://github.com/theforeman/foreman/pull/9328 added

Actions

Copy link

#11

Updated by The Foreman Bot almost 2 years ago

Pull request https://github.com/theforeman/foreman/pull/10299 added

Actions

Copy link

#12

Updated by The Foreman Bot almost 2 years ago

Fixed in Releases 3.13.0 added

Actions

Copy link

#13

Updated by Oleh Fedorenko almost 2 years ago

Status changed from Ready For Testing to Closed

Applied in changeset foreman|ce33bec59deb69c4ce4c864caa2669f88044ad9f.

Actions

Copy link

#14

Updated by The Foreman Bot over 1 year ago

Pull request deleted (~~https://github.com/theforeman/foreman/pull/10131, https://github.com/theforeman/foreman/pull/9328, https://github.com/theforeman/foreman/pull/10299~~)

Actions

Copy link

#15