You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by gi...@apache.org on 2019/04/05 05:56:02 UTC
[mesos] 05/08: Added a TODO for additional URLs support.
This is an automated email from the ASF dual-hosted git repository.
gilbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git
commit 0c6975345cf647838240f90ad97c0b7b49a9bba1
Author: Gilbert Song <so...@gmail.com>
AuthorDate: Sat Mar 23 21:14:15 2019 -0700
Added a TODO for additional URLs support.
Review: https://reviews.apache.org/r/70289
---
src/uri/fetchers/docker.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/uri/fetchers/docker.cpp b/src/uri/fetchers/docker.cpp
index e11e54f..631db2d 100644
--- a/src/uri/fetchers/docker.cpp
+++ b/src/uri/fetchers/docker.cpp
@@ -968,6 +968,8 @@ Future<Nothing> DockerFetcherPluginProcess::urlFetchBlob(
return Failure("Schema 2 manifest does not exist");
}
+ // TODO(gilbert): Support v2s2 additional urls for non-windows platforms.
+ // We should avoid parsing the manifest for each layer.
Try<spec::v2_2::ImageManifest> manifest = spec::v2_2::parse(_manifest.get());
if (manifest.isError()) {
return Failure(