[jira] [Updated] (MESOS-5354) Update "driver" as optional for DockerVolume.

["Guangya Liu (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/MESOS-5354?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Guangya Liu updated MESOS-5354:
-------------------------------
    Description: 
After some test with docker API, I found that when "docker run" to create a container, the volume name is required but volume driver is optional. When using "dvdcli", both name and driver are required. We are now defining the "driver" as required, we should update "driver" to optional so that the DockerContainerizer still works even if user did not specify driver when creating a container with volume.

{code}
message DockerVolume {
      // Driver of the volume, it can be flocker, convoy, raxrey etc.
      required string driver = 1;  <<<<<<<<<Update this to optional

      // Name of the volume.
      required string name = 2;

      // Volume driver specific options.
      optional Parameters driver_options = 3;
    }
{code}


  was:
After some test with docker API, I found that when "docker run" to create a container, the volume name is required but volume driver is optional. When using "dvdcli", both name and driver are required. We are now defining the "driver" as required, we should update "driver" to optional so that the DockerContainerizer still works even if user did not specify driver when creating a container with volume.

{code}
message DockerVolume {
      // Driver of the volume, it can be flocker, convoy, raxrey etc.
      required string driver = 1;  <<<<<<<<< Shall we update this to optional?

      // Name of the volume.
      required string name = 2;

      // Volume driver specific options.
      optional Parameters driver_options = 3;
    }
{code}



> Update "driver" as optional for DockerVolume.
> ---------------------------------------------
>
>                 Key: MESOS-5354
>                 URL: https://issues.apache.org/jira/browse/MESOS-5354
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Guangya Liu
>            Assignee: Guangya Liu
>
> After some test with docker API, I found that when "docker run" to create a container, the volume name is required but volume driver is optional. When using "dvdcli", both name and driver are required. We are now defining the "driver" as required, we should update "driver" to optional so that the DockerContainerizer still works even if user did not specify driver when creating a container with volume.
> {code}
> message DockerVolume {
>       // Driver of the volume, it can be flocker, convoy, raxrey etc.
>       required string driver = 1;  <<<<<<<<<Update this to optional
>       // Name of the volume.
>       required string name = 2;
>       // Volume driver specific options.
>       optional Parameters driver_options = 3;
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)