You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by James Peach <jp...@apache.org> on 2017/10/12 15:57:06 UTC
Re: Review Request 62518: Fixed default executor handling of nested
container status.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62518/
-----------------------------------------------------------
(Updated Oct. 12, 2017, 3:57 p.m.)
Review request for mesos, Andrei Budnik and Anand Mazumdar.
Bugs: MESOS-8080
https://issues.apache.org/jira/browse/MESOS-8080
Repository: mesos
Description
-------
The default executor was not handling a missing nested container
exit status correctly. It was assuming the protobuf accessor was
returning an Option rather than explicitly checking whether the
`exit_status` field was present in the message.
Added the explicit check for the `exit_status` field, and always
propagated an appropriate message into the status update, even
when the `exit_status` is absent. Added some documentation of
the `exit_status` field to the protobuf definition files.
Diffs
-----
include/mesos/agent/agent.proto 7c8c8a7d8298e91e4e002327b3b27d4c74b5cbae
include/mesos/v1/agent/agent.proto 3e199124b23fa027232790d99370fe2f33660096
src/launcher/default_executor.cpp 136c000917b2c4fdaf68fd460764c3a15fdf3bf0
Diff: https://reviews.apache.org/r/62518/diff/4/
Testing
-------
make check (Fedora 26)
Thanks,
James Peach