-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/70546/
-----------------------------------------------------------
Review request for mesos, Andrei Sekretenko, Benno Evers, and Greg Mann.
Bugs: MESOS-9740
https://issues.apache.org/jira/browse/MESOS-9740
Repository: mesos
Description
-------
As part of MESOS-6874, the master validates protobuf unions passed as
part of an ExecutorInfo::ContainerInfo. This prevents a task from
specifying, for example, a ContainerInfo::MESOS, but filling
out the docker field (which is then ignored by the agent).
This validation change is actually an API change, because previously
runnable ExecutorInfo's and TaskInfo's will now fail validation.
This has two visible effects on clusters:
* Agents running containers with invalid protobuf unions will not
be able to reregister with the master.
* Existing frameworks will not be able to re-launch the same tasks
that were working before a Mesos master upgrade.
This changes the validation to print a warning instead. Where possible,
the warning will provide some information to indicate which task
or executor is sending the invalid protobuf.
Diffs
-----
src/common/validation.cpp 458f2258cc5fb76e65e2988dd3ab8bb827b0ac2d
src/master/validation.cpp d7f210fc1ed228113c7f97bce9a43916840b2252
src/tests/master_validation_tests.cpp
c98f7517a1c29eea36f9a3c3da2cda1441967b77
Diff: https://reviews.apache.org/r/70546/diff/1/
Testing
-------
Thanks,
Joseph Wu
