The branch, master has been updated via 0580a23 s4:torture: Improve error message in whoami test via 8d90f2a s3:test: Always validate the join after changing the secret via e131ce4 waf: Remove build system info (uname -a) via c69938e systemd: Fix kill path via a653b62 systemd: Add documentation to Unit files via df68af9 systemd: syslog.target is obsolete from 0abe16a torture: Add test for channel sequence number handling
https://git.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 0580a23d64be346e4526056c695249de4241cfa6 Author: Andreas Schneider <a...@samba.org> Date: Fri Jan 12 13:45:33 2018 +0100 s4:torture: Improve error message in whoami test Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Ralph Boehme <s...@samba.org> Autobuild-User(master): Andreas Schneider <a...@cryptomilk.org> Autobuild-Date(master): Sun Jan 14 22:09:24 CET 2018 on sn-devel-144 commit 8d90f2a03e58f34f36d345d31c5ca6090d37ff54 Author: Andreas Schneider <a...@samba.org> Date: Thu Jan 11 11:00:43 2018 +0100 s3:test: Always validate the join after changing the secret Signed-off-by: Andreas Schneider <a...@samba.org> Signed-off-by: Ralph Boehme <s...@samba.org> Pair-Programmed-With: Ralph Boehme <s...@samba.org> commit e131ce4ddc074dc146c81edb99a203d1dd2883b8 Author: Mathieu Parent <math.par...@gmail.com> Date: Thu Jan 11 21:18:46 2018 +0100 waf: Remove build system info (uname -a) Preventing reproducible builds while adding minor benefit. More information at <https://reproducible-builds.org/>. BUG: https://bugzilla.samba.org/show_bug.cgi?id=13213 Signed-off-by: Mathieu Parent <math.par...@gmail.com> Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Andrew Bartlet <abart...@samba.org> commit c69938ef5cb9ee49565fae834a25541f6a5da0a8 Author: Mathieu Parent <math.par...@gmail.com> Date: Thu May 12 22:16:24 2016 +0200 systemd: Fix kill path Bug-Debian: https://bugs.debian.org/828730 BUG: https://bugzilla.samba.org/show_bug.cgi?id=12402 Signed-off-by: Mathieu Parent <math.par...@gmail.com> Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Alexander Bokovoy <a...@samba.org> commit a653b62ad71b65c280ee4ba6bfbb560ac3b1f90e Author: Mathieu Parent <math.par...@gmail.com> Date: Thu May 12 22:16:24 2016 +0200 systemd: Add documentation to Unit files BUG: https://bugzilla.samba.org/show_bug.cgi?id=12402 Signed-off-by: Mathieu Parent <math.par...@gmail.com> Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Alexander Bokovoy <a...@samba.org> commit df68af9fa5900b5f7b22a972666fd213fef17106 Author: Mathieu Parent <math.par...@gmail.com> Date: Thu Jan 11 10:07:17 2018 +0100 systemd: syslog.target is obsolete After=syslog.target is unnecessary by now because syslog is socket-activated and will therefore be started when needed. Ref: https://lintian.debian.org/tags/systemd-service-file-refers-to-obsolete-target.html BUG: https://bugzilla.samba.org/show_bug.cgi?id=12402 Signed-off-by: Mathieu Parent <math.par...@gmail.com> Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Alexander Bokovoy <a...@samba.org> ----------------------------------------------------------------------- Summary of changes: buildtools/wafsamba/wscript | 4 ---- ctdb/config/ctdb.service | 1 + packaging/systemd/nmb.service.in | 5 +++-- packaging/systemd/samba.service.in | 5 +++-- packaging/systemd/smb.service.in | 5 +++-- packaging/systemd/winbind.service.in | 5 +++-- source3/script/tests/test_net_cred_change.sh | 7 ++++--- source4/smbd/server.c | 3 --- source4/torture/unix/whoami.c | 9 +++++++-- 9 files changed, 24 insertions(+), 20 deletions(-) Changeset truncated at 500 lines: diff --git a/buildtools/wafsamba/wscript b/buildtools/wafsamba/wscript index 430d164..3b36b57 100644 --- a/buildtools/wafsamba/wscript +++ b/buildtools/wafsamba/wscript @@ -313,10 +313,6 @@ def configure(conf): conf.env.GIT_LOCAL_CHANGES = Options.options.GIT_LOCAL_CHANGES - conf.CHECK_COMMAND(['uname', '-a'], - msg='Checking build system', - define='BUILD_SYSTEM', - on_target=False) conf.CHECK_UNAME() # see if we can compile and run a simple C program diff --git a/ctdb/config/ctdb.service b/ctdb/config/ctdb.service index 63cdfa9..189f2f4 100644 --- a/ctdb/config/ctdb.service +++ b/ctdb/config/ctdb.service @@ -1,5 +1,6 @@ [Unit] Description=CTDB +Documentation=man:ctdbd(1) man:ctdb(7) After=network-online.target time-sync.target [Service] diff --git a/packaging/systemd/nmb.service.in b/packaging/systemd/nmb.service.in index b0ba92f..a308123 100644 --- a/packaging/systemd/nmb.service.in +++ b/packaging/systemd/nmb.service.in @@ -1,7 +1,8 @@ [Unit] Description=Samba NMB Daemon +Documentation=man:nmbd(8) man:samba(7) man:smb.conf(5) Wants=network-online.target -After=syslog.target network.target network-online.target +After=network.target network-online.target [Service] Type=notify @@ -9,7 +10,7 @@ NotifyAccess=all PIDFile=@PIDDIR@/nmbd.pid EnvironmentFile=-@SYSCONFDIR@/sysconfig/samba ExecStart=@SBINDIR@/nmbd --foreground --no-process-group $NMBDOPTIONS -ExecReload=/usr/bin/kill -HUP $MAINPID +ExecReload=/bin/kill -HUP $MAINPID LimitCORE=infinity @systemd_nmb_extra@ diff --git a/packaging/systemd/samba.service.in b/packaging/systemd/samba.service.in index c418150..f8ea527 100644 --- a/packaging/systemd/samba.service.in +++ b/packaging/systemd/samba.service.in @@ -1,7 +1,8 @@ [Unit] Description=Samba AD Daemon +Documentation=man:samba(8) man:samba(7) man:smb.conf(5) Wants=network-online.target -After=syslog.target network.target network-online.target +After=network.target network-online.target [Service] Type=notify @@ -10,7 +11,7 @@ PIDFile=@PIDDIR@/samba.pid LimitNOFILE=16384 EnvironmentFile=-@SYSCONFDIR@/sysconfig/samba ExecStart=@SBINDIR@/samba --foreground --no-process-group $SAMBAOPTIONS -ExecReload=/usr/bin/kill -HUP $MAINPID +ExecReload=/bin/kill -HUP $MAINPID @systemd_samba_extra@ [Install] diff --git a/packaging/systemd/smb.service.in b/packaging/systemd/smb.service.in index f829bcb..dcaf7ef 100644 --- a/packaging/systemd/smb.service.in +++ b/packaging/systemd/smb.service.in @@ -1,6 +1,7 @@ [Unit] Description=Samba SMB Daemon -After=syslog.target network.target nmb.service winbind.service +Documentation=man:smbd(8) man:samba(7) man:smb.conf(5) +After=network.target nmb.service winbind.service [Service] Type=notify @@ -9,7 +10,7 @@ PIDFile=@PIDDIR@/smbd.pid LimitNOFILE=16384 EnvironmentFile=-@SYSCONFDIR@/sysconfig/samba ExecStart=@SBINDIR@/smbd --foreground --no-process-group $SMBDOPTIONS -ExecReload=/usr/bin/kill -HUP $MAINPID +ExecReload=/bin/kill -HUP $MAINPID LimitCORE=infinity @systemd_smb_extra@ diff --git a/packaging/systemd/winbind.service.in b/packaging/systemd/winbind.service.in index 5ac5adc..c28d94c 100644 --- a/packaging/systemd/winbind.service.in +++ b/packaging/systemd/winbind.service.in @@ -1,6 +1,7 @@ [Unit] Description=Samba Winbind Daemon -After=syslog.target network.target nmb.service +Documentation=man:winbindd(8) man:samba(7) man:smb.conf(5) +After=network.target nmb.service [Service] Type=notify @@ -8,7 +9,7 @@ NotifyAccess=all PIDFile=@PIDDIR@/winbindd.pid EnvironmentFile=-@SYSCONFDIR@/sysconfig/samba ExecStart=@SBINDIR@/winbindd --foreground --no-process-group "$WINBINDOPTIONS" -ExecReload=/usr/bin/kill -HUP $MAINPID +ExecReload=/bin/kill -HUP $MAINPID LimitCORE=infinity @systemd_winbind_extra@ diff --git a/source3/script/tests/test_net_cred_change.sh b/source3/script/tests/test_net_cred_change.sh index de56be5..794b054 100755 --- a/source3/script/tests/test_net_cred_change.sh +++ b/source3/script/tests/test_net_cred_change.sh @@ -9,8 +9,9 @@ fi incdir=`dirname $0`/../../../testprogs/blackbox . $incdir/subunit.sh -testit "first change" $VALGRIND $BINDIR/wbinfo -c || failed=`expr $failed + 1` -testit "first join" $VALGRIND $BINDIR/net rpc testjoin $@ || failed=`expr $failed + 1` -testit "second change" $VALGRIND $BINDIR/wbinfo -c || failed=`expr $failed + 1` +testit "1: change machine secret" $VALGRIND $BINDIR/wbinfo --change-secret || failed=`expr $failed + 1` +testit "1: validate secret" $VALGRIND $BINDIR/net rpc testjoin $@ || failed=`expr $failed + 1` +testit "2: change machine secret" $VALGRIND $BINDIR/wbinfo --change-secret || failed=`expr $failed + 1` +testit "2: validate secret" $VALGRIND $BINDIR/net rpc testjoin $@ || failed=`expr $failed + 1` testok $0 $failed diff --git a/source4/smbd/server.c b/source4/smbd/server.c index adee388..431fbad 100644 --- a/source4/smbd/server.c +++ b/source4/smbd/server.c @@ -323,9 +323,6 @@ static void show_build(void) printf("Samba version: %s\n", SAMBA_VERSION_STRING); printf("Build environment:\n"); -#ifdef BUILD_SYSTEM - printf(" Build host: %s\n", BUILD_SYSTEM); -#endif printf("Paths:\n"); for (i=0; config_options[i].name; i++) { diff --git a/source4/torture/unix/whoami.c b/source4/torture/unix/whoami.c index bb70e47..efd9efa 100644 --- a/source4/torture/unix/whoami.c +++ b/source4/torture/unix/whoami.c @@ -91,8 +91,13 @@ static struct smbcli_state *connect_to_server(struct torture_context *tctx, lpcfg_gensec_settings(tctx, tctx->lp_ctx)); if (!NT_STATUS_IS_OK(status)) { - printf("failed to connect to //%s/%s: %s\n", - host, share, nt_errstr(status)); + torture_comment(tctx, + "FATAL: Failed to connect to //%s/%s " + "with %s - %s\n", + host, + share, + cli_credentials_get_username(creds), + nt_errstr(status)); return NULL; } -- Samba Shared Repository