diff options
| author | Amitay Isaacs <amitay@gmail.com> | 2013-06-24 14:31:50 +1000 |
|---|---|---|
| committer | Amitay Isaacs <amitay@gmail.com> | 2013-07-02 12:59:08 +1000 |
| commit | ea00a5ecf558811a02f50209f59468c1c2f5f084 (patch) | |
| tree | aa1ccee73e487efc1665d732ac48d122b4c418ef | |
| parent | 622ccd09f9812e965d4e3be33daad2176c0ff9aa (diff) | |
| download | samba-ea00a5ecf558811a02f50209f59468c1c2f5f084.tar.gz samba-ea00a5ecf558811a02f50209f59468c1c2f5f084.tar.xz samba-ea00a5ecf558811a02f50209f59468c1c2f5f084.zip | |
banning: Log ban state changes for other nodes at higher debug level
Signed-off-by: Amitay Isaacs <amitay@gmail.com>
(This used to be ctdb commit c6f8407648abb37f2ed781afa5171dad8c9f59e9)
| -rw-r--r-- | ctdb/server/ctdb_banning.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/ctdb/server/ctdb_banning.c b/ctdb/server/ctdb_banning.c index 2ad7fbf9a38..0d1729dbe47 100644 --- a/ctdb/server/ctdb_banning.c +++ b/ctdb/server/ctdb_banning.c @@ -76,12 +76,16 @@ int32_t ctdb_control_set_ban_state(struct ctdb_context *ctdb, TDB_DATA indata) return -1; } if (bantime->time == 0) { - DEBUG(DEBUG_INFO,("unbanning node %d\n", bantime->pnn)); + DEBUG(DEBUG_NOTICE,("unbanning node %d\n", bantime->pnn)); ctdb->nodes[bantime->pnn]->flags &= ~NODE_FLAGS_BANNED; } else { - DEBUG(DEBUG_INFO,("banning node %d\n", bantime->pnn)); + DEBUG(DEBUG_NOTICE,("banning node %d\n", bantime->pnn)); if (ctdb->tunable.enable_bans == 0) { - DEBUG(DEBUG_INFO,("Bans are disabled - ignoring ban of node %u\n", bantime->pnn)); + /* FIXME: This is bogus. We really should be + * taking decision based on the tunables on + * the banned node and not local node. + */ + DEBUG(DEBUG_WARNING,("Bans are disabled - ignoring ban of node %u\n", bantime->pnn)); return 0; } |