mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrei Budnik <>
Subject Review Request 69171: Added validation of cache files to the URI Fetcher.
Date Thu, 25 Oct 2018 20:47:31 GMT

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

Review request for mesos, Gilbert Song, James Peach, and Joseph Wu.

Bugs: MESOS-7474

Repository: mesos


Previously, missing cache files could lead to task launch failures.
This patch introduces validation of cache files which happens each
time a downloaded cache entry is requested via the `get()` method.
If validation fails, `get()` returns `None()`, so that the fetcher
retries downloading URI. Currently, we only check the existence of
the cache file during validation.


  src/slave/containerizer/fetcher.cpp e848c86261b75e5549e80276541e5932162fc012 
  src/slave/containerizer/fetcher_process.hpp 56ed6e5455b7d23b4ce84873fe6734b4213df691 



1. sudo make check (Fedora 25)
2. internal CI


Andrei Budnik

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