Hi Klaus, I doublechecked this and indeed, you were correct about the config being updated properly without that.
Thanks for taking care of this. Cheers, Helga On Mon, 20 Apr 2015 at 17:48 Helga Velroyen <[email protected]> wrote: > LGTM > > On Mon, Apr 20, 2015, 5:03 PM 'Klaus Aehlig' via ganeti-devel < > [email protected]> wrote: > >> ...as it was accidentally readded in the merge. >> >> Signed-off-by: Klaus Aehlig <[email protected]> >> --- >> lib/cmdlib/cluster.py | 4 ---- >> 1 file changed, 4 deletions(-) >> >> diff --git a/lib/cmdlib/cluster.py b/lib/cmdlib/cluster.py >> index f22c810..ac3fa6d 100644 >> --- a/lib/cmdlib/cluster.py >> +++ b/lib/cmdlib/cluster.py >> @@ -220,10 +220,6 @@ class LUClusterRenewCrypto(NoHooksLU): >> " master candidate cert list. Current state of the" >> " list: %s.", cluster.candidate_certs) >> >> - # Trigger another update of the config now with the new master cert >> - logging.debug("Trigger an update of the configuration on all nodes.") >> - self.cfg.Update(cluster, feedback_fn) >> - >> >> class LUClusterActivateMasterIp(NoHooksLU): >> """Activate the master IP on the master node. >> -- >> 2.2.0.rc0.207.ga3a616c >> >>
