mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Mann <>
Subject Re: Review Request 55955: Added validation tests to ensure environment variable value is set.
Date Thu, 26 Jan 2017 02:36:45 GMT

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

(Updated Jan. 26, 2017, 2:36 a.m.)

Review request for mesos, Jan Schlicht and Vinod Kone.

Bugs: MESOS-6991

Repository: mesos


The `value` field within `Environment::Variable` is being
changed to `optional`, but for the time being we will
enforce that it must be set for backward compatibility.
This patch adds tests to ensure that environment variables
with unset values are correctly rejected.

Diffs (updated)

  src/tests/check_tests.cpp c88cd34fd214f111cff62591aa5fc03eb62567e4 
  src/tests/health_check_tests.cpp debbd3c09b7555145aaf3f62a24d795d1423a269 
  src/tests/master_validation_tests.cpp edb57407e08cdbd8fbf10a9e1493cab3b4979bb8 
  src/tests/slave_validation_tests.cpp 5de771114982751e7796f55dcacd4384c6989efb 



bin/ --gtest_filter="*Validation*"


Greg Mann

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