From 2d17f19ae2c50e4fc6665a58bb586679d1875dca Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Thu, 14 Dec 2017 11:41:41 +0530 Subject: [PATCH] users: Add a note about using separate first setup action Instead of a separate action that is conditionally called for a version number, we can check if the domain currently configured is what we want and then based on the value do a reconfiguration. This approach will work better when Plinth state is reset etc. Signed-off-by: Sunil Mohan Adapa --- actions/users | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/actions/users b/actions/users index d1c9532cc..9e778d327 100755 --- a/actions/users +++ b/actions/users @@ -95,6 +95,10 @@ def subcommand_first_setup(_): """Perform initial setup of LDAP.""" # Avoid reconfiguration of slapd during module upgrades, because # this will move the existing database. + # XXX: Instead of a separate action that is conditionally called for a + # version number, we can check if the domain currently configured is what + # we want and then based on the value do a reconfiguration. This approach + # will work better when Plinth state is reset etc. action_utils.dpkg_reconfigure('slapd', {'domain': 'thisbox'})