> On June 6, 2018, 10:26 a.m., Mesos Reviewbot Windows wrote: > > FAIL: Some of the unit tests failed. Please check the relevant logs. > > > > Reviews applied: `['67357']` > > > > Failed command: `Start-MesosCITesting` > > > > All the build artifacts available at: > > http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/67357 > > > > Relevant logs: > > > > - > > [mesos-tests-stdout.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/67357/logs/mesos-tests-stdout.log): > > > > ``` > > [ RUN ] SlaveTest.ResourceVersions > > [ OK ] SlaveTest.ResourceVersions (214 ms) > > [ RUN ] SlaveTest.ReconfigurationPolicy > > [ OK ] SlaveTest.ReconfigurationPolicy (317 ms) > > [ RUN ] SlaveTest.ResourceProviderReconciliation > > [ OK ] SlaveTest.ResourceProviderReconciliation (358 ms) > > [ RUN ] SlaveTest.RunTaskResourceVersions > > [ OK ] SlaveTest.RunTaskResourceVersions (306 ms) > > [----------] 83 tests from SlaveTest (70519 ms total) > > > > [----------] 3 tests from SlaveStateTest > > [ RUN ] SlaveStateTest.CheckpointString > > [ OK ] SlaveStateTest.CheckpointString (4 ms) > > [ RUN ] SlaveStateTest.CheckpointProtobufMessage > > [ OK ] SlaveStateTest.CheckpointProtobufMessage (9 ms) > > [ RUN ] SlaveStateTest.CheckpointRepeatedProtobufMessages > > [ OK ] SlaveStateTest.CheckpointRepeatedProtobufMessages (10 ms) > > [----------] 3 tests from SlaveStateTest (25 ms total) > > > > [----------] 30 tests from SlaveRecoveryTest/0, where TypeParam = class > > mesos::internal::slave::MesosContainerizer > > [ RUN ] SlaveRecoveryTest/0.RecoverSlaveState > > [ OK ] SlaveRecoveryTest/0.RecoverSlaveState (1573 ms) > > [ RUN ] SlaveRecoveryTest/0.RecoverTaskStatusUpdateManager > > [ OK ] SlaveRecoveryTest/0.RecoverTaskStatusUpdateManager (3328 ms) > > [ RUN ] SlaveRecoveryTest/0.ReconnectExecutor > > [ OK ] SlaveRecoveryTest/0.ReconnectExecutor (3707 ms) > > [ RUN ] SlaveRecoveryTest/0.ReconnectExecutorRetry > > [ OK ] SlaveRecoveryTest/0.ReconnectExecutorRetry (1205 ms) > > [ RUN ] SlaveRecoveryTest/0.PingTimeoutDuringRecovery > > ``` > > > > - > > [mesos-tests-stderr.log](http://dcos-win.westus.cloudapp.azure.com/mesos-build/review/67357/logs/mesos-tests-stderr.log): > > > > ``` > > @ 00007FF71117D888 > > std::invoke<<lambda_9f5bb6c728b761604e288ae85a7b250c>,process::Future<Option<mesos::MasterInfo> > > >,process::ProcessBase *> > > @ 00007FF71119257B > > lambda::internal::Partial<<lambda_9f5bb6c728b761604e288ae85a7b250c>,process::Future<Option<mesos::MasterInfo> > > >,std::_Ph<1> > > >::invoke_expand<<lambda_9f5bb6c728b761604e288ae85a7b250c>,std::tuple<process::Future<Option<mesos::MasterInfo> > > >,std::_Ph<1> >,st > > @ 00007FF7110C08BA )<process::ProcessBase * > > @ 00007FF7110F058C > > std::_Invoker_functor::_Call<lambda::internal::Partial<<lambda_9f5bb6c728b761604e288ae85a7b250c>,process::Future<Option<mesos::MasterInfo> > > >,std::_Ph<1> >,process::ProcessBase *> > > @ 00007FF711183EBC > > std::invoke<lambda::internal::Partial<<lambda_9f5bb6c728b761604e288ae85a7b250c>,process::Future<Option<mesos::MasterInfo> > > >,std::_Ph<1> >,process::ProcessBase *> > > @ 00007FF7110C9F21 > > )<lambda::internal::Partial<<lambda_9f5bb6c728b761604e288ae85a7b250c>,process::Future<Option<mesos::MasterInfo> > > >,std::_Ph<1> >,process::ProcessBase * > > @ 00007FF711236416 process::ProcessBase > > *)>::CallableFn<lambda::internal::Partial<<lambda_9f5bb6c728b761604e288ae85a7b250c>,process::Future<Option<mesos::MasterInfo> > > >,std::_Ph<1> > >::operator( > > @ 00007FF712C1A25D process::ProcessBase *)>::operator( > > @ 00007FF712ACB2F9 process::ProcessBase::consume > > @ 00007FF712C738CA process::DispatchEvent::consume > > @ 00007FF70ECE7B07 process::ProcessBase::serve > > @ 00007FF712AD93B0 process::ProcessManager::resume > > @ 00007FF712C07371 ?? > > @ 00007FF712B2B130 > > std::_Invoker_functor::_Call<<lambda_124422ac022fa041208b80c1460630d7> > > > @ 00007FF712B8B8E0 > > std::invoke<<lambda_124422ac022fa041208b80c1460630d7> > > > @ 00007FF712B4076C > > std::_LaunchPad<std::unique_ptr<std::tuple<<lambda_124422ac022fa041208b80c1460630d7> > > >,std::default_delete<std::tuple<<lambda_124422ac022fa041208b80c1460630d7> > > > > > >::_Execute<0> > > @ 00007FF712C5A60A > > std::_LaunchPad<std::unique_ptr<std::tuple<<lambda_124422ac022fa041208b80c1460630d7> > > >,std::default_delete<std::tuple<<lambda_124422ac022fa041208b80c1460630d7> > > > > > >::_Run > > @ 00007FF712C45E78 > > std::_LaunchPad<std::unique_ptr<std::tuple<<lambda_124422ac022fa041208b80c1460630d7> > > >,std::default_delete<std::tuple<<lambda_124422ac022fa041208b80c1460630d7> > > > > > >::_Go > > @ 00007FF712C2C3CD std::_Pad::_Call_func > > @ 00007FFF9BE53428 _register_onexit_function > > @ 00007FFF9BE53071 _register_onexit_function > > @ 00007FFFB6391FE4 BaseThreadInitThunk > > @ 00007FFFB69FF061 RtlUserThreadStart > > ll containerizers > > I0606 10:25:26.680230 18356 slave.cpp:7158] Recovering executors > > I0606 10:25:26.680230 18356 slave.cpp:7182] Sending reconnect request to > > executor '3f11d255-bb7b-4e99-967b-055fef95b595' of framework > > 62cf792a-dc69-4e3c-b54f-d83f98fb9451-0000 at executor(1)@192.10.1.5:55652 > > I0606 10:25:26.688225 22560 slave.cpp:4984] Received re-registration > > message from executor '3f11d255-bb7b-4e99-967b-055fef95b595' of framework > > 62cf792a-dc69-4e3c-b54f-d83f98fb9451-0000 > > I0606 10:25:26.691216 22888 slave.cpp:5901] No pings from master received > > within 75secs > > F0606 10:25:26.692219 22888 slave.cpp:1249] Check failed: state == > > DISCONNECTED || state == RUNNING || state == TERMINATING RECOVERING > > ```
WTF is this... Can you please check the JIRA and file an issue if it is a new one? - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67357/#review204374 ----------------------------------------------------------- On June 6, 2018, 8:54 a.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67357/ > ----------------------------------------------------------- > > (Updated June 6, 2018, 8:54 a.m.) > > > Review request for Alexander Rukletsov. > > > Repository: mesos > > > Description > ------- > > A vulnerability in our JWT implementation allows an unauthenticated > remote attacker to execute to execute timing attacks [1]. > > This patch removes the vulnerability by adding a constant time > comparison of hashes, where the whole message is visited during > the comparison instead of returning at the first failure. > > [1] https://codahale.com/a-lesson-in-timing-attacks/ > > > Diffs > ----- > > 3rdparty/libprocess/src/jwt.cpp 4477ddd17dede2b924a47e33942b39244f10316f > > > Diff: https://reviews.apache.org/r/67357/diff/3/ > > > Testing > ------- > > ```sh > make check > ``` > > > Thanks, > > Alexander Rojas > >