samba.git
7 years agoWHATSNEW: document new samba-tool drs replicate options
Stefan Metzmacher [Thu, 28 Jul 2016 06:40:51 +0000 (08:40 +0200)]
WHATSNEW: document new samba-tool drs replicate options

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@samba.org>
7 years agotests:samba_tool_drs: test samba-tool drs replicate with --async-op
Stefan Metzmacher [Thu, 28 Jul 2016 06:29:39 +0000 (08:29 +0200)]
tests:samba_tool_drs: test samba-tool drs replicate with --async-op

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@samba.org>
7 years agotests:samba_tool_drs: test samba-tool drs replicate with implicit machine credentials
Stefan Metzmacher [Thu, 28 Jul 2016 06:29:39 +0000 (08:29 +0200)]
tests:samba_tool_drs: test samba-tool drs replicate with implicit machine credentials

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@samba.org>
7 years agosamba-tool: add --async-rep option to 'samba-tool drs replicate'
Stefan Metzmacher [Thu, 28 Jul 2016 05:50:03 +0000 (07:50 +0200)]
samba-tool: add --async-rep option to 'samba-tool drs replicate'

We may not want to wait for the replication to finish.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@samba.org>
7 years agosamba-tool: use a timeout of 5 minutes in 'samba-tool drs replicate'
Stefan Metzmacher [Thu, 28 Jul 2016 05:48:44 +0000 (07:48 +0200)]
samba-tool: use a timeout of 5 minutes in 'samba-tool drs replicate'

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@samba.org>
7 years agos4:pyrpc: correctly implement .request_timeout
Stefan Metzmacher [Thu, 28 Jul 2016 04:36:05 +0000 (06:36 +0200)]
s4:pyrpc: correctly implement .request_timeout

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Garming Sam <garming@samba.org>
7 years agorpc_server/drsuapi: Don't set msDS_IntId as attid for linked attributes if schema
Garming Sam [Wed, 27 Jul 2016 02:22:39 +0000 (14:22 +1200)]
rpc_server/drsuapi: Don't set msDS_IntId as attid for linked attributes if schema

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agogetncchanges: Set is_schema_nc when EXOP_OBJ
Garming Sam [Wed, 27 Jul 2016 03:51:47 +0000 (15:51 +1200)]
getncchanges: Set is_schema_nc when EXOP_OBJ

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agomsds_intid: Add test for schema linked attributes
Garming Sam [Tue, 26 Jul 2016 04:57:24 +0000 (16:57 +1200)]
msds_intid: Add test for schema linked attributes

This test only covers the forward link case.

NOTE: We can't confirm this against Windows because they prevent us from
modifying the schema for the schema classes.

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agoreplmd: Send replicated update OID for forward links
Garming Sam [Wed, 27 Jul 2016 01:06:09 +0000 (13:06 +1200)]
replmd: Send replicated update OID for forward links

(The backward link case needs to be tested)

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agoreplmd: Remove data field on DSDB_CONTROL_REPLICATED_UPDATE_OID
Garming Sam [Wed, 27 Jul 2016 04:09:22 +0000 (16:09 +1200)]
replmd: Remove data field on DSDB_CONTROL_REPLICATED_UPDATE_OID

There were no users of the data, and it added additional complexity

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agorpc_server/drsuapi: Set msDS_IntId as attid for linked attributes if exists
Evgeny Sinelnikov [Tue, 29 Mar 2016 15:37:40 +0000 (18:37 +0300)]
rpc_server/drsuapi: Set msDS_IntId as attid for linked attributes if exists

We got WERR_DS_DRA_SCHEMA_MISMATCH for linked attributes with 8418 error for
extended attributes when using same attid as attribute object.

Signed-off-by: Evgeny Sinelnikov <sin@altlinux.ru>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agomsds_intid: Add test for (non-schema) linked attributes
Garming Sam [Tue, 26 Jul 2016 04:12:05 +0000 (16:12 +1200)]
msds_intid: Add test for (non-schema) linked attributes

Prior to this, none of the linked attributes would be checked for their
ids.

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agovalgrind: Avoid a warning about uninitialized memory
Garming Sam [Mon, 25 Jul 2016 23:54:07 +0000 (11:54 +1200)]
valgrind: Avoid a warning about uninitialized memory

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agoreplmd: Check dsdb_dn for syntax errors
Garming Sam [Tue, 26 Jul 2016 21:50:33 +0000 (09:50 +1200)]
replmd: Check dsdb_dn for syntax errors

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agotevent: version 0.9.28 tevent-0.9.29
Stefan Metzmacher [Tue, 26 Jul 2016 18:25:53 +0000 (20:25 +0200)]
tevent: version 0.9.28

* performance improvements
* documentation updates
* build fixes

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Thu Jul 28 08:45:34 CEST 2016 on sn-devel-144

7 years agotalloc: version 2.1.8 talloc-2.1.8
Stefan Metzmacher [Tue, 26 Jul 2016 18:23:27 +0000 (20:23 +0200)]
talloc: version 2.1.8

* performance improvements
* build fixes
* documentation updates

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agotdb: version 1.3.10 tdb-1.3.10
Stefan Metzmacher [Tue, 26 Jul 2016 18:21:21 +0000 (20:21 +0200)]
tdb: version 1.3.10

* performance improvements
* make pthread mutex code more portable
* builf fixes

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agofss_agent: Fix a signed/unsigned mixup
Volker Lendecke [Fri, 22 Jul 2016 13:43:58 +0000 (15:43 +0200)]
fss_agent: Fix a signed/unsigned mixup

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
7 years agolib: Move "message_send_all" to serverid.c
Volker Lendecke [Thu, 14 Jul 2016 10:47:16 +0000 (12:47 +0200)]
lib: Move "message_send_all" to serverid.c

Trying to trim down messages.c a bit: Sending to all processes that are
registered in serverid.tdb and filtering to me is not really logic of general
messaging but more of the serverid code.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>
7 years agos3/notifyd: add async send/recv functions
Ralph Boehme [Fri, 22 Jul 2016 06:29:13 +0000 (08:29 +0200)]
s3/notifyd: add async send/recv functions

Previously, without this patch, if notifyd died for whatever reason, it
would be restarted from smbd. However, if its initialization failed and
it exitted again, there would be no child entry in smbd for it and it
wouldn't be attempted to restart it again.

This patch adds async send/recv methods for starting cleanupd that will
reschedule restart attempt every second in case initilisation failed.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12022

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
7 years agos3/cleanupd: use smbd_cleanupd.tdb
Ralph Boehme [Thu, 21 Jul 2016 17:08:47 +0000 (19:08 +0200)]
s3/cleanupd: use smbd_cleanupd.tdb

Instead of using messaging to send individual cleanup events, it works
this way:

o parent smbd stores cleanup events (ie exitted children) in
  smbd_cleanup.tdb

o it sends cleanupd an empty MSG_SMB_NOTIFY_CLEANUP message

o cleanupd does a traverse on the smbd_cleanupd.tdb and collects all
  childs in a list

o after the traverse cleanupd walks the list and does the real work

It would have been possible to optimize for the common case by passing
info about exitted childs with the message (as was done before this
patch), adding a new message type for triggering a db traverse that
would be used when cleanupd had to be restarted and cleanup events may
have been accumulated in cleanup.tdb.

But this could be subject to subtle race conditions and could loose
events if cleanupd dies randomly.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12022

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
7 years agos3/smbd: add cleanupd_init_send()/recv()
Ralph Boehme [Thu, 14 Jul 2016 14:31:44 +0000 (16:31 +0200)]
s3/smbd: add cleanupd_init_send()/recv()

Previously, without this patch, if cleanupd died for whatever reason, it
would be restarted from smbd. However, if cleanupd initialization
failed and it exitted again, there would be no child entry in smbd for
it and it wouldn't be attempted to restart it again.

This patch adds async send/recv methods for starting cleanupd that will
reschedule restart attempt every second in case initilisation failed.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12022

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
7 years agos3/lib: add smbd_cleanupd.tdb
Ralph Boehme [Thu, 21 Jul 2016 14:53:15 +0000 (16:53 +0200)]
s3/lib: add smbd_cleanupd.tdb

This will be used between cleanupd and smbd for passing information
about exitted smbd childs.

Bug: https://bugzilla.samba.org/show_bug.cgi?id=12022

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
7 years agoctdb-daemon: Fix CID 1364527/8/9: Null pointer dereferences (NULL_RETURNS)
Martin Schwenke [Wed, 27 Jul 2016 01:45:49 +0000 (11:45 +1000)]
ctdb-daemon: Fix CID 1364527/8/9: Null pointer dereferences (NULL_RETURNS)

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Volker Lendecke <vl@samba.org>
7 years agoctdb-tests: Pretend not to ignore return from fgets()
Martin Schwenke [Wed, 27 Jul 2016 05:44:20 +0000 (15:44 +1000)]
ctdb-tests: Pretend not to ignore return from fgets()

With some compilers and picky compiler settings, ignoring the return
value of fgets() can cause compilation to fail.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Avoid portability issue in porting tests
Martin Schwenke [Wed, 27 Jul 2016 05:24:58 +0000 (15:24 +1000)]
ctdb-tests: Avoid portability issue in porting tests

EUID is bash-only, so this condition always fails on other shells.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Remove duplicate EOF terminators in some tool unit tests
Martin Schwenke [Wed, 27 Jul 2016 04:41:39 +0000 (14:41 +1000)]
ctdb-tests: Remove duplicate EOF terminators in some tool unit tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Avoid division by zero in NFS eventscript unit test
Martin Schwenke [Wed, 27 Jul 2016 04:40:52 +0000 (14:40 +1000)]
ctdb-tests: Avoid division by zero in NFS eventscript unit test

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Removing sleep from porting_tests
Amitay Isaacs [Wed, 27 Jul 2016 05:21:39 +0000 (15:21 +1000)]
ctdb-tests: Removing sleep from porting_tests

This avoids test failures due to timing issues.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Do not add $VALGRIND to ctdb command
Amitay Isaacs [Wed, 27 Jul 2016 04:59:02 +0000 (14:59 +1000)]
ctdb-tests: Do not add $VALGRIND to ctdb command

unit_test wrapper already adds VALGRIND wrapper.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix valgrind unintialized error
Amitay Isaacs [Wed, 27 Jul 2016 04:57:58 +0000 (14:57 +1000)]
ctdb-tests: Fix valgrind unintialized error

struct ctdb_dbid has padding and needs to be initialized.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoWHATSNEW: ctdb updates
Amitay Isaacs [Tue, 26 Jul 2016 05:38:44 +0000 (15:38 +1000)]
WHATSNEW: ctdb updates

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-packaging: Move ctdb configuration to ctdbd.conf
Amitay Isaacs [Tue, 28 Jun 2016 06:11:47 +0000 (16:11 +1000)]
ctdb-packaging: Move ctdb configuration to ctdbd.conf

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-protocol: Deprecate global transaction controls
Amitay Isaacs [Mon, 25 Jul 2016 05:55:12 +0000 (15:55 +1000)]
ctdb-protocol: Deprecate global transaction controls

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-protocol: Drop marshalling for global transaction controls
Amitay Isaacs [Mon, 25 Jul 2016 05:54:16 +0000 (15:54 +1000)]
ctdb-protocol: Drop marshalling for global transaction controls

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-client: Drop client code for global transaction controls
Amitay Isaacs [Mon, 25 Jul 2016 05:49:20 +0000 (15:49 +1000)]
ctdb-client: Drop client code for global transaction controls

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Drop implementation of global transaction controls
Amitay Isaacs [Mon, 25 Jul 2016 05:41:34 +0000 (15:41 +1000)]
ctdb-daemon: Drop implementation of global transaction controls

These were used in serial recovery and for restoring databases using
older ctdb tool.  New code uses database specific transaction controls.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-protocol: Deprecate THAW control
Amitay Isaacs [Thu, 21 Jul 2016 04:22:39 +0000 (14:22 +1000)]
ctdb-protocol: Deprecate THAW control

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Drop the implementation of THAW control
Amitay Isaacs [Thu, 21 Jul 2016 04:17:02 +0000 (14:17 +1000)]
ctdb-daemon: Drop the implementation of THAW control

This control was used by the older implementation of tool to restore a
database from backup.  In the new implemenation of tool, it freezes and thaws
only the database being restored.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Adjust unit test output matching new ctdb
Amitay Isaacs [Tue, 26 Jul 2016 03:55:41 +0000 (13:55 +1000)]
ctdb-tests: Adjust unit test output matching new ctdb

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tool: Add replacement ctdb tool using new client API
Amitay Isaacs [Fri, 17 Jul 2015 12:45:04 +0000 (22:45 +1000)]
ctdb-tool: Add replacement ctdb tool using new client API

New tool breaks some of the tool unit tests due to improved error
messages.  Those changes are in the following patch.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tool: Remove old ctdb tool
Amitay Isaacs [Tue, 26 Jul 2016 03:34:11 +0000 (13:34 +1000)]
ctdb-tool: Remove old ctdb tool

Removing and adding replacement code makes the commits cleaner.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Deletion of IPs is deferred until the next takeover run
Martin Schwenke [Wed, 22 Jan 2014 06:12:09 +0000 (17:12 +1100)]
ctdb-daemon: Deletion of IPs is deferred until the next takeover run

This drastically simplifies the code.  "ctdb reloadips" behaves the
same, since it causes a takeover run immediately after IPs are
deleted.  "ctdb delip" now needs to be followed with an explicit "ctdb
ipreallocate".

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-daemon: Use consistent naming for monitoring mode
Amitay Isaacs [Fri, 22 Jul 2016 05:41:59 +0000 (15:41 +1000)]
ctdb-daemon: Use consistent naming for monitoring mode

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Reduce QueueBufferSize from 16k to 1k
Amitay Isaacs [Tue, 26 Jul 2016 04:50:10 +0000 (14:50 +1000)]
ctdb-daemon: Reduce QueueBufferSize from 16k to 1k

This avoids CTDB spending too much time processing multiple packets and
return to the event loop sooner.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Add QueueBufferSize tunable
Amitay Isaacs [Fri, 13 May 2016 14:25:31 +0000 (00:25 +1000)]
ctdb-daemon: Add QueueBufferSize tunable

This controls the maximum size of data ctdb will read from sockets at
a time.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-protocol: Add function ctdb_sock_addr_same_ip
Amitay Isaacs [Fri, 22 Jul 2016 03:35:00 +0000 (13:35 +1000)]
ctdb-protocol: Add function ctdb_sock_addr_same_ip

... to check if only the addresses are the same.

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-common: Fix parsing of debug level
Amitay Isaacs [Fri, 22 Jul 2016 04:31:02 +0000 (14:31 +1000)]
ctdb-common: Fix parsing of debug level

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Add machinereadable output tests
Amitay Isaacs [Tue, 26 Jul 2016 03:30:18 +0000 (13:30 +1000)]
ctdb-tests: Add machinereadable output tests

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Add more tests for ctdb setdbsticky and setdbreadonly
Amitay Isaacs [Tue, 26 Jul 2016 04:18:35 +0000 (14:18 +1000)]
ctdb-tests: Add more tests for ctdb setdbsticky and setdbreadonly

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Add tests for idempotence
Amitay Isaacs [Tue, 26 Jul 2016 04:17:20 +0000 (14:17 +1000)]
ctdb-tests: Add tests for idempotence

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Improve test to match exact output
Amitay Isaacs [Tue, 26 Jul 2016 04:32:39 +0000 (14:32 +1000)]
ctdb-tests: Improve test to match exact output

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix typo
Amitay Isaacs [Tue, 26 Jul 2016 03:55:18 +0000 (13:55 +1000)]
ctdb-tests: Fix typo

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Clean database before the test
Amitay Isaacs [Fri, 17 Jun 2016 08:06:07 +0000 (18:06 +1000)]
ctdb-tests: Clean database before the test

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Fix statistics update macro
Amitay Isaacs [Fri, 27 Mar 2015 05:32:11 +0000 (16:32 +1100)]
ctdb-daemon: Fix statistics update macro

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-doc: Drop documentation for obsolete tunable
Amitay Isaacs [Tue, 26 Jul 2016 04:53:38 +0000 (14:53 +1000)]
ctdb-doc: Drop documentation for obsolete tunable

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix CID 1364526 (Argument cannot be negative)
Amitay Isaacs [Wed, 27 Jul 2016 02:22:45 +0000 (12:22 +1000)]
ctdb-tests: Fix CID 1364526 (Argument cannot be negative)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix CID 1364525 (Argument cannot be negative)
Amitay Isaacs [Wed, 27 Jul 2016 02:22:06 +0000 (12:22 +1000)]
ctdb-tests: Fix CID 1364525 (Argument cannot be negative)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix CID 1364524 (Argument cannot be negative)
Amitay Isaacs [Wed, 27 Jul 2016 02:21:25 +0000 (12:21 +1000)]
ctdb-tests: Fix CID 1364524 (Argument cannot be negative)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix CID 1364523 (Argument cannot be negative)
Amitay Isaacs [Wed, 27 Jul 2016 02:20:21 +0000 (12:20 +1000)]
ctdb-tests: Fix CID 1364523 (Argument cannot be negative)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix CID 1364522 (Argument cannot be negative)
Amitay Isaacs [Wed, 27 Jul 2016 02:19:45 +0000 (12:19 +1000)]
ctdb-tests: Fix CID 1364522 (Argument cannot be negative)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Fix CID 1364521 (Argument cannot be negative)
Amitay Isaacs [Wed, 27 Jul 2016 02:19:04 +0000 (12:19 +1000)]
ctdb-tests: Fix CID 1364521 (Argument cannot be negative)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-common: Fix CID 1363227 (Resource leak)
Amitay Isaacs [Wed, 27 Jul 2016 02:02:19 +0000 (12:02 +1000)]
ctdb-common: Fix CID 1363227 (Resource leak)

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Move CTDB VNN structure to IP takeover code
Martin Schwenke [Mon, 13 Jun 2016 20:44:29 +0000 (06:44 +1000)]
ctdb-daemon: Move CTDB VNN structure to IP takeover code

It is only used in this code.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-ipalloc: Use a cumulative timeout for takeover run stages
Martin Schwenke [Fri, 27 May 2016 05:22:27 +0000 (15:22 +1000)]
ctdb-ipalloc: Use a cumulative timeout for takeover run stages

RELEASE_IP sometimes times out because killing TCP connections can
take a long time.

The aim of the takeover timeout is actually to limit the total amount
of time for an IP takeover run.  So, calculate a combined timeout
offset once and use it for each of the RELEASE_IP, TAKEOVER_IP,
IPREALLOCATED stages.  This gives RELEASE_IP more time to kill TCP
connections but still limits the total time.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-doc: ctdb_diagnostics(1) tweaks and cross-references
Martin Schwenke [Tue, 26 Jul 2016 00:58:36 +0000 (10:58 +1000)]
ctdb-doc: ctdb_diagnostics(1) tweaks and cross-references

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-doc: Integrate ctdb_diagnostics man page into build
Martin Schwenke [Fri, 12 Feb 2016 08:40:43 +0000 (19:40 +1100)]
ctdb-doc: Integrate ctdb_diagnostics man page into build

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-doc: Add ctdb_diagnostics man page
martijn van brummelen [Fri, 9 Oct 2015 13:59:50 +0000 (15:59 +0200)]
ctdb-doc: Add ctdb_diagnostics man page

Signed-off-by: martijn van brummelen <martijn@brumit.nl>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agodsdb: Fix CID 1364520 Incorrect expression (EVALUATION_ORDER)
Volker Lendecke [Tue, 26 Jul 2016 10:42:49 +0000 (12:42 +0200)]
dsdb: Fix CID 1364520 Incorrect expression (EVALUATION_ORDER)

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
Autobuild-Date(master): Tue Jul 26 23:48:19 CEST 2016 on sn-devel-144

7 years agoldb_ldb: Do not re-scan the index list for new DNs
Andrew Bartlett [Tue, 26 Jul 2016 02:17:47 +0000 (14:17 +1200)]
ldb_ldb: Do not re-scan the index list for new DNs

This DN can not already be in the list, because it is being
added to this DB for the first time just now.

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Pair-programmed-with: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>

Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Tue Jul 26 16:42:16 CEST 2016 on sn-devel-144

7 years agokcc: Clean up repsTo attribute for old DCs
Garming Sam [Mon, 25 Jul 2016 00:51:13 +0000 (12:51 +1200)]
kcc: Clean up repsTo attribute for old DCs

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Garming Sam <garming@samba.org>
Autobuild-Date(master): Tue Jul 26 05:11:57 CEST 2016 on sn-devel-144

7 years agokcc: Add a TODO for msDS[-RO]-Replica-Locations
Garming Sam [Fri, 22 Jul 2016 04:33:12 +0000 (16:33 +1200)]
kcc: Add a TODO for msDS[-RO]-Replica-Locations

When you modify the replica locations to exclude a DSA, it should be respected.

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agokcc: Add corresponding methods for repsTo
Garming Sam [Fri, 22 Jul 2016 02:14:20 +0000 (14:14 +1200)]
kcc: Add corresponding methods for repsTo

Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
7 years agoWHATSNEW: CTDB updates
Martin Schwenke [Tue, 31 May 2016 05:50:25 +0000 (15:50 +1000)]
WHATSNEW: CTDB updates

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Tue Jul 26 01:21:57 CEST 2016 on sn-devel-144

7 years agoctdb-tests: Add database related tool tests
Martin Schwenke [Thu, 21 Jul 2016 06:45:48 +0000 (16:45 +1000)]
ctdb-tests: Add database related tool tests

Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement database related controls in fake_ctdbd
Martin Schwenke [Thu, 21 Jul 2016 06:45:01 +0000 (16:45 +1000)]
ctdb-tests: Implement database related controls in fake_ctdbd

Pair-programmed-with: Amitay Isaacs <amitay@gmail.com>
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tools: Simplify "ctdb getdbseqnum" output format
Martin Schwenke [Fri, 22 Jul 2016 05:02:24 +0000 (15:02 +1000)]
ctdb-tools: Simplify "ctdb getdbseqnum" output format

Print the sequence number without preamble.  Print it in hex to match
the logs.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add "ctdb disable/enable" tool tests
Martin Schwenke [Thu, 21 Jul 2016 04:53:04 +0000 (14:53 +1000)]
ctdb-tests: Add "ctdb disable/enable" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement MODIFY_FLAGS control in fake_ctdbd
Martin Schwenke [Thu, 21 Jul 2016 04:52:13 +0000 (14:52 +1000)]
ctdb-tests: Implement MODIFY_FLAGS control in fake_ctdbd

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add "ctdb ban/unban" tool tests
Martin Schwenke [Thu, 21 Jul 2016 02:45:44 +0000 (12:45 +1000)]
ctdb-tests: Add "ctdb ban/unban" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement SET_BAN_STATE control in fake_ctdbd
Martin Schwenke [Thu, 21 Jul 2016 02:44:48 +0000 (12:44 +1000)]
ctdb-tests: Implement SET_BAN_STATE control in fake_ctdbd

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add "ctdb stop/continue" tool tests
Martin Schwenke [Wed, 20 Jul 2016 11:37:50 +0000 (21:37 +1000)]
ctdb-tests: Add "ctdb stop/continue" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement TAKEOVER_RUN message in fake_ctdbd
Martin Schwenke [Wed, 20 Jul 2016 11:35:18 +0000 (21:35 +1000)]
ctdb-tests: Implement TAKEOVER_RUN message in fake_ctdbd

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement STOP_NODE and CONTINUE_NODE controls in fake_ctdbd
Martin Schwenke [Wed, 20 Jul 2016 11:34:05 +0000 (21:34 +1000)]
ctdb-tests: Implement STOP_NODE and CONTINUE_NODE controls in fake_ctdbd

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add "ctdb getreclock" tool tests
Martin Schwenke [Tue, 19 Jul 2016 10:10:34 +0000 (20:10 +1000)]
ctdb-tests: Add "ctdb getreclock" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement GET_RECLOCK_FILE control in fake_ctdbd
Martin Schwenke [Tue, 19 Jul 2016 10:08:19 +0000 (20:08 +1000)]
ctdb-tests: Implement GET_RECLOCK_FILE control in fake_ctdbd

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add "ctdb getmonmode/disablemonitor/enablemonitor" tool tests
Martin Schwenke [Fri, 15 Jul 2016 07:21:59 +0000 (17:21 +1000)]
ctdb-tests: Add "ctdb getmonmode/disablemonitor/enablemonitor" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add monitoring related controls in fake_ctdbd
Amitay Isaacs [Thu, 7 Jul 2016 07:53:16 +0000 (17:53 +1000)]
ctdb-tests: Add monitoring related controls in fake_ctdbd

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tools: Simplify "ctdb getmonmode" output format
Martin Schwenke [Fri, 22 Jul 2016 05:40:00 +0000 (15:40 +1000)]
ctdb-tools: Simplify "ctdb getmonmode" output format

