-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18381/#review40253
-----------------------------------------------------------
Bad patch!
Reviews applied: [20302, 20301, 18381]
Failed command: make -j3 check GTEST_FILTER='' >/dev/null
Error:
ev.c:1531:31: warning: 'ev_default_loop_ptr' initialized and declared 'extern'
[enabled by default]
ev.c: In function 'evpipe_write':
ev.c:2160:17: warning: ignoring return value of 'write', declared with
attribute warn_unused_result [-Wunused-result]
ev.c:2172:17: warning: ignoring return value of 'write', declared with
attribute warn_unused_result [-Wunused-result]
ev.c: In function 'pipecb':
ev.c:2193:16: warning: ignoring return value of 'read', declared with attribute
warn_unused_result [-Wunused-result]
ev.c:2207:16: warning: ignoring return value of 'read', declared with attribute
warn_unused_result [-Wunused-result]
In file included from /usr/include/c++/4.6/ext/hash_set:61:0,
from src/glog/stl_logging.h:54,
from src/stl_logging_unittest.cc:34:
/usr/include/c++/4.6/backward/backward_warning.h:33:2: warning: #warning This
file includes at least one deprecated or antiquated header which may be removed
without further notice at a future date. Please use a non-deprecated interface
with equivalent functionality instead. For a listing of replacement headers and
interfaces, consult the file backward_warning.h. To disable this warning use
-Wno-deprecated. [-Wcpp]
In file included from src/utilities.h:73:0,
from src/googletest.h:38,
from src/stl_logging_unittest.cc:48:
src/base/mutex.h:137:0: warning: "_XOPEN_SOURCE" redefined [enabled by default]
/usr/include/features.h:166:0: note: this is the location of the previous
definition
warning: no files found matching 'Makefile' under directory 'docs'
warning: no files found matching 'indexsidebar.html' under directory 'docs'
zip_safe flag not set; analyzing archive contents...
WARNING: '.' not a valid package name; please use only.-separated package names
in setup.py
package init file 'src/__init__.py' not found (or not a regular file)
cc1plus: warning: command line option '-Wstrict-prototypes' is valid for
Ada/C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option '-Wstrict-prototypes' is valid for
Ada/C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option '-Wstrict-prototypes' is valid for
Ada/C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option '-Wstrict-prototypes' is valid for
Ada/C/ObjC but not for C++ [enabled by default]
cc1plus: warning: command line option '-Wstrict-prototypes' is valid for
Ada/C/ObjC but not for C++ [enabled by default]
zip_safe flag not set; analyzing archive contents...
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
tests/authentication_tests.cpp: In member function 'virtual void
AuthenticationTest_MasterFailoverDuringSlaveAuthentication_Test::TestBody()':
tests/authentication_tests.cpp:530:59: error: no matching function for call to
'AuthenticationTest_MasterFailoverDuringSlaveAuthentication_Test::StartSlave(process::Owned<mesos::internal::MasterDetector>,
mesos::internal::slave::Flags&)'
tests/authentication_tests.cpp:530:59: note: candidates are:
./tests/mesos.hpp:100:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(const
Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:100:44: note: candidate expects 1 argument, 2 provided
./tests/mesos.hpp:104:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::tests::MockExecutor*,
const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:104:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::tests::MockExecutor*'
./tests/mesos.hpp:109:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::slave::Containerizer*,
const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:109:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::slave::Containerizer*'
./tests/mesos.hpp:114:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::slave::Containerizer*,
mesos::internal::MasterDetector*, const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:114:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::slave::Containerizer*'
./tests/mesos.hpp:120:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::MasterDetector*,
const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:120:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::MasterDetector*'
./tests/mesos.hpp:126:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::tests::MockExecutor*,
mesos::internal::MasterDetector*, const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:126:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::tests::MockExecutor*'
tests/authentication_tests.cpp: In member function 'virtual void
AuthenticationTest_LeaderElectionDuringSlaveAuthentication_Test::TestBody()':
tests/authentication_tests.cpp:610:59: error: no matching function for call to
'AuthenticationTest_LeaderElectionDuringSlaveAuthentication_Test::StartSlave(process::Owned<mesos::internal::MasterDetector>,
mesos::internal::slave::Flags&)'
tests/authentication_tests.cpp:610:59: note: candidates are:
./tests/mesos.hpp:100:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(const
Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:100:44: note: candidate expects 1 argument, 2 provided
./tests/mesos.hpp:104:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::tests::MockExecutor*,
const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:104:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::tests::MockExecutor*'
./tests/mesos.hpp:109:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::slave::Containerizer*,
const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:109:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::slave::Containerizer*'
./tests/mesos.hpp:114:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::slave::Containerizer*,
mesos::internal::MasterDetector*, const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:114:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::slave::Containerizer*'
./tests/mesos.hpp:120:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::MasterDetector*,
const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:120:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::MasterDetector*'
./tests/mesos.hpp:126:44: note: virtual
Try<process::PID<mesos::internal::slave::Slave> >
mesos::internal::tests::MesosTest::StartSlave(mesos::internal::tests::MockExecutor*,
mesos::internal::MasterDetector*, const Option<mesos::internal::slave::Flags>&)
./tests/mesos.hpp:126:44: note: no known conversion for argument 1 from
'process::Owned<mesos::internal::MasterDetector>' to
'mesos::internal::tests::MockExecutor*'
make[3]: *** [tests/mesos_tests-authentication_tests.o] Error 1
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [check-am] Error 2
make[1]: *** [check] Error 2
make: *** [check-recursive] Error 1
- Mesos ReviewBot
On April 14, 2014, 6:34 a.m., Adam B wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18381/
> -----------------------------------------------------------
>
> (Updated April 14, 2014, 6:34 a.m.)
>
>
> Review request for mesos and Vinod Kone.
>
>
> Bugs: MESOS-804
> https://issues.apache.org/jira/browse/MESOS-804
>
>
> Repository: mesos-git
>
>
> Description
> -------
>
> Added authentication support for slaves.
> Fixes MESOS-804.
>
>
> Diffs
> -----
>
> include/mesos/mesos.proto 37f8a7f
> src/master/flags.hpp 024f86d
> src/master/master.hpp fef59c9
> src/master/master.cpp 3c3c989
> src/sasl/authenticatee.hpp 42a4eba
> src/slave/flags.hpp d5c54c0
> src/slave/slave.hpp 08f6005
> src/slave/slave.cpp cddb241
> src/tests/authentication_tests.cpp 127c5e6
> src/tests/mesos.cpp ae3aeee
> src/tests/slave_recovery_tests.cpp 72b6d42
>
> Diff: https://reviews.apache.org/r/18381/diff/
>
>
> Testing
> -------
>
> make check; manually tested flatfile slave authentication success/failure.
> Added new slave authentication unit tests in authentication_tests.cpp.
>
>
> Thanks,
>
> Adam B
>
>
