mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <jos...@mesosphere.io>
Subject Re: Review Request 59464: Add Fetcher total and success metrics.
Date Tue, 23 May 2017 01:04:07 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59464/#review175740
-----------------------------------------------------------



What do you think about making fetch failures a sub-metric of `container_launch_errors`? 
The fetcher is only called during container launch.  If fetching fails, container launch will
also fail (and increment this counter).  It might be useful to separate out fetch failures
from other types of launch failures.  This approach might be a bit more work to do so, since
the parent metric lives inside the Agent, rather than the Fetcher process.

- Joseph Wu


On May 22, 2017, 2:15 p.m., James Peach wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59464/
> -----------------------------------------------------------
> 
> (Updated May 22, 2017, 2:15 p.m.)
> 
> 
> Review request for mesos and Mesos Reviewbot.
> 
> 
> Bugs: MESOS-7524
>     https://issues.apache.org/jira/browse/MESOS-7524
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Add the Fetcher metrics to track the number of fetch requests sent to
> the Fetcher (`containerizer/fetcher/total`) and the number of errors
> reported by the Fetcher (`containerizer/fetcher/errors`).
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/fetcher.hpp 9e3018dc087ed55c61b2824d0105bc5339b83043 
>   src/slave/containerizer/fetcher.cpp a910fea5a5556afb376524c5bb2ff98d7d84e611 
> 
> 
> Diff: https://reviews.apache.org/r/59464/diff/2/
> 
> 
> Testing
> -------
> 
> make check (Fedora 25)
> 
> 
> Thanks,
> 
> James Peach
> 
>


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