mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <>
Subject Re: Review Request 40512: Libprocess Reinit: Add a test-only method to reinitialize libprocess.
Date Thu, 14 Jul 2016 00:21:24 GMT

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

(Updated July 13, 2016, 5:21 p.m.)

Review request for mesos, Artem Harutyunyan and Joris Van Remoortere.


Rebase and revive the chain.  This does not address any comments.

Bugs: MESOS-3820

Repository: mesos


This builds upon earlier changes to complete `process::finalize`.  
Tests which need to reconfigure libprocess, such as some SSL-related 
tests, can use `process::reinitialize` to do so.  

This method may also be useful for providing additional isolation 
between tests, such as cleaning up all processes after each test case.

Diffs (updated)

  3rdparty/libprocess/src/openssl.cpp 63916ff66b4daa29120b7e6b12b329b68f746694 
  3rdparty/libprocess/src/process.cpp 9661386afd4fddd1877d55941fa403afc9230280 


Testing (updated)

TODO: Define `process::reinitialize` in several tests, such as `HTTPTest`, `MetricsTest`,
and `ReapTest` and call `process::reinitialize` before and after tests.


Joseph Wu

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