Project

General

Profile

Bug #8571

loader.conf/.local cleanup is a bit too aggressive

Added by Jim Pingle about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Operating System
Target version:
Start date:
06/13/2018
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.4.x
Affected Architecture:
All

Description

We have code that cleans up and eliminates duplicate settings in loader.conf and loader.conf.local to avoid foot-shooting and to make sure that hardware works properly.

The code that checks/matches these values is a bit too aggressive and over-matches, and removes some values from loader.conf.local that it makes sense to allow a user to override.

The matching could be improved, and it could handle the local case slightly differently so that some values can be overridden by users.

Associated revisions

Revision 7f943a22 (diff)
Added by Jim Pingle about 1 year ago

Rework loader.conf(.local) filtering. Fixes #8571

If this isn't aggressive enough, we could remove the "local" changes and only
keep the new matching method.

Revision a1b69b57 (diff)
Added by Jim Pingle about 1 year ago

Rework loader.conf(.local) filtering. Fixes #8571

If this isn't aggressive enough, we could remove the "local" changes and only
keep the new matching method.

(cherry picked from commit 7f943a2269dea1efd9bf42320d14ae7e0ca4a4f7)

History

#1 Updated by Jim Pingle about 1 year ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100

#2 Updated by James Dekker about 1 year ago

On 2.4.4.a.20180705.0032 (SG-2440) added autoboot_delay=10 to /boot/loader.conf.local (which didn't exist before that), rebooted a couple of times with option 5 at console and the file remained with the contents unchanged. For the loader.conf part, saved settings on the System > Advanced page and after that the contents of /boot/loader.conf remained unchanged through a couple of reboots.

#3 Updated by Jim Pingle about 1 year ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF