summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Eventscripts: Clean up 11.routingMartin Schwenke2012-07-301-9/+8
| | | | | | | | | | | | | | | | | | | | The loops can all be done without cat or grep. The pair of loops in updateip is combined into a single loop. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 96fdda124f5511fb76190e7c7a7f0b98e6b01a31)
| * ctdbd: Log a meaningful message if the nodes file/list is emptyMartin Schwenke2012-07-261-0/+9
| | | | | | | | | | | | | | | | | | Right now the message says it can't bind to any of the addresses... even when there aren't any! Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 553455b386aa7848a516a921dfc14eb87c8a3fc1)
| * ctdbd: Remove the worked "Forced" from message about running eventscriptsMartin Schwenke2012-07-261-1/+1
| | | | | | | | | | | | | | | | | | | | The eventscripts are run after a takeover run and in this case they're not forced. The messages seems to imply that somone has run "ctdb eventscript" when that is not necessarily the case. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 3880589db4d563e438126cf5080261fa06b9e242)
| * ctdbd: Fix ctdb_control_release_ip() on local daemonsMartin Schwenke2012-07-261-5/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When running on local daemons no IPs are actually assigned to interfaces. Commit 9a806dec8687e2ec08a308853b61af6aed5e5d1e broke ctdb_control_release_ip() for local daemons because it asks the system which interface the given IP is on, instead of the old behaviour of trusting CTDB's internal records. For local deamons (i.e. !ctdb->do_checkpublicip) revert to the old behaviour of looking up the interface internally. This is good enough, given that the tests don't tend to misconfigure the addresses. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 38e8651b955afdbaf0ae87c24c55c052f8209290)
| * Initscript: clean up drop_all_public_ips()Martin Schwenke2012-07-261-7/+3
| | | | | | | | | | | | | | | | | | This makes the case implicit where $CTDB_PUBLIC_ADDRESSES is unset. This is OK because that's not an interesting code path. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 5b2725d1ae052e848c2487cb10c5393a877d118c)
| * tests/tool: Run ctdb_tool_* under $VALGRINDMartin Schwenke2012-07-261-1/+1
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 6616a5712b5d4db2b9ba6a88cec79378696c2184)
| * tests/eventscripts: Rewrite the testparm stubMartin Schwenke2012-07-261-8/+23
| | | | | | | | | | | | | | | | | | | | It currently needs the real testparm command installed even though it only uses limited features. It is easy enough to fake up the functionality that 50.samba uses. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 7ef9916bd95ff2472359a412eac5489f1aad2dce)
| * tests/complex: Fix broken ctdb_test_check_real_cluster()Martin Schwenke2012-07-261-0/+3
| | | | | | | | | | | | | | | | It doesn't set $h at all... Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 3f268805c14c51f23024267916eae161bada8a0e)
| * tests/simple: ctdb stop/continue tests weren't actually checking IPsMartin Schwenke2012-07-262-2/+2
| | | | | | | | | | | | | | | | The correct variable is $test_node_ips, not $ips. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 8d17dacee415dd0b4268805a366a86f83e33f27c)
| * tests: select_test_node_and_ips() should try to avoid failingMartin Schwenke2012-07-261-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | Sometimes "ctdb sync" doesn't do its job, so we end up with unassigned IPs. If $test_node isn't set then this is bad. However, try a few times to ensure it is set. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 2fd0157382b42aa5c5212b8e743c6f589edc6662)
| * tests: simple tests against local daemons should check $TEST_LOCAL_DEAMONSMartin Schwenke2012-07-263-4/+4
| | | | | | | | | | | | | | | | Note the old $CTDB_TEST_REAL_CLUSTER - it doesn't exist anymore... Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 47180dc75d15f3d61470705603565b718491c9f8)
| * tests: run_tests should exit with $status with -e optionMartin Schwenke2012-07-261-1/+1
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 619af3e857c2ced3840abfd86135cc954796da97)
| * tests/simple: ctdb reloadips test should use $test_ipMartin Schwenke2012-07-261-6/+5
| | | | | | | | | | | | | | | | There's no point recalculating this value. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 6e7bd9685406ae024d413a5d9d8c6e0d89b15567)
| * tests: select_test_node_and_ips() should never select non-node -1Martin Schwenke2012-07-261-5/+4
| | | | | | | | | | | | | | | | Instead of selecting the 1st pnn found, select the 1st one that isn't -1. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit f02e501342112aab67aee95f253e29a670b29273)
| * util: Do not lock down memory when running with local daemonsAmitay Isaacs2012-07-261-1/+6
| | | | | | | | | | | | | | | | | | Thanks to Ronnie for highlighting the issue of memory lockdown on AIX. Fix typo, use getuid and not getpid. Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 21a5cbf9518fafc610939f14874371a52b1dc8b3)
| * statd-callout: Fix a bug in the calculations of $STATEMartin Schwenke2012-07-261-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | It is just meant to be even, so divided *and* multiplied by 2. Use $(( )) to make it more readable. While touching this code, make the related calculation a bit more readable too. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 25d45e69f4ffc2b26061ac13038d52a353e79e61)
| * Eventscripts: Default route on NAT gateway should have a metric of 10Martin Schwenke2012-07-264-91/+99
| | | | | | | | | | | | | | | | | | | | | | | | At the moment routes from 11.routing can fail to be added because they conflict with the default route added by 11.natgw. NAT gateway is meant to be a last resort, so routes from 11.routing should override it. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 624f4677e99ed1710a0ace76201150349b1a0335)
| * Eventscripts: Update/remove stale comments in 11.natgwMartin Schwenke2012-07-261-7/+2
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 5d713d5e5be67f5914a661694c15d938bd67dea3)
| * Eventscripts: Retrieve and build NAT gateway details better in 11.natgwMartin Schwenke2012-07-261-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * "ctdb natgw" is run twice when it doesn't need to be. * Tweak the parsing of "ctdb natgw" output so that it is done by the shell instead of a bunch of external processes. * Make default NAT gateway be -1, even on error. If the process failed entirely then it could previously be empty. * Streamline the error handling using die() for when there is no NAT gateway. * Downcase script-local variable names. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 630cfe6451ba23d959fa4907fbba42702337ed3b)
| * Eventscripts: Optimise building the host address in 11.natgwMartin Schwenke2012-07-261-3/+3
| | | | | | | | | | | | | | | | | | | | It can be build without forking unnecessary processes. Also downcase variable name because it is local to script. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 34f58a0773618c4508a55ad75fc4602dad5a5f4c)
| * Eventscripts: Clean up startup sanity check in 11.natgwMartin Schwenke2012-07-261-8/+3
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit f6e421e8bf935cae790a6dc2b861eb9c7f8610b4)
| * Eventscripts: remove redundant firewall rules from 11.natgwMartin Schwenke2012-07-261-7/+0
| | | | | | | | | | | | | | | | | | | | | | aeb70c7e7822854eb87873a5c7783e27e6e72318 said it moved these but it redundantly duplicated them instead. That commit also fixed the problem because it moved the rules after delete_all() not out of the startup event as claimed. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 07149edaecb3caa672163e5a3b89715557d5205a)
| * Eventscripts: 11.natgw $CTDB_NATGW_PUBLIC_IP splitting optimisationMartin Schwenke2012-07-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | $CTDB_NATGW_PUBLIC_IP can be split into $_ip and $_maskbits without forking lots of processes. Also "local" isn't supported by POSIX. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit e20fdb974158061f4627d6f360c168d764690e6f)
| * web: Add my name to the developer list.Amitay Isaacs2012-07-241-0/+1
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit b3e798f357606648f04d8a67ffee775b34fdede7)
| * Remove tevent_loop_allow_nesting()Amitay Isaacs2012-07-162-2/+0
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 538c68d0e83e14f0000981ee06408b8f0035be37)
| * ctdbd: Return explicit boolean values for function returning boolAmitay Isaacs2012-07-162-2/+2
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 3de2830ae68241ee95bcc14dc1bb896ff18d86ce)
| * util: Do not try to lockdown memory when running in local daemons modeAmitay Isaacs2012-07-161-1/+6
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 25f84797a64a683c303b04057aa8113e9fc47c49)
| * Fix compiler warnings.Amitay Isaacs2012-07-132-2/+2
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit d29e1880c8ce7219e065d31b47b0e8ad9e83146d)
| * run_tests: improve spacingMichael Adam2012-07-031-1/+2
| | | | | | | | (This used to be ctdb commit a0a0f5588445aeabe07b0e4d65087db454dc09da)
| * run_tests.sh: fix a commentMichael Adam2012-07-031-1/+1
| | | | | | | | (This used to be ctdb commit 0e515115b3c21cb179fd7a6356164ac1b5d423e0)
| * ctdb: use correct "persistent" state for ctdb_attach in "ctdb cattdb"Michael Adam2012-07-031-1/+1
| | | | | | | | | | | | | | | | | | Originally, "ctdb cattdb" attached explicitly as non-persistent, which is now forbidden for persistent databases by the server. Pair-Programmed-With: Gregor Beck <gbeck@sernet.de> (This used to be ctdb commit 85a367005bd669309bb7e532b60d27621110180d)
| * ctdbd: refuse attaching with "persistent" to a non-persistent db and v.v.Gregor Beck2012-07-031-0/+6
| | | | | | | | | | | | Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 1ebbaa620b3cfb9ff373828e4aaa84246cf3ec25)
| * When we find an ip we shouldnt host, just release itRonnie Sahlberg2012-06-201-2/+6
| | | | | | | | | | | | Dont call a full blown clusterwide ipreallocation, just release it locally (This used to be ctdb commit 9a806dec8687e2ec08a308853b61af6aed5e5d1e)
| * When we release an ip, get the interface name from the kernelRonnie Sahlberg2012-06-201-13/+9
| | | | | | | | | | | | | | | | instead of using the interface where ctdb thinks the ip is hosted at. The difference is that this now allows us to handle cases where we want to release an ip but ctdbd does not know which interface the ip is assigned on. (user has used 'ip addr add...' and manually assigned an ip to the wrong interface) (This used to be ctdb commit c6bf22ba5c01001b7febed73dd16a03bd3fd2bed)
| * Add new command to find which interface is located onRonnie Sahlberg2012-06-203-0/+105
| | | | | | | | (This used to be ctdb commit f07376309e70f5ccdb7de8453caacc71b451ab48)
| * STATISTICS: Add tracking of the 10 hottest keys per database measured in ↵Ronnie Sahlberg2012-06-135-7/+144
| | | | | | | | | | | | | | | | hopcount and add mechanisms to dump it using the ctdb dbstatistics command (This used to be ctdb commit 8307c70ed98996b430c470e9641a09fdeeb81bd8)
| * Reimplement logging of long running eventsMartin Schwenke2012-06-121-0/+46
| | | | | | | | | | | | | | | | | | Reimplement 5aba53e6adcfcd7edbdac9e30aa5fcba176aca00 using tevent trace points. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 98e1b46adba11b9549b5c5976e1f561fe732fa6e)
| * tevent: change version to 0.9.16Stefan Metzmacher2012-06-121-0/+82
| | | | | | | | | | | | | | | | | | | | | | This adds tevent_*_trace_*() and tevent_context_init_ops() metze Autobuild-User(master): Stefan Metzmacher <metze@samba.org> Autobuild-Date(master): Fri Jun 8 20:47:41 CEST 2012 on sn-devel-104 (This used to be ctdb commit 0dc204988eadff214dd149a756d756ab6e96e410)
| * tevent: expose tevent_context_init_opsStefan Metzmacher2012-06-122-3/+19
| | | | | | | | | | | | | | | | | | | | This can be used to implement wrapper backends, while passing a private pointer to the backens init function via ev->additional_data. metze (This used to be ctdb commit 7ebc00dc6a89043a971a720e7c21baf5f2a0233d)
| * lib/tevent: Add trace point callbackMartin Schwenke2012-06-127-0/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Set/get a single callback function to be invoked at various trace points. Define "before wait" and "after wait" trace points - more trace points can be added later if required. CTDB wants this to log long waits and events. Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Signed-off-by: Stefan Metzmacher <metze@samba.org> (This used to be ctdb commit cb2bbe93628c1ab932c2e1ad6e2ec199a98f74c6)
| * Revert "TEVENT: Add back tracking of long runnig events to the local copy ↵Martin Schwenke2012-06-125-66/+0
| | | | | | | | | | | | | | | | | | | | of tevent library" This reverts commit 5aba53e6adcfcd7edbdac9e30aa5fcba176aca00. Do this using new tevent trace point callback. (This used to be ctdb commit 88040778aace229d724de1ba7556aded12e22f86)
| * lib/tevent: In poll_event_context, add a pointer back to the tevent_contextMartin Schwenke2012-06-121-0/+4
| | | | | | | | | | | | | | | | | | This makes it consistent with the other backends. Signed-off-by: Martin Schwenke <martin@meltin.net> Signed-off-by: Stefan Metzmacher <metze@samba.org> (This used to be ctdb commit e0c9200c05b1f7a04e002f505ebb5ba9340c0ca1)
| * lib/tevent/testsuite: no longer use 'compat' symbolsStefan Metzmacher2012-06-121-12/+12
| | | | | | | | | | | | metze (This used to be ctdb commit 6559106b8b853920f325f2dba532f4008e931fa3)
| * Run the shutdown eventscript before we tear down the transportRonnie Sahlberg2012-05-301-1/+1
| | | | | | | | | | | | This allows eventscripts to still be able to call and use ctdb during the shutdown phase. (This used to be ctdb commit 1a6a011c772f7d302d114d7c8a151fa7820ec85f)
| * tests: Increment RSN always in ctdb_update_record_persistent testAmitay Isaacs2012-05-281-2/+2
| | | | | | | | | | | | | | | | | | | | If the record does not exist in persistent DB, RSN for that record is considered 0. To write a record, RSN for that record should be set to 1, otherwise the RSN check would fail. Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit ac89da4eea98fa686408c5671a6c44c0fd1d7a58)
| * tests: Fix ctdb_fetch test (parse extra lines of output)Amitay Isaacs2012-05-281-1/+1
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 0be452958db95c8253c362a1c08a1966e53a1f99)
| * tests: Fix flakey behavior of ctdb_fetch testAmitay Isaacs2012-05-281-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There were two issues with this test: 1. Since the messages are sent from one node to the next, if a node does not register for messages before CTDB on that nodes receives the message, it will never be seen by ctdb_fetch and it would block on receive and would not send any messages to next node. The crude solution is to sleep just before the messages are sent, so that ctdb_fetch on all nodes have registered for the messages. 2. If ctdb_fetch stops sending messages after timelimit expiry, the next node will keep waiting to receive messages in event_loop_once(). The default timeout is 30 seconds for event_loop_once(). Adding a timed event will always set the timeout value to the time remaining for the timed event to expire. Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit bc55e09fdac9f743d6428bfe0be77840ad0fd1ba)
| * server: Replace BOOL datatype with bool, True/False with true/falseAmitay Isaacs2012-05-2810-24/+19
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 6e5cbe8fff71985e5a2fc16b7e9f2b868011ff5d)
| * tests/eventscripts: Tweak expected output for lockd:b restartMartin Schwenke2012-05-251-1/+1
| | | | | | | | | | | | | | Commit 13acd58c41fba1a33894fbd654fed69ea0eac322 mades this test fail, since lockd:b and lockd:bs were incorrectly producing the same output. (This used to be ctdb commit fd3b73d7e634f16cbb99d7d5a548e12f00d1aadb)
| * tests: Complex tests must not be run from a cluster nodeMartin Schwenke2012-05-251-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Tickle tests fail if run from a node involved in the test. The condition is actually weaker than this: the test can't be run from a CTDB node that is hosting public addresses that may be used by the test. Rework ctdb_test_check_real_cluster() to support checking this. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 14012781c3751a514055df29ea70adfb12ecb2d9)