Project

General

Profile

Bug #32963

Load only core setting values the first time we load them

Added by Ondřej Ezr 3 months ago. Updated 29 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Settings
Target version:
-

Description

We need to load setting values early, to get the main core settings in init proces, but it is failing if we try to load values of setting STI classes, that have not been loaded yet, as that fails to find STI class.


Related issues

Related to Foreman - Feature #30862: Introduce SettingRegistry as a setting inventoryClosed

Associated revisions

Revision ff98d230 (diff)
Added by Ondřej Ezr 3 months ago

Fixes #32963 - Load only present categories

We need to load values only for categories that has already been loaded
into memory. This is issue in development.

Revision 35fb314e (diff)
Added by Ondřej Ezr about 2 months ago

Fixes #32963 - Load only present categories

This got accidently reverted by bad rebase in
3ff1fd27fa32f7a52ecd995417521b5b91c9a093.
I'm reapplying this.

History

#1 Updated by Ondřej Ezr 3 months ago

  • Related to Feature #30862: Introduce SettingRegistry as a setting inventory added

#2 Updated by The Foreman Bot 3 months ago

  • Assignee set to Ondřej Ezr
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/8651 added

#3 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases 3.0.0 added

#4 Updated by Ondřej Ezr 3 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by The Foreman Bot about 2 months ago

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

#6 Updated by The Foreman Bot about 2 months ago

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

#7 Updated by Amit Upadhye 29 days ago

  • Fixed in Releases 2.5.3 added

Also available in: Atom PDF