Skip to content

Commit

Permalink
BDB tuning should be applied only when BDB backend is used
Browse files Browse the repository at this point in the history
389DS supports BDB and LMDB backends. FreeIPA installation fails with
LMDB backend since it tries to apply tuning for BDB backend.

Instead, tuning for BDB should be applied only when 389DS uses BDB
backend.

Fixes: https://pagure.io/freeipa/issue/9435

Signed-off-by: Viktor Ashirov <vashirov@redhat.com>
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
  • Loading branch information
vashirov authored and rcritten committed Sep 1, 2023
1 parent 45fa435 commit 953c7cc
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion ipaserver/install/dsinstance.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
from lib389.instance.options import General2Base, Slapd2Base
from lib389.instance.remove import remove_ds_instance as lib389_remove_ds
from lib389.instance.setup import SetupDs
from lib389.utils import get_default_db_lib

from ipalib import x509
from ipalib.install import certmonger, certstore
Expand Down Expand Up @@ -227,7 +228,8 @@ def __init__(self, realm_name=None, domain_name=None, fstore=None,
def __common_setup(self):

self.step("creating directory server instance", self.__create_instance)
self.step("tune ldbm plugin", self.__tune_ldbm)
if get_default_db_lib() == 'bdb':
self.step("tune ldbm plugin", self.__tune_ldbm)
if self.config_ldif is not None:
self.step("stopping directory server", self.__stop_instance)
self.step(
Expand Down

0 comments on commit 953c7cc

Please sign in to comment.