mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gilbert Song <>
Subject Re: Review Request 51669: Changed ContainerConfig::ExecutorInfo from required to optional.
Date Tue, 27 Sep 2016 00:12:58 GMT

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

(Updated Sept. 26, 2016, 5:12 p.m.)

Review request for mesos, Benjamin Hindman, Artem Harutyunyan, Jie Yu, Joseph Wu, and Kevin

Repository: mesos


For nested container launch in containerizer, a new launch method
was introduced for nested container only. However, 'ExecutorInfo'
is not included in this new launch method, only 'CommandInfo'
instead. To construct the 'ContainerConfig' for nested container
launch, we need to update the 'ExecutorInfo' in 'ContainerConfig'
from required to optional.

Diffs (updated)

  include/mesos/slave/containerizer.proto 20db010ea158a813034b411111ce9cddac7d8317 



make check


Gilbert Song

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