Project

General

Profile

Actions
Copy link

Bug #43746

closed

mgr default value handling is broken

Added by Sage Weil over 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Sage Weil
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
nautilus, octopus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
32755
Crash signature (v1):
Crash signature (v2):

Description

most mgr modules have a config_notify() that does

             setattr(self,
                     opt['name'],  # type: ignore
                     self.get_module_option(opt['name']) or opt['default'])  # type: ignore

which is applied after the get_module_option in mgr_module.py already sorts out the default values. this breaks any values of 0 or false or none or ''.

Related issues 2 (0 open — 2 closed)

Copied to mgr - Backport #43820: nautilus: mgr default value handling is brokenResolvedNathan CutlerActions
Copied to mgr - Backport #44754: octopus: mgr default value handling is brokenResolvedNathan CutlerActions
Actions
Copy link

Also available in: Atom PDF