Created attachment 6720 [details] Patcheset for v3-6-test In the rewrite of idmapping, I missed a few minor things for idmap_tdb2: (1) There is an old, undocumented option with the silly name "tdb:idmap2.tdb" which I have never seen used and which is completely poinltess since ctdb supports persistent databases (i.e. since ages). (2) The "idmap:script" option does not match the rewritten systematic configuration. This should be "idmap config * : script" (or more general "idmap congfig DOMAIN : script"). Attached find a patchset for 3.6, already pushed to master, that fixes the two by (1) removing the "tdb:idmap2.tdb" parameter and by (2) deprecating "idmap:script" in favour of "idmap config * : script" keeping the old one as a fallback. Additionally, the patches fix a couple of legacy code comments and adapt the idmap_tdb2 manpage to be more correct and reflect the new configuration. Cheers - Michael
Comment on attachment 6720 [details] Patcheset for v3-6-test Patchset for 3.6. Ambi: please review...
==> karolin
IMHO it's way too late in the release process for cleanups, but taking it for 3.6.1 would not make sense. Pushed to v3-6-test and praying that we don't need rc4... Closing out bug report.