diff options
| author | Martin Schwenke <martin@meltin.net> | 2013-09-18 17:06:16 +1000 |
|---|---|---|
| committer | Amitay Isaacs <amitay@gmail.com> | 2013-09-19 12:57:36 +1000 |
| commit | 07d3a1b234c59837fbc4bca96a539c09e1a40435 (patch) | |
| tree | 2c1bbfc3028321614c3ee80955bda518de32773d /ctdb/server/ctdb_recoverd.c | |
| parent | 236b2524de5b68b60f70db6ef3c3b78604976deb (diff) | |
| download | samba-07d3a1b234c59837fbc4bca96a539c09e1a40435.tar.gz | |
recoverd: Improve logging for takeover runs
Takeover runs are currently silent when they succeed. However, they
are important, so log something by default.
Signed-off-by: Martin Schwenke <martin@meltin.net>
(This used to be ctdb commit b39aa2e401fbb581207d986bac93778e9c01acdc)
Diffstat (limited to 'ctdb/server/ctdb_recoverd.c')
| -rw-r--r-- | ctdb/server/ctdb_recoverd.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/ctdb/server/ctdb_recoverd.c b/ctdb/server/ctdb_recoverd.c index 14f1fa8cde3..cb492f5d179 100644 --- a/ctdb/server/ctdb_recoverd.c +++ b/ctdb/server/ctdb_recoverd.c @@ -1665,6 +1665,8 @@ static bool do_takeover_run(struct ctdb_recoverd *rec, int ret; bool ok; + DEBUG(DEBUG_NOTICE, ("Takeover run starting\n")); + if (rec->takeover_run_in_progress) { DEBUG(DEBUG_ERR, (__location__ " takeover run already in progress \n")); @@ -1725,7 +1727,7 @@ static bool do_takeover_run(struct ctdb_recoverd *rec, } if (ret != 0) { - DEBUG(DEBUG_ERR, ("IP reallocation failed\n")); + DEBUG(DEBUG_ERR, ("ctdb_takeover_run() failed\n")); ok = false; goto done; } @@ -1742,6 +1744,8 @@ done: rec->need_takeover_run = !ok; talloc_free(nodes); rec->takeover_run_in_progress = false; + + DEBUG(DEBUG_NOTICE, ("Takeover run %s\n", ok ? "completed successfully" : "unsuccessful")); return ok; } |