summaryrefslogtreecommitdiffstats
path: root/ctdb/config
Commit message (Collapse)AuthorAgeFilesLines
...
| * Eventscripts - conditionally inherit ctdbd debug level in each monitor eventMartin Schwenke2011-08-171-0/+5
| | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit a7eebc06f81a7b0a3fba93759bcbdeabc8c2e86e)
| * Eventscripts - new function ctdb_set_current_debuglevel()Martin Schwenke2011-08-171-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function ensures that CTDB_CURRENT_DEBUGLEVEL is set. It works like this: 1. If it is already set then do nothing, since it might have been set some other way. The recommended "other way" would be to add a file in rc.local.d/. 2. If it is not set then set it by sourcing /var/ctdb/eventscript_debuglevel. 3. If this file does not exist then create it using output from "ctdb getdebug". If the optional 1st argument is set to "create" then don't source an existing file but create a new one instead - this is useful for creating the file just once in each event run in, say, 00.ctdb. If there's a problem getting the debug level from ctdb then it is silently set to 0 - no use spamming logs if our debug code is broken... Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 93910921c8a25f2b029733cd938069ff7c7bdab7)
| * Eventscripts - ensure the statd update-trigger file always exists.Martin Schwenke2011-08-161-3/+10
| | | | | | | | | | | | | | | | See the comment in the code for details. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 8ee9856996a8ec738e9d3ea7f1561605da526b8c)
| * Eventscripts: remove "return 0" from 50.samba service_stop().Martin Schwenke2011-08-161-2/+0
| | | | | | | | | | | | | | | | This potentially masks errors and was basically included by accident. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit e7e4a1b4f31118027fd13a6223192f9957cf2e74)
* | dont use a too big persistence timeout valueRonnie Sahlberg2011-08-171-2/+2
|/ | | | (This used to be ctdb commit 82628e32c431d66b806399ffb9657c3a031f6428)
* Change the errors for 10.interface to clearly state ERROR: for error messagesRonnie Sahlberg2011-08-151-5/+5
| | | | | | Update the tests system to catch the new error strings generated by this change (This used to be ctdb commit a2c30d88348da47d1a733a16e4c7d83c3becb6df)
* Merge remote branch 'martins/eventscript.10.interface'Ronnie Sahlberg2011-08-151-71/+69
|\ | | | | | | (This used to be ctdb commit 0d17daab38d4086f922a8006d4c545133adca191)
| * Eventscripts: 10.interfaces - make startup event actually mark interfaces up!Martin Schwenke2011-08-121-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | The startup event intends to mark interfaces up. However, it doesn't actually do that because $INTERFACES is empty. This uses the function get_all_interfaces() to list the interfaces... and then mark them up. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit fc62bf0975c6059ee467285565d0dc3b4daaf238)
| * Eventscripts: 10.interfaces - startup comment says assume all interfaces good.Martin Schwenke2011-08-121-1/+1
| | | | | | | | | | | | | | | | | | Interfaces are currently marked down. Mark them up instead, as per the comment... and discussion with Ronnie. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 35942841229cc72ce363a7236aec708f1a33136b)
| * Eventscripts: 10.interfaces - new function get_all_interfaces().Martin Schwenke2011-08-121-10/+16
| | | | | | | | | | | | | | | | | | | | | | | | Move existing interface listing code to new function in preparation for using it in startup event. While we're here change the "sort | uniq" into "sort -u" and save some complexity. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit cd1442531ad079b11c60f46ee9d34f5104bef219)
| * Eventscripts: 10.interface clean-ups - minor tweaks and new comments.Martin Schwenke2011-08-121-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | * sed can read files, it doesn't need a file piped to it * use $() subshells instead of `` - they seem to quote better in dash * tweak the uniquifying code so that it is easier to read * add comments * remove some extraneous semicolons at ends of lines Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 5f49537889a92c3cb68d9203912188bedf00ecd4)
| * Eventscripts: 10.interface clean-ups - variable name fix-ups.Martin Schwenke2011-08-121-28/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change most of the uppercase variable names to lowercase for consistency with other variables, readability and so they can be easily distinguished from environment/configuration variables. Change the name of 2 of the variabless to add some clarity. Changes are as follows: INTERFACES -> all_interfaces IFACES -> ctdb_interfaces IFACE -> iface I -> i REALIFACE -> realiface Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 7b201c1087b1433cfbc95de76cb4205e484ccd6f)
| * Eventscripts: 10.interfaces clean-ups - push logic into monitor_interfaces().Martin Schwenke2011-08-121-16/+4
| | | | | | | | | | | | | | | | | | | | | | | | The logic in the monitor event itself is very complex. Nearly all of it can go away by adding a single check of $CTDB_PARTIALLY_ONLINE_INTERFACES to the return logic of monitor_interfaces() and reversing the sense of the corresponding check. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit fa93177442c65c2a4eb2d5d5dba0a0da1c486969)
| * Eventscripts: 10.interfaces clean-up - use more descriptive variable names.Martin Schwenke2011-08-121-11/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | The name of variable $ok gives no clue to its meaning/use so this changes that variable to be named $up_interfaces_found. The return logic relating to $ok and $fail is difficult to read, so these variables are given true/fale values, allowing the return logic to be simplified. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 3402930319d462eab5525410f6a676952e120182)
| * Eventscripts: 10.interfaces cleanup - new functions mark_up(), mark_down().Martin Schwenke2011-08-121-16/+20
| | | | | | | | | | | | | | | | | | | | The same few lines of logic are used every time an interface up or down. This encapsulates those few lines in 2 new functions. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit ab443c4d7d282f282792abc6a6ac224ab06abe30)
* | Merge remote branch 'martins/60_nfs_regression'Ronnie Sahlberg2011-08-151-3/+3
|\ \ | | | | | | | | | (This used to be ctdb commit 845fb0ba24cf9118470c58fae7103ab8322ce079)
| * | Eventscripts: fix regression in 60.nfs export checking.Martin Schwenke2011-08-111-3/+3
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 35a60a63a9b5c7d98dde514ae552239506b691c9 introduced a regression, reported by "Jonathan Buzzard" <J.Buzzard@dundee.ac.uk>, as follows: Basically the use of sed in the following code snippet does not work for long exports where exportfs wraps the host or network onto the next line. exportfs | grep -v '^#' | grep '^/' | sed -e 's/[[:space:]]*[^[:space:]]*$//' | ctdb_check_directories The result is that the you get lots of blank lines being sent to ctdb_check_directories which causes the host to be marked as unhealthy and then thrashing sets in of the managed IP's making the whole cluster unusable. This tightens up the sed expression so that it is less likely to produce a spurious empty line. It also removes an unnecessary "grep -v". Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit bd39b91ad12fd05271a7fced0e6f9d8c4eba92e6)
* | Eventscripts: In 60.nfs don't restart NFS when restarting rpc.lockd.Martin Schwenke2011-08-121-1/+0
| | | | | | | | | | | | | | | | | | This effectively reverts 953dbfbddad656a64e30a6aca115cb1479d11573 and is a policy decision. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 380c9263eb37db5a250264316e250c2160908263)
* | Eventscripts: change failure counts and behaviour for statd and nfsd.Martin Schwenke2011-08-121-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | We reduce the number of failures before attempting a restart. However, after 6 failures we mark the cluster unhealthy and no longer try to restart. If the previous 2 attempts didn't work then there isn't any use in bogging the system down with an attempted restart on every monitor event. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit f654739080b40b7ac1b7f998cacc689d3d4e3193)
* | Eventscripts: clean up 60.nfs monitor event.Martin Schwenke2011-08-122-115/+152
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a helper function called nfs_check_rpc_service() and uses it to make the monitor event much more readable. An example of usage is as follows: nfs_check_rpc_service "mountd" \ -ge 10 "verbose restart:b unhealthy" \ -eq 5 "restart:b" The first argument to nfs_check_rpc_service() is the name of the RPC service to be checked. The RPC service corresponding to this command is checked for availability using the rpcinfo command. If the service is available then the function succeeds and subsequent arguments are ignored. If the rpcinfo check fails then a failure counter for that particular RPC service is incremented and subsequent arguments are processed in groups of 3: 1. An integer comparison operator supported by test. 2. An integer failure limit. 3. An action string. The value of the failure counter is checked using (1) and (2) above. The first check that succeeds has its action string processed - note that this explains the somewhat curious reverse ordering of checks. It the example above: * If the counter is >= 10 then a verbose message is printed describing the failure, the service is restarted in the background and the node is marked as unhealthy (via an "exit 1" from the function). * If the counter is == 5 then the service us restarted in the background. For more action options please see the code. This also changes the ctdb_check_rpc() function so that it no longer takes a program number to check. It now just takes a real RPC program name that rpcinfo can resolve via /etc/rpc. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 9b66057964756a6245bafb436eb6106fb6a2866e)
* Merge remote branch 'martins/eventscript.10.interface'Ronnie Sahlberg2011-08-111-1/+1
|\ | | | | | | (This used to be ctdb commit 84ac667af408816e5508719b9fdb7c5e25408640)
| * Eventscripts - fix 10.interface bash incompatibility.Martin Schwenke2011-08-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In dash, this fails gracefully with nothing to stderr: t=$(cat /does_not_exist) 2>/dev/null In bash the error from cat is still printed due to different order of evaluation. This works everywhere: t=$(cat /does_not_exist 2>/dev/null) Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit a6e61867c7a58d5a77cd8641d8df0b105cddff77)
* | Merge remote branch 'martins/eventscript_infrastructure'Ronnie Sahlberg2011-08-116-162/+265
|\ \ | | | | | | | | | (This used to be ctdb commit 20864822372b6d574c545287002a429b273c4bcc)
| * | Eventscripts: in 60.nfs move statd-notify code to service_reconfigure().Martin Schwenke2011-08-111-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | This means that it now occurs on every reconfigure event. As a result the ipreallocated event is removed. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit c45a89418ba733ff91d48340d72bdb6d2ef80051)
| * | Eventscripts - 60.nfs should define service_reconfigure().Martin Schwenke2011-08-111-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | Not $service_reconfigure. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 642292d7ba7a95567964b4160c7ee31a4f8985d1)
| * | Evenscripts: improvements to ctdb_service_check_reconfigure().Martin Schwenke2011-08-111-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Make this function applicable to "ipreallocated" event too. * Monitor event should not always succeed just because we reconfigure. If the service was unhealthy before the reconfigure and we end the reconfigure with "exit 0" then we can cause the node's health status to flip-flop. To avoid this we return the status of the service from the previous monitor event. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 21dfcbbdccd906fcd6ab7bba81418ce565bf63aa)
| * | Eventscripts: 50.samba - only start/stop nmbd if $CTDB_SERVICE_NMB set.Martin Schwenke2011-08-111-4/+10
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit defaec99df8c279d8e315d5010f9146e013afda2)
| * | Eventscripts: 50.samba needs null service_reconfigure() function.Martin Schwenke2011-08-111-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | Samba doesn't need to do anything for configuration changes. It will notice configuration changes and reload automatically. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit de13350c17261032a7468c2cf4d2cf4a8d66a840)
| * | Eventscripts: 40.vsftpd service_stop() no longer /dev/null's output.Martin Schwenke2011-08-111-1/+1
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit f928c201b6d0e1cd3e5568ae65186e3cee7c4988)
| * | Eventscripts: improvements to 41.httpd.Martin Schwenke2011-08-111-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Reduce the failure counts so that restart attempts happen sooner. * Use service_start() and service_stop() for the restart. ctdb_service_start() resets the failure count, which isn't very useful in this context. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 01776b9f29af9ad5c8534649ece1bd100e450434)
| * | Eventscript functions: new function ctdb_check_counter().Martin Schwenke2011-08-111-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should eventually be able to replace ctdb_check_counter_limit() and ctdb_check_counter_equal(), although it doesn't issue warnings like the former. It takes 4 optional arguments: 1. _msg - If "error" then over limit causes an error message and and exit 1. Anything else fails silently but the function returns 1. Default is "error". 2. _op - An integer operator supported by test (e.g. -eq, -ge, -gt). Default is -ge. 3. _limit - Limit for the counter to be used in comparison. Default is $service_fail_limit. 4. _service_name - Used to identify the counter. Default is $service_name. For example: ctdb_check_counter error -ge 5 foo will print a message and exit 1 if the counter for foo is >= 5, whereas ctdb_check_counter check -ge 5 foo will just return 1 if the counter for foo is >= 5, and ctdb_counter_check with print a message and exit 1 if the counter for $service_name is >= $service_fail_limit. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 5b01b7233515669e995e037205796e265643b176)
| * | Eventscripts: remove unused remove_ip() function.Martin Schwenke2011-08-111-12/+0
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 881af7c1417962b9b3ade6565b3e8eb9f9df7a97)
| * | Eventscripts: startstop_nfs stop no longer redirects output to /dev/null.Martin Schwenke2011-08-111-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | When stopping (as opposed to restarting) it is useful to see this information. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit a9ab1937239761dc32b143c9d225447bc6f090b4)
| * | Eventscripts: fix typo in _ctdb_counter_common().Martin Schwenke2011-08-111-1/+1
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit f57d1722b6aa082f3f826171acc57d7d796ea95c)
| * | Eventscripts: improve log messages in ctdb_start_stop_service().Martin Schwenke2011-08-111-2/+2
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 6da7095192fb172a06b434cfb02f4bfa6221b343)
| * | Eventscript functions: fix counter regression.Martin Schwenke2011-08-111-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d362be7d32079ac1390d67056ce107bfbca2c937 wasn't well thought out. Subsequent commits depend on ctdb_counter_init() taking an argument, so this makes those cases work. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 05a8fcfbac3da2b5843b31e0fe258255cc761190)
| * | Eventscript functions: ctdb_service_check-reconfigure() acts only on monitor.Martin Schwenke2011-08-111-0/+2
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit beabf506a5eb68fc50fdbf8772c1d2bb0f7951e3)
| * | Eventscripts: make 50.samba use $service_state_dir.Martin Schwenke2011-08-111-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 0f003f05e28037eefdce3a686fcb52cd2289af9d)
| * | Evenscripts: update 60.nfs to use ctdb_service_check_reconfigure.Martin Schwenke2011-08-111-5/+2
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 7c070b0bc86b3b9a91a9dc263b72c0567934535c)
| * | Evenscripts: update 60.nfs to use ctdb_setup_service_state_dir.Martin Schwenke2011-08-111-7/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | The state directory basename becomes "nfs" rather than "statd". One line of code i moved from the "startup" event to service_start(). Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit cc4c5c19af7efe01c48f73bb5ec5e607ed79db4c)
| * | Evenscripts: update 40.vsftpd to use ctdb_service_check_reconfigure.Martin Schwenke2011-08-111-13/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | To simplify we also remove the reconfigure from the recovered event because the monitor event will handle this very quickly anyway. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit da3aedd1a472b430b75989d3c157efedd382e327)
| * | Evenscripts: update 41.httpd to use ctdb_service_check_reconfigure.Martin Schwenke2011-08-111-5/+2
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 51c45b1c4751af41e5f9fd252763e0025f8cce3a)
| * | Eventscripts: rejig the reconfigure infrastructure.Martin Schwenke2011-08-113-13/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add an optional service name argument to existing reconfigure functions. * User function service_reconfigure() instead of variable $service_reconfigure to specify how a service is reconfigured. * New function ctdb_service_check_reconfigure() reconfigures a service if it is flagged for reconfigure. * Remove $service_reconfigure settings from 40.vsftpd and 41.httpd - they're the defaults. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 15d4111d0761d82f57d5d4f0b1227812d14e4d7c)
| * | Eventscript functions: move flagging of managed services.Martin Schwenke2011-08-111-6/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move flagging of managed or unmanaged services into ctdb_service_start() and ctdb_service_stop(). That way services will be correctly flagged if they are started from the startup and shutdown events. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 8675744cbd90b5a5095ed6fff7b36ae82004a457)
| * | Eventscript function: change service_start into a function.Martin Schwenke2011-08-116-50/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | service_start is currently a variable. This makes passing arguments hard. We change it to be a function and put default definitions into the functions file. We use a convention that if a service name argument is passed to a redefined version of service_start() or service_stop() then it will act unconditionally. If no argument is passed then it can use internal logic to decide if services should really be started. This is useful when a single eventscript handles multiple services. This is a cherry-pick of ae38895 that needed to be reset mid-stream. There is still some breakage following this commit. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 86e4aefed9fd1028660c98e3ea758c2b75ffc1d8)
| * | Eventscript functions: add optional event name argument to fail count functions.Martin Schwenke2011-08-111-5/+6
| | | | | | | | | | | | | | | | | | Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit b14f18649f42aab80ce0336c15ab6159f241c9af)
| * | Eventscript functions - optimise is_ctdb_managed_service().Martin Schwenke2011-08-111-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function generates a lot of trace when running under "set -x". This is due to the backward compatibility code. This adds 3 optimisations: 1. Before invoking the backward compatiblity code, is_ctdb_managed_service() returns early if the service is listed in $CTDB_MANAGED_SERVICES. 2. ctdb_compat_managed_service() actually now updates $CTDB_MANAGED_SERVICES instead of temporary variable $t. This means that a subsequent call to is_ctdb_managed_service() will short circuit due to optimisation (1). 3. ctdb_compat_managed_service() only adds a service to $CTDB_MANAGED_SERVICES if it is the service being checked by is_ctdb_managed_service(). This stops irrelevant services being added to $CTDB_MANAGED_SERVICES multiple times by multiple calls to is_ctdb_managed_service(). Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 758f4667c60089e09a0439c1eb74f5e426ca5e2e)
| * | 50.samba eventscript should use is_ctdb_managed_service "winbind".Martin Schwenke2011-08-111-19/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently it checks $CTDB_MANAGES_WINBIND directly in several places. This doesn't work when someone sets $CTDB_MANAGED_SERVICES directly. This modifies check_ctdb_manages_winbind() so that it return a condition rather than modifying $CTDB_MANAGES_WINBIND. This makes some code more readable. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 538902fbc1e74134a03987b36b3733ad641f8971)
| * | 50.samba eventscript should use is_ctdb_managed_service "samba".Martin Schwenke2011-08-111-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | Currently it checks $CTDB_MANAGES_SAMBA directly. This doesn't work when someone sets $CTDB_MANAGED_SERVICES directly. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit d8f0f8948abd340088720718fef7dc858661ba23)
| * | 50.samba eventscript should stop/start services when they become (un)managed.Martin Schwenke2011-08-111-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the value of $CTDB_MANAGES_SAMBA or $CTDB_MANAGES_WINBIND (or corresponding changes are made to $CTDB_MANAGED_VERSIONS), the associated service should be started or stopped as necessary. This add calls to ctdb_start_stop_service() to manage starting/stopping samba and winbind. An associated cleanup is made to the initial checks that one of $CTDB_MANAGES_SAMBA or $CTDB_MANAGES_WINBIND is set, replacing them with calls to is_ctdb_managed_service(). To handle the winbind cases ctdb_start_stop_service() and is_ctdb_managed_service() are updated to take an optional service name parameter. Signed-off-by: Martin Schwenke <martin@meltin.net> Conflicts: config/events.d/50.samba Most of this merged elsewhere. This just removes a check that this is the monitor event. Signed-off-by: Martin Schwenke <martin@meltin.net> (This used to be ctdb commit 257a2e350280c0b76ed2fac588cad167381fda52)