You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Gilbert Song <so...@gmail.com> on 2015/12/17 03:38:56 UTC
Review Request 41487: Provisioner: Changed docker v2 manifest naming.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/
-----------------------------------------------------------
Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
Bugs: MESOS-4186
https://issues.apache.org/jira/browse/MESOS-4186
Repository: mesos
Description
-------
Provisioner: Changed docker v2 manifest naming.
Diffs
-----
src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
Diff: https://reviews.apache.org/r/41487/diff/
Testing
-------
make check
Thanks,
Gilbert Song
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Gilbert Song <so...@gmail.com>.
> On Dec. 17, 2015, 11:46 a.m., Jie Yu wrote:
> > Since you're on it. Can you rename docker::DockerImageManifest to docker::v2::ImageManifest?
Done.
- Gilbert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/#review111024
-----------------------------------------------------------
On Dec. 18, 2015, 1:19 a.m., Gilbert Song wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41487/
> -----------------------------------------------------------
>
> (Updated Dec. 18, 2015, 1:19 a.m.)
>
>
> Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
>
>
> Bugs: MESOS-4186
> https://issues.apache.org/jira/browse/MESOS-4186
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Provisioner: Changed docker v2 manifest naming.
>
>
> Diffs
> -----
>
> src/Makefile.am 9762f8567e32d70b8df2d694a1fef5c692fc730e
> src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
> src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
> src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
> src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
> src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
> src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
> src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
> src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
> src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
>
> Diff: https://reviews.apache.org/r/41487/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Jie Yu <yu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/#review111024
-----------------------------------------------------------
Since you're on it. Can you rename docker::DockerImageManifest to docker::v2::ImageManifest?
- Jie Yu
On Dec. 17, 2015, 9:18 a.m., Gilbert Song wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41487/
> -----------------------------------------------------------
>
> (Updated Dec. 17, 2015, 9:18 a.m.)
>
>
> Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
>
>
> Bugs: MESOS-4186
> https://issues.apache.org/jira/browse/MESOS-4186
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Provisioner: Changed docker v2 manifest naming.
>
>
> Diffs
> -----
>
> src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
> src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
> src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
> src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
> src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
> src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
> src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
>
> Diff: https://reviews.apache.org/r/41487/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Gilbert Song <so...@gmail.com>.
> On Dec. 18, 2015, 9:16 a.m., Jie Yu wrote:
> > src/slave/containerizer/mesos/provisioner/docker/spec.hpp, line 35
> > <https://reviews.apache.org/r/41487/diff/3/?file=1170682#file1170682line35>
> >
> > s/validateManifest/validate/
This is fixed in https://reviews.apache.org/r/41489/
> On Dec. 18, 2015, 9:16 a.m., Jie Yu wrote:
> > src/slave/containerizer/mesos/provisioner/docker/spec.cpp, line 33
> > <https://reviews.apache.org/r/41487/diff/3/?file=1170683#file1170683line33>
> >
> > Ditto.
Ditto.
- Gilbert
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/#review111204
-----------------------------------------------------------
On Dec. 18, 2015, 1:19 a.m., Gilbert Song wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41487/
> -----------------------------------------------------------
>
> (Updated Dec. 18, 2015, 1:19 a.m.)
>
>
> Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
>
>
> Bugs: MESOS-4186
> https://issues.apache.org/jira/browse/MESOS-4186
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Provisioner: Changed docker v2 manifest naming.
>
>
> Diffs
> -----
>
> src/Makefile.am 9762f8567e32d70b8df2d694a1fef5c692fc730e
> src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
> src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
> src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
> src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
> src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
> src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
> src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
> src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
> src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
>
> Diff: https://reviews.apache.org/r/41487/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Jie Yu <yu...@gmail.com>.
> On Dec. 18, 2015, 5:16 p.m., Jie Yu wrote:
> > src/slave/containerizer/mesos/provisioner/docker/spec.hpp, line 35
> > <https://reviews.apache.org/r/41487/diff/3/?file=1170682#file1170682line35>
> >
> > s/validateManifest/validate/
>
> Gilbert Song wrote:
> This is fixed in https://reviews.apache.org/r/41489/
Cool, I saw you did that in the subsequent patch.
- Jie
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/#review111204
-----------------------------------------------------------
On Dec. 18, 2015, 9:19 a.m., Gilbert Song wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41487/
> -----------------------------------------------------------
>
> (Updated Dec. 18, 2015, 9:19 a.m.)
>
>
> Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
>
>
> Bugs: MESOS-4186
> https://issues.apache.org/jira/browse/MESOS-4186
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Provisioner: Changed docker v2 manifest naming.
>
>
> Diffs
> -----
>
> src/Makefile.am 9762f8567e32d70b8df2d694a1fef5c692fc730e
> src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
> src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
> src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
> src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
> src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
> src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
> src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
> src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
> src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
>
> Diff: https://reviews.apache.org/r/41487/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Jie Yu <yu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/#review111204
-----------------------------------------------------------
Ship it!
src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp (line 73)
<https://reviews.apache.org/r/41487/#comment171305>
No need to do this in this patch. Let rename this method to be 'getImageManifest'
src/slave/containerizer/mesos/provisioner/docker/spec.hpp (line 35)
<https://reviews.apache.org/r/41487/#comment171303>
s/validateManifest/validate/
src/slave/containerizer/mesos/provisioner/docker/spec.cpp (line 33)
<https://reviews.apache.org/r/41487/#comment171304>
Ditto.
- Jie Yu
On Dec. 18, 2015, 9:19 a.m., Gilbert Song wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41487/
> -----------------------------------------------------------
>
> (Updated Dec. 18, 2015, 9:19 a.m.)
>
>
> Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
>
>
> Bugs: MESOS-4186
> https://issues.apache.org/jira/browse/MESOS-4186
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Provisioner: Changed docker v2 manifest naming.
>
>
> Diffs
> -----
>
> src/Makefile.am 9762f8567e32d70b8df2d694a1fef5c692fc730e
> src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
> src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
> src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
> src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
> src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
> src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
> src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
> src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
> src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
>
> Diff: https://reviews.apache.org/r/41487/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Gilbert Song <so...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/
-----------------------------------------------------------
(Updated Dec. 18, 2015, 1:31 p.m.)
Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
Bugs: MESOS-4186
https://issues.apache.org/jira/browse/MESOS-4186
Repository: mesos
Description
-------
Provisioner: Changed docker v2 manifest naming.
Diffs (updated)
-----
src/Makefile.am e6d48dc16135b5d147d036e851422686eff7d5ef
src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
src/tests/containerizer/provisioner_docker_tests.cpp b57ee8681e0861cf93f5dcf9368c004a57df1374
Diff: https://reviews.apache.org/r/41487/diff/
Testing
-------
make check
Thanks,
Gilbert Song
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Gilbert Song <so...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/
-----------------------------------------------------------
(Updated Dec. 18, 2015, 12:10 p.m.)
Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
Bugs: MESOS-4186
https://issues.apache.org/jira/browse/MESOS-4186
Repository: mesos
Description
-------
Provisioner: Changed docker v2 manifest naming.
Diffs (updated)
-----
src/Makefile.am 9762f8567e32d70b8df2d694a1fef5c692fc730e
src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
Diff: https://reviews.apache.org/r/41487/diff/
Testing
-------
make check
Thanks,
Gilbert Song
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Gilbert Song <so...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/
-----------------------------------------------------------
(Updated Dec. 18, 2015, 1:19 a.m.)
Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
Bugs: MESOS-4186
https://issues.apache.org/jira/browse/MESOS-4186
Repository: mesos
Description
-------
Provisioner: Changed docker v2 manifest naming.
Diffs (updated)
-----
src/Makefile.am 9762f8567e32d70b8df2d694a1fef5c692fc730e
src/slave/containerizer/mesos/provisioner/docker/message.hpp 162e4c689bba832e523ff6b7d4e1e3c8e6713803
src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
src/slave/containerizer/mesos/provisioner/docker/v2.proto PRE-CREATION
src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
Diff: https://reviews.apache.org/r/41487/diff/
Testing
-------
make check
Thanks,
Gilbert Song
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Gilbert Song <so...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/
-----------------------------------------------------------
(Updated Dec. 17, 2015, 1:18 a.m.)
Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
Bugs: MESOS-4186
https://issues.apache.org/jira/browse/MESOS-4186
Repository: mesos
Description
-------
Provisioner: Changed docker v2 manifest naming.
Diffs (updated)
-----
src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
Diff: https://reviews.apache.org/r/41487/diff/
Testing
-------
make check
Thanks,
Gilbert Song
Re: Review Request 41487: Provisioner: Changed docker v2 manifest
naming.
Posted by Timothy Chen <tn...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41487/#review110909
-----------------------------------------------------------
Ship it!
Ship It!
src/tests/containerizer/provisioner_docker_tests.cpp (line 303)
<https://reviews.apache.org/r/41487/#comment170906>
Let's make the naming consistent -> SerializeV2DockerManifest
src/tests/containerizer/provisioner_docker_tests.cpp (line 484)
<https://reviews.apache.org/r/41487/#comment170907>
ditto
- Timothy Chen
On Dec. 17, 2015, 2:38 a.m., Gilbert Song wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41487/
> -----------------------------------------------------------
>
> (Updated Dec. 17, 2015, 2:38 a.m.)
>
>
> Review request for mesos, Artem Harutyunyan, Jie Yu, and Timothy Chen.
>
>
> Bugs: MESOS-4186
> https://issues.apache.org/jira/browse/MESOS-4186
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Provisioner: Changed docker v2 manifest naming.
>
>
> Diffs
> -----
>
> src/slave/containerizer/mesos/provisioner/docker/message.proto 5c032701671b275d86c6d9276791a46df814396c
> src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 9b02b6ff6dc5c6e8aabdc4ac0aa4df337764ef30
> src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp e69bab43d5f5359cfd8eb4cb7c5ad4a1d22c4e05
> src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp 89f61c20e52e5eff8d8e92748f03b3b461516cd2
> src/slave/containerizer/mesos/provisioner/docker/spec.hpp e674b6f2e705f8b3e49770560eeab4c127473f94
> src/slave/containerizer/mesos/provisioner/docker/spec.cpp 1f05c75dc9473bd5e4c0d3f74fa0ef996b96a84e
> src/tests/containerizer/provisioner_docker_tests.cpp 3f1717b770e139c3759aab0aeda9dbcf5029b0c2
>
> Diff: https://reviews.apache.org/r/41487/diff/
>
>
> Testing
> -------
>
> make check
>
>
> Thanks,
>
> Gilbert Song
>
>