mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexander Rojas <>
Subject Re: Review Request 42648: Moved http authenticator initialization to main.
Date Wed, 27 Jan 2016 18:50:52 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Jan. 27, 2016, 7:50 p.m.)

Review request for mesos, Alexander Rukletsov, Benjamin Bannier, Jan Schlicht, and Till Toenshoff.


Actually removes functionality from `mesos::Master`

Repository: mesos

Description (updated)

Moves the code which initializes instances of `process::http::authentication::Authenticator`
from `mesos::internal::Master` to the master main file.

The change is needed in order to show that libprocess http authentication is turned at the
system process level and not at the libprocess process level.

It also adds the same initialization as in main to the test function `StartMaster`.

Diffs (updated)

  src/master/main.cpp 0d5ac49121491152d8426cd764a1f1a0f37483ae 
  src/master/master.cpp 9ee56277c8a472be9e683d5db505becfb5f7c422 
  src/tests/cluster.cpp 1a3a038e78999bca2fc3f08725f4ffaf4290bcb9 



make check

manual tests.


Alexander Rojas

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message