summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * tools/ctdb: Add pdelete command to delete a record from persistent databaseAmitay Isaacs2013-01-091-0/+67
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit d23adec89b69e7c6f96c8e1417ef4ca4c9edc57e)
| * daemon: Update the comment and remove redundant check in ctdb_start_transport()Amitay Isaacs2013-01-091-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | ctdb_start_transport() is called just before "setup" event, when CTDB is ready to process the requests. "startup" event happens much later after a successful recovery. Transport method ctdb->methods is successfully initialized before ctdb_start_transport() is called. No need to check again. Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 9a70a4d23d00f6cb996c061ba3dfb7c47b4f6a4f)
| * eventscripts: Fail the setup event if CTDB does not become readyMartin Schwenke2013-01-091-4/+3
| | | | | | | | | | | | | | | | Currently it silently continues without attempting to set tunables. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 735ec99b99c7bb579851ce8293011aaf1dcc552a)
| * scripts: Make script_log() use supplied message, stop logger from hangingMartin Schwenke2013-01-081-1/+1
| | | | | | | | | | | | | | | | | | | | When using syslog any provided message arguments are ignored and not passed to logger. This means that logger blocks waiting on stdin. That's bad. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 50abf597cefe6f8ea2a2ff7694bf84641344a9b1)
| * scripts: Rework ctdb-crash-cleanup.sh so that it uses existing functionsMartin Schwenke2013-01-081-32/+17
| | | | | | | | | | | | | | | | This improves maintainability. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit e2aaa64925cca359c71520e01a18fc9461b0da4d)
| * scripts: Make drop_all_public_ips() more robustMartin Schwenke2013-01-081-3/+32
| | | | | | | | | | | | | | | | | | | | | | | | Incorporate some of the logic from ctdb-crash-cleanup.sh that ensures IPs are deleted even if they have the wrong netmask or are on the wrong interface. Factoring out some of the code will allow it to be used elsewhere. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 03356fd5ae7a3ac35fde0289cbea7c71ecf07367)
| * ctdbd: Default value for debug_hung_script should use ETCDIRMartin Schwenke2013-01-081-1/+1
| | | | | | | | | | | | | | | | | | That is, it should use whatever was specified in ./configure and should not hardcode /etc. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 13e5e609b262847b607e7af7e0685f44e7cb8e36)
| * scripts: debug-hung-script.sh doesn't need functions/loadconfigMartin Schwenke2013-01-081-3/+0
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 8507303b525d20c74e8ec4e7c4f5f275945cd3b6)
| * scripts: statd-callout should calculate CTDB_BASE if it is not setMartin Schwenke2013-01-081-3/+2
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 376015ba5ad6b7703ae9949a1d40a0c72dfaba0c)
| * eventscripts: Each script should set CTDB_BASE if it is not setMartin Schwenke2013-01-0819-0/+57
| | | | | | | | | | | | | | | | This makes it easier to run the scripts externally. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 740ea8ea5084149c8b552a01ee1c98c558b12384)
| * scripts: Move drop_all_public_ips() to the functions fileMartin Schwenke2013-01-082-6/+6
| | | | | | | | | | | | | | | | ... so it can be improved and used elsewhere. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit b23c30253cc9eb274b895cac0f8c65245ba0a200)
| * tests/simple: Add test to check recovery daemon IP verificationMartin Schwenke2013-01-082-6/+121
| | | | | | | | | | | | | | | | Also update ips_are_on_nodeglob() to handle negation. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 13a5944f8a27d43006acfffba76958693cae7702)
| * tests/eventscripts: Ratchet down debug level for ctdb_takeover_testsMartin Schwenke2013-01-081-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The default IP allocation algorithm used by ctdb_takeover_tests changed from "non-deterministic IPs" to "LCP2". The latter generates a lot more debug output. ctdb_takeover_tests is used by the ctdb tool stub to calculate IP address changes for failovers. This resulted in unexpected debug output that caused tests to fail. Since eventscript tests don't care how IP allocations are arrived at, the best solution is to turn down the debug level. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 3cc596d2b459d834f9785b3a98027e46431ff2b9)
| * recoverd: Separate each IP allocation algorithm into its own functionMartin Schwenke2013-01-081-66/+96
| | | | | | | | | | | | | | | | | | | | This makes the code much more readable and maintainable. As a side effect, fix a memory leak in LCP2. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 6a1d88a17321f7e1dc84b4823d5e7588516a6904)
| * recoverd: New function unassign_unsuitable_ips()Martin Schwenke2013-01-081-25/+34
| | | | | | | | | | | | | | | | | | Move the code into a new function so it can be called from a number of places. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 8adb255e62dbe60d1e983047acd7b9c941231d11)
| * recoverd: Move failback retry loop into basic_failback() and lcp2_failback()Martin Schwenke2013-01-082-31/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | The retry loop is currently in ctdb_takeover_run_core(). Pushing it into each function will make it possible to put each algorithm into a separate top-level function. This will make the code much clearer and more maintainable. Also keep associated test code compatible. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit f6ce18d011dd9043b04256690d826deb2640cd89)
| * recoverd: Trying to failback more IPs no longer allocates unassigned IPsMartin Schwenke2013-01-082-20/+2
| | | | | | | | | | | | | | | | | | | | | | | | Neither basic_failback() nor lcp2_failback() unassign IPs anymore, so there's no point looping back that far. Also fix a unit test that now fails because looping back to handle unassigned IPs is no longer logged. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit c09aeaecad7d3232b1c07bab826b96818756f5e0)
| * recoverd: basic_failback() can call find_takeover_node() directlyMartin Schwenke2013-01-081-4/+2
| | | | | | | | | | | | | | | | | | Instead of unassigning, looping back and depending on basic_allocate_unassigned. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 4dc08e37dec464c8785a2ddae15c7c69d3c81ac3)
| * recoverd: Don't do failback at all when deterministic IPs are in useMartin Schwenke2013-01-081-13/+5
| | | | | | | | | | | | | | | | | | This seems to be the right thing to do instead of calling into the failback code and continually skipping the release of an IP. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 4c87e7cb3fa2cf2e034fa8454364e0a7fe0c8f81)
| * recoverd: Move the test for both 'DeterministicIPs' and 'NoIPFailback' setMartin Schwenke2013-01-081-3/+8
| | | | | | | | | | | | | | | | | | If this is done earlier then some other logic can be improved. Also, this should be a warning since no error condition is set. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit e06476e07197b7327b8bdac9c0b2e7281798ffec)
| * recoverd: Fix a memory leak in IP allocationMartin Schwenke2013-01-081-0/+2
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit bcd5f587aff3ba536cb0b5ef00d2d802352bae25)
| * tests/takeover: Add some LCP2 tests for case when no node are healthyMartin Schwenke2013-01-086-0/+189
| | | | | | | | | | | | | | | | | | | | 3 tests should assign IPs to all nodes. 3 tests set NoIPTakeoverOnDisabled=1 and should drop all IPs. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit edda58a45915494027785608126b5da7c98fee85)
| * tests/takeover: Initial tests for deterministic IPsMartin Schwenke2013-01-083-0/+90
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 5c820b2398a42af0e94bc524854a1ad144a63f7b)
| * tests/takeover: Do output filtering for deterministic IPs algorithm tooMartin Schwenke2013-01-081-1/+2
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 98bd58a98d34ecca89c9042417d7527a18a5ecf9)
| * tests/takeover: Support testing of NoIPTakeoverOnDisabledMartin Schwenke2013-01-081-0/+5
| | | | | | | | | | | | | | | | Via $CTDB_SET_NoIPTakeoverOnDisabled. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit d357d52dbd533444a4af6151d04ba119a1533068)
| * tests/takeover: IP allocation now selected via $CTDB_IP_ALGORITHMMartin Schwenke2013-01-082-16/+18
| | | | | | | | | | | | | | | | | | Default to LCP2, like ctdbd. Also support "det" for deterministic IPs. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 20631f5f29859920844dd8f410e24917aabd3dfd)
| * tests/takeover: Support valgrinding the takeover codeMartin Schwenke2013-01-081-1/+1
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 06ad6b8a19f830472b0ed65cb52e7c3ea74ed1dc)
| * tests: new simple integration test for delip interface garbage collectionMartin Schwenke2013-01-071-0/+62
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 1a5410e8349cdb96fdc51aa5ecd4f5734f6798a5)
| * tests: new function ip2ipmask() for integration testingMartin Schwenke2013-01-071-0/+7
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 8164d9b29bf9080ccc76b1305fb6c07f1ed61d55)
| * ctdbd: Clean up orphaned interfaces when an IP is deletedMartin Schwenke2013-01-071-4/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new function ctdb_remove_orphaned_ifaces() and call it in ctdb_control_del_public_address(). ctdb_remove_orphaned_ifaces() uses a naive implementation that does things in a very obvious way. There are many ways to improve the performance - some are mentioned in a comment in the code. However, I doubt that this will be a bottleneck even with a large number of public IPs. Running the eventscript is likely to outweigh the cost of this cleanup. Signed-off-by: Martin Schwenke <martin@meltin.net> Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit cc1a3ae911d3fee8b87fda5de5ab6d9499d7510a)
| * tests/complex: Add NFS test when CTDB is killed on one of the nodesAmitay Isaacs2013-01-071-0/+88
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit b849fb4923d6a34141fe19006a974de81508ceda)
| * Eventscripts: Change the default reconfigure action to do nothingMartin Schwenke2013-01-075-14/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | A default action of restarting the service doesn't obey the principle of least surprise. It cause the NFS service to be implicitly reintroduced. This allows no-op functions to be removed from some eventscripts and service restart functions to be added to others. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit c75b5e5b4d000f5c7dab403df8238ceed390c1c0)
| * Eventscripts: Do not restart NFS on reconfigureMartin Schwenke2013-01-077-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like this restart was accidentally reintroduced in commit fc0678d351187cfa4c71123f97c0f493aacd5d16 when $service_reconfigure became unset so the default action of restarting the service would occur. From there cleanups have explicitly reintroduced it and carried it through the code. Also update the unit tests affected by this change. The restart was originally removed in commit bc481c3f1a44c50648488c4f8a7f15ec395d446f. The default reconfigure action of restarting a service is clearly suboptimal and will be addressed in a separate patch. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 2629de72e1f37b5e46772c2ef8d8d0012fc4ed37)
| * ctdbd: Initialise the node flags in just one placeMartin Schwenke2013-01-074-23/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently flags are initialised in 2 places. One of them is in ctdb_tcp_listen_automatic(), which just seems wrong. This makes the code easier to follow by just doing it in ctdb_start_daemon(). This means that the flags are now initialised later than previously. However, it is still done before the transport is started and before clients can connect. In future it might make sense to do a similar thing with setting the PNN. However, the current optimisation is reasonably obvious... Signed-off-by: Martin Schwenke <martin@meltin.net> Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 2bbee8ac23ad5b7adf7122d8c91d5f0d54582507)
| * ctdbd: Remove debug option --node-ip, use --listen insteadMartin Schwenke2013-01-073-54/+21
| | | | | | | | | | | | | | | | | | This effectively reverts d96cb02c2c24f9eabbc53d3d38e90dea49cff3e0 Signed-off-by: Martin Schwenke <martin@meltin.net> Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 496387a585b2c5778c808cf02b8e1435abde4c3e)
| * tests: Local daemons should use --listen instead of --node-ipMartin Schwenke2013-01-071-1/+1
| | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit 3221fce9ee2f6fdd3bb17a5e1629ad52a32f90d6)
| * Initscript: when checking status, print output of "ctdb ping" if it failsMartin Schwenke2013-01-071-1/+4
| | | | | | | | | | | | | | | | | | At the moment the caller has no idea why it thinks CTDB isn't running and we can't debug failures... Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 776590bf84d221092298346a28d7fc0552a67c9d)
| * ctdb:recover: fix a comment typoMichael Adam2013-01-051-1/+1
| | | | | | | | | | | | Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 5067392d2e06795559f25828b65c129608b65c0b)
| * events/50.samba: fix testparm background updateMichael Adam2013-01-051-1/+1
| | | | | | | | | | | | | | | | | | | | creating the smb.conf cache with "-v" results in a cache file that fails to load with "testparm -s ..." later on due to "copy = " not being processable. (Copying the empty service name fails). Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 81788cfabe960497b050c5ee4e4e487ee061012a)
| * daemon: Add a tunable to enable automatic database priority settingAmitay Isaacs2013-01-054-5/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Samba versions 3.6.x and older do not set the database priority. This can cause deadlock between Samba and CTDB since the locking order of database will be different. A hack was added for automatic promotion of priority for specific databases to avoid deadlock. This code should not be invoked with Samba version 4.x which correctly specifies the priority for each database. Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 4a9e96ad3d8fc46da1cd44cd82309c1b54301eb7)
| * daemon: Check if log_latency_ms is set before using itAmitay Isaacs2012-11-301-1/+1
| | | | | | | | | | | | | | | | This fixes a bug where wrong variable is checked. Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit f81e9add466b1d9b2796c09c6ba63b77296ea149)
| * Git should ignore generated include/version.h fileMartin Schwenke2012-11-271-0/+1
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 905cd1293aa97dc7839a59b4f68eca02981f0891)
| * vacuum: Avoid some tallocs in ctdb recoveryVolker Lendecke2012-11-261-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | In a heavily loaded and volatile database a lot of SCHEDULE_FOR_DELETION requests can come in between fast vacuuming runs. This can lead to significant ctdb cpu load due to the cost of doing talloc_free. This reduces the number of objects a bit by coalescing the two objects of delete_record_data into one. It will also avoid having to allocate another talloc header for a SCHEDULE_FOR_DELETION key. Not the full fix for this problem, but it might contribute a bit. (This used to be ctdb commit 9a02f61547ddf74629aca21639d8fb61c1df7cbb)
| * doc: Update ping_pong documentation to add -c optionAmitay Isaacs2012-11-211-0/+17
| | | | | | | | | | | | Signed-off-by: Amitay Isaacs <amitay@gmail.com> (This used to be ctdb commit d05faf294e58e22ae3fbc76162258f1ae8178129)
| * utils:ping_pong: add a -c switch to check the lock before reading/writingMichael Adam2012-11-201-2/+40
| | | | | | | | | | | | | | | | | | | | This is to verify that the fcntl F_GETLK call reports F_UNLCK if called from a process already holding a lock. This is for example used by samba's strict locking code in combination with "posix locking = true". Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 4f42d17b74ce891691eee1cead498959cc8e4837)
| * recovery: data corruption of persistent DBs after recoveries: don't delete ↵Michael Adam2012-11-201-2/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | emtpy records The record-by-record mode of recovery deletes empty records. For persistent databases, this can lead to data corruption by deleting records that should be there: - Assume the cluster has been running for a while. - A record R in a persistent database has been created and deleted a couple of times, the last operation being deletion, leaving an empty record with a high RSN, say 10. - Now a node N is turned off. - This leaves the local database copy of D on N with the empty copy of R and RSN 10. On all other nodes, the recovery has deleted the copy of record R. - Now the record is created again while node N is turned off. This creates R with RSN = 1 on all nodes except for N. - Now node N is turned on again. The following recovery will chose the older empty copy of R due to RSN 10 > RSN 1. ==> Hence the record is gone after the recovery. On databases like Samba's registry, this can damage the higher-level data structures built from the various tdb-level records. This patch fixes that problem by not deleting empty records in recoveries for persistent databases. Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 6860c79aea416f56cfd7a6af790bbdf495dbc54e)
| * recoverd: fix a comment typoMichael Adam2012-11-201-1/+1
| | | | | | | | | | | | Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit 909269a4a3690e1245117ca1af935401455785e6)
| * vacuum: fix a comment typoMichael Adam2012-11-191-1/+1
| | | | | | | | | | | | | | Pair-Programmed-With: Volker Lendecke <vl@samba.org> Signed-off-by: Michael Adam <obnox@samba.org> (This used to be ctdb commit bab744e3c49efef2e05dc09e8ea9bd3e3fa58716)
| * Eventscripts: 10.interface should list configured interfacesMartin Schwenke2012-11-191-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | The current code lists available interfaces. If IPs are configured in some other way than the public addresses file (e.g. ctdb addip) and their interfaces default to being marked down then, since down interfaces are not available, these interfaces can never be marked up. The configured interfaces should be listed instead. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit d8f010355b715e49709836e057a5d0f110919897)
| * ctdbd: Make the link status of new interfaces more flexibleMartin Schwenke2012-11-191-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neither up nor down is a good default value for the link status of a new interface. Up means that IPs can be assigned to interfaces before the true state is known and they can move away quickly if the interface is actually down. Down means that IPs can't be assigned to an interface for a variable amount of time - until a monitor cycle occurs - and this can result in imbalanced IPs. This is a neat compromise. Before the startup event completes, IPs can't be assigned to interfaces because all interfaces begin in a down state. As soon as the startup event completes, IPs can be allocated to any interface that has been marked up by the eventscript. Later, during normal operation, newly added IPs can be assigned to new interfaces immediately. The IPs will still move away if an interface is noticed to be down in the next monitor cycle, but that is the exception rather than the rule. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 9275a69a414482f1053ae14528d5972575b9214e)
generated by cgit (git 2.34.1) at 2025-09-29 15:27:03 +0000