-P looks at the wrong secrets.tdb when running clustered. cli_credentials_set_machine_account(), which is called from popt_common_credentials_callback(), only uses dbwrap_local_open(). For the source3/ utilities it must use db_open() that also looks at ctdb-based databases.
Created attachment 17000 [details] Patch This fixes the problem for me. Not sure we want this in master, to me the solution looks really ugly, but giving time constraints I don't have a better solution
Thanks for looking into this! It won't win us a price, but that seems to be the best thing we can do atm. :) Looks good to me, give me a test and I'd push the same second. :) Andreas, maybe you want to take a look as well?
(In reply to Ralph Böhme from comment #2) > give me a test and I'd push the same second. :) Correct me if I'm wrong but it *has* a test, doesn't it ? :-).
Swooooooossssshhhhh..... the sound of a patchset being pushed to autobuild.
This bug was referenced in samba master: 9faa3173193ddcb95905993d960cc10d4366524e d6270525699fbc856b217cf18ece7f1d063b144d 63c80f25da8829a7bd3244afea29c13f699efac1 cdc0268c1987f36ab400ea01df88d55c02dccfdb
Created attachment 17001 [details] Patch for 4.15 with cherry-pick info
Comment on attachment 17000 [details] Patch 7620e471b51
Reassigning to Jule for inclusion in 4.15.
Looks fine for me. Thanks for fixing!
Pushed to autobuild-v4-15-test.
This bug was referenced in samba v4-15-test: 3309ab5fa0245ca791e66c585c131abb82d24a8b 575e620ad6cf57634c3357f8677f8beaf7c12caa f1c064e792ae1b9d3cc57380205246dc21e8bb3c bf9fdf5b45531749d5e533fe91dd957d64f81d6b
Closing out bug report. Thanks!
This bug was referenced in samba v4-15-stable (Release samba-4.15.3): 3309ab5fa0245ca791e66c585c131abb82d24a8b 575e620ad6cf57634c3357f8677f8beaf7c12caa f1c064e792ae1b9d3cc57380205246dc21e8bb3c bf9fdf5b45531749d5e533fe91dd957d64f81d6b