You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by al...@apache.org on 2017/11/15 15:34:42 UTC

mesos git commit: Added context to new agent processing calls.

Repository: mesos
Updated Branches:
  refs/heads/master 69e0fd908 -> 71c2d6ebf


Added context to new agent processing calls.

This change adds context information to log entries for agent
HTTP API handlers. It impacts the calls `LAUNCH_CONTAINER`,
`WAIT_CONTAINER`, `KILL_CONTAINER`, and `REMOVE_CONTAINER`.

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


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

Branch: refs/heads/master
Commit: 71c2d6ebfbf0f1b3d6faa765a37b81f1a8c0c293
Parents: 69e0fd9
Author: Armand Grillet <ag...@mesosphere.io>
Authored: Wed Nov 15 16:34:08 2017 +0100
Committer: Alexander Rukletsov <al...@apache.org>
Committed: Wed Nov 15 16:34:08 2017 +0100

----------------------------------------------------------------------
 src/slave/http.cpp | 12 ++++++++++++
 1 file changed, 12 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/71c2d6eb/src/slave/http.cpp
----------------------------------------------------------------------
diff --git a/src/slave/http.cpp b/src/slave/http.cpp
index c4d8098..394e910 100644
--- a/src/slave/http.cpp
+++ b/src/slave/http.cpp
@@ -2392,6 +2392,9 @@ Future<Response> Http::launchContainer(
   CHECK_EQ(mesos::agent::Call::LAUNCH_CONTAINER, call.type());
   CHECK(call.has_launch_container());
 
+  LOG(INFO) << "Processing LAUNCH_CONTAINER call for container '"
+            << call.launch_container().container_id() << "'";
+
   Future<Owned<AuthorizationAcceptor>> authorizer =
     AuthorizationAcceptor::create(
         principal,
@@ -2611,6 +2614,9 @@ Future<Response> Http::waitContainer(
   CHECK_EQ(mesos::agent::Call::WAIT_CONTAINER, call.type());
   CHECK(call.has_wait_container());
 
+  LOG(INFO) << "Processing WAIT_CONTAINER call for container '"
+            << call.wait_container().container_id() << "'";
+
   Future<Owned<AuthorizationAcceptor>> authorizer =
     AuthorizationAcceptor::create(
         principal,
@@ -2772,6 +2778,9 @@ Future<Response> Http::killContainer(
   CHECK_EQ(mesos::agent::Call::KILL_CONTAINER, call.type());
   CHECK(call.has_kill_container());
 
+  LOG(INFO) << "Processing KILL_CONTAINER call for container '"
+            << call.kill_container().container_id() << "'";
+
   Future<Owned<AuthorizationAcceptor>> authorizer =
     AuthorizationAcceptor::create(
         principal,
@@ -2874,6 +2883,9 @@ Future<Response> Http::removeContainer(
   CHECK_EQ(mesos::agent::Call::REMOVE_CONTAINER, call.type());
   CHECK(call.has_remove_container());
 
+  LOG(INFO) << "Processing REMOVE_CONTAINER call for container '"
+            << call.remove_container().container_id() << "'";
+
   Future<Owned<AuthorizationAcceptor>> authorizer =
     AuthorizationAcceptor::create(
         principal,