This is an automated email from the ASF dual-hosted git repository.

bennoe pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git


The following commit(s) were added to refs/heads/master by this push:
     new 4a33d2a  Updated comments in v1/mesos.proto.
4a33d2a is described below

commit 4a33d2aba67686098f8384bf3a0e72a8c3eb1f2c
Author: Benno Evers <[email protected]>
AuthorDate: Thu Apr 11 15:58:04 2019 +0200

    Updated comments in v1/mesos.proto.
    
    Updated the comments in `mesos/v1/mesos.proto` to be the same
    as in `mesos/mesos.proto`, after the latter was updated in
    commit 93aca1eb0efcec941e19e976f683a35ecd9a840b.
---
 include/mesos/v1/mesos.proto | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/include/mesos/v1/mesos.proto b/include/mesos/v1/mesos.proto
index 3507909..e8086e0 100644
--- a/include/mesos/v1/mesos.proto
+++ b/include/mesos/v1/mesos.proto
@@ -3326,6 +3326,8 @@ message TTYInfo {
  */
 message ContainerInfo {
   // All container implementation types.
+  // For each type there should be a field in the ContainerInfo itself
+  // with exactly matching name in lowercase.
   enum Type {
     DOCKER = 1;
     MESOS = 2;
@@ -3379,8 +3381,8 @@ message ContainerInfo {
   repeated Volume volumes = 2;
   optional string hostname = 4;
 
-  // Only one of the following *Info messages should be set to match
-  // the type.
+  // At most one of the following *Info messages should be set to match
+  // the type, i.e. the "protobuf union" in ContainerInfo should be valid.
   optional DockerInfo docker = 3;
   optional MesosInfo mesos = 5;
 

Reply via email to