You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by me...@apache.org on 2015/06/21 10:46:20 UTC

[3/4] mesos git commit: Fixed authentication failure triggered slave crash.

Fixed authentication failure triggered slave crash.

Review: https://reviews.apache.org/r/31838


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/6d9f9b7a
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/6d9f9b7a
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/6d9f9b7a

Branch: refs/heads/master
Commit: 6d9f9b7ac6c690ed3a1690a108f95577375d6f68
Parents: b3a3d53
Author: Till Toenshoff <to...@me.com>
Authored: Sun Jun 21 08:28:14 2015 +0000
Committer: Adam B <ad...@mesosphere.io>
Committed: Sun Jun 21 08:32:50 2015 +0000

----------------------------------------------------------------------
 src/slave/slave.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/6d9f9b7a/src/slave/slave.cpp
----------------------------------------------------------------------
diff --git a/src/slave/slave.cpp b/src/slave/slave.cpp
index e73913b..08ff50b 100644
--- a/src/slave/slave.cpp
+++ b/src/slave/slave.cpp
@@ -601,12 +601,14 @@ void Slave::shutdown(const UPID& from, const string& message)
   if (from) {
     LOG(INFO) << "Slave asked to shut down by " << from
               << (message.empty() ? "" : " because '" + message + "'");
-  } else {
+  } else if (info.has_id()) {
     LOG(INFO) << message << "; unregistering and shutting down";
 
     UnregisterSlaveMessage message_;
     message_.mutable_slave_id()->MergeFrom(info.id());
     send(master.get(), message_);
+  } else {
+    LOG(INFO) << message << "; shutting down";
   }
 
   state = TERMINATING;