The branch, master has been updated via da9472a934b prefork tests: re-enable restart tests for MIT Kerberos via 1c4b9ba36c7 prefork restart tests: Use echo server for back off via 15196c7561f s4 echo_server: Enable prefork process model from 5522aa1a4c3 vfs:glusterfs_fuse: ensure fileids are constant across nodes
https://git.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit da9472a934bf242c7dec1a7a355fb4eb043e5b15 Author: Gary Lockyer <g...@catalyst.net.nz> Date: Thu Jul 11 09:31:31 2019 +1200 prefork tests: re-enable restart tests for MIT Kerberos Previously the restart back off tests were disabled for MIT Kerberos, as they used the kdc, which is not run in that case. Tests now use the echo server. Signed-off-by: Gary Lockyer <g...@catalyst.net.nz> Reviewed-by: Andreas Schneider <a...@samba.org> Autobuild-User(master): Andreas Schneider <a...@cryptomilk.org> Autobuild-Date(master): Tue Jul 16 10:10:23 UTC 2019 on sn-devel-184 commit 1c4b9ba36c724b6fcc26a51af5202483ab3f1c23 Author: Gary Lockyer <g...@catalyst.net.nz> Date: Thu Jul 11 09:28:30 2019 +1200 prefork restart tests: Use echo server for back off Use the echo server for the restart back off tests in the samba.tests.prefork_restart tests instead of the kdc. The kdc is not enabled when the ADDC is built to run MIT Kerberos. Changing the test to use the echo server means it can be run when MIT Kerberos is enabled. Signed-off-by: Gary Lockyer <g...@catalyst.net.nz> Reviewed-by: Andreas Schneider <a...@samba.org> commit 15196c7561ff3afdb3d61afd6951d22e4cf698c7 Author: Gary Lockyer <g...@catalyst.net.nz> Date: Thu Jul 11 09:29:28 2019 +1200 s4 echo_server: Enable prefork process model Enable the prefork process model, for the echo server. This is allows it to be used in the prefork_restart tests. Signed-off-by: Gary Lockyer <g...@catalyst.net.nz> Reviewed-by: Andreas Schneider <a...@samba.org> ----------------------------------------------------------------------- Summary of changes: python/samba/tests/prefork_restart.py | 14 +++++++------- selftest/skip_mit_kdc | 4 ---- source4/echo_server/echo_server.c | 2 +- 3 files changed, 8 insertions(+), 12 deletions(-) Changeset truncated at 500 lines: diff --git a/python/samba/tests/prefork_restart.py b/python/samba/tests/prefork_restart.py index 5233ca1f805..9ab97d062b4 100644 --- a/python/samba/tests/prefork_restart.py +++ b/python/samba/tests/prefork_restart.py @@ -404,7 +404,7 @@ class PreforkProcessRestartTests(TestCase): def test_master_restart_backoff(self): # get kdc master process - pid = self.get_process("prefork-master-kdc") + pid = self.get_process("prefork-master-echo") self.assertIsNotNone(pid) # @@ -417,13 +417,13 @@ class PreforkProcessRestartTests(TestCase): # Get the worker processes workers = self.get_worker_pids("kdc", NUM_WORKERS) - process = self.get_process("prefork-master-kdc") + process = self.get_process("prefork-master-echo") os.kill(process, signal.SIGTERM) # wait for the process to restart start = time.time() - self.wait_for_process("prefork-master-kdc", process, 0, 1, 30) + self.wait_for_process("prefork-master-echo", process, 0, 1, 30) # wait for the workers to restart as well - self.wait_for_workers("kdc", workers) + self.wait_for_workers("echo", workers) end = time.time() duration = end - start @@ -434,7 +434,7 @@ class PreforkProcessRestartTests(TestCase): self.assertGreaterEqual(duration, expected) # check that the worker processes have restarted - new_workers = self.get_worker_pids("kdc", NUM_WORKERS) + new_workers = self.get_worker_pids("echo", NUM_WORKERS) for x in range(NUM_WORKERS): self.assertNotEquals(workers[x], new_workers[x]) @@ -449,12 +449,12 @@ class PreforkProcessRestartTests(TestCase): # prefork maximum backoff = 10 backoff_increment = 5 for expected in [0, 5, 10, 10]: - process = self.get_process("prefork-worker-kdc-2") + process = self.get_process("prefork-worker-echo-2") self.assertIsNotNone(process) os.kill(process, signal.SIGTERM) # wait for the process to restart start = time.time() - self.wait_for_process("prefork-worker-kdc-2", process, 0, 1, 30) + self.wait_for_process("prefork-worker-echo-2", process, 0, 1, 30) end = time.time() duration = end - start diff --git a/selftest/skip_mit_kdc b/selftest/skip_mit_kdc index bef52f50f86..4a51c98ea0b 100644 --- a/selftest/skip_mit_kdc +++ b/selftest/skip_mit_kdc @@ -3,7 +3,3 @@ .*RODC ^samba4.ntvfs.cifs.ntlm.base.unlink ^samba4.ntvfs.cifs.krb5.base.unlink -# -# We skip the pre-fork restart tests as they use the kdc process to test -# restart back off -^samba.tests.prefork_restart diff --git a/source4/echo_server/echo_server.c b/source4/echo_server/echo_server.c index f38999ae139..c81fc140a0a 100644 --- a/source4/echo_server/echo_server.c +++ b/source4/echo_server/echo_server.c @@ -327,7 +327,7 @@ NTSTATUS server_service_echo_init(TALLOC_CTX *ctx) { static const struct service_details details = { .inhibit_fork_on_accept = true, - .inhibit_pre_fork = true, + .inhibit_pre_fork = false, .task_init = echo_task_init, .post_fork = NULL -- Samba Shared Repository