There are several configuration issues in CTDB in 4.9.0rc3: * "no realtime" is a negative option Negative options can be confusing. I meant to change this to a positive option months ago, but forgot. * TDBMutexEnabled tunable is commonly used, so should be a config option Tunables are being removed, so this should be a configuration option to minimise pain for migration to the next version. * DisableIPFailover tunable is commonly used, so should be a config option Tunables are being removed, so this should be a configuration option to minimise pain for migration to the next version. * Boolean config values can only have true/false values This is inconsistent with the previous use of using yes/no values. Both should be allows. * Sample config migration script does not correct recognise CTDB_BASE as removed * There was no test to confirm that an invalid key does not stop the rest of the configuration from loading
Created attachment 14447 [details] Patch for 4.9 Patch includes upstream changes plus WHATSNEW.txt update. I thought I'd make this a little more complete than it was... plus the latest changes.
Created attachment 14448 [details] Patches for 4.9.x Added Reviewed-by tag for WHATSNEW patch. All other patches RB+.
Hi Karolin, This is ready for 4.9. Thanks...
(In reply to Martin Schwenke from comment #3) Pushed to autobuild-v4-9-test.
A few more patches coming via master to improve logging when loading a conf file with invalid options.
Created attachment 14463 [details] Additional patches for 4.9 These patches are in additional to those already applied to v4-9-test.
Hi Karolin, The extra patches are ready for v4-9. Thanks.
(In reply to Amitay Isaacs from comment #7) Pushed additional patches to v4-9-test.
Pushed to v4-9-test. Closing out bug report. Thanks!