No preamble, just print keyword ENABLED or DISABLED.  Fix the
documentation to reflect this and remove the text that is simply
wrong.

Also remove output from "ctdb enablemonitor" and "ctdb disablemonitor"
on success.  This is just noise.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Add "ctdb setifacelink" tool tests
Martin Schwenke [Fri, 15 Jul 2016 07:12:32 +0000 (17:12 +1000)]
ctdb-tests: Add "ctdb setifacelink" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement SET_IFACE_LINK_STATE control in fake_ctdbd
Amitay Isaacs [Thu, 7 Jul 2016 07:45:31 +0000 (17:45 +1000)]
ctdb-tests: Implement SET_IFACE_LINK_STATE control in fake_ctdbd

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Add "ctdb listvars/getvar/setvar" tool tests
Martin Schwenke [Fri, 15 Jul 2016 06:51:16 +0000 (16:51 +1000)]
ctdb-tests: Add "ctdb listvars/getvar/setvar" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement controls related to tunables in fake_ctdbd
Amitay Isaacs [Thu, 7 Jul 2016 07:30:07 +0000 (17:30 +1000)]
ctdb-tests: Implement controls related to tunables in fake_ctdbd

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-daemon: Use refactored tunable code
Amitay Isaacs [Thu, 7 Jul 2016 07:04:32 +0000 (17:04 +1000)]
ctdb-daemon: Use refactored tunable code

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-common: Refactor tunable related functions
Amitay Isaacs [Thu, 7 Jul 2016 06:49:40 +0000 (16:49 +1000)]
ctdb-common: Refactor tunable related functions

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Add "ctdb runstate" tool tests
Martin Schwenke [Fri, 15 Jul 2016 06:12:26 +0000 (16:12 +1000)]
ctdb-tests: Add "ctdb runstate" tool tests

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement GET_RUNSTATE control in fake_ctdbd
Amitay Isaacs [Thu, 7 Jul 2016 05:54:27 +0000 (15:54 +1000)]
ctdb-tests: Implement GET_RUNSTATE control in fake_ctdbd

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
7 years agoctdb-tests: Add "ctdb setdebug" tool tests
Martin Schwenke [Fri, 15 Jul 2016 05:51:00 +0000 (15:51 +1000)]
ctdb-tests: Add "ctdb setdebug" tool tests

They also test "ctdb getdebug".

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
7 years agoctdb-tests: Implement GET_DEBUG and SET_DEBUG controls in fake_ctdbd
Amitay Isaacs [Thu, 7 Jul 2016 05:45:43 +0000 (15:45 +1000)]
ctdb-tests: Implement GET_DEBUG and SET_DEBUG controls in fake_ctdbd

Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>