mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ben Mahler" <>
Subject Review Request 41178: Fixed a message dropping bug in the health checker.
Date Thu, 10 Dec 2015 02:01:07 GMT

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

Review request for mesos, Artem Harutyunyan and Timothy Chen.

Bugs: MESOS-1613 and MESOS-4106

Repository: mesos


Much like in the command executor, we need to sleep after we send
the final message in the health checker. Otherwise, we may exit
before libprocess is able to finish sending the message over the
local network.

This led to the following issues:


  src/health-check/main.cpp 83ee38cd853325b3adc7cb6bc2d1d67b343037f5 
  src/tests/health_check_tests.cpp b1454b085b36bb7c4d8ef012c764cd8466b4fb02 



Running the `HealthCheckTest.DISABLED_ConsecutiveFailures` test in repetition on a machine
loaded with many `openssl speed` commands in the background reproduces the flakiness. After
this patch it is no longer flaky in this setup.


Ben Mahler

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