mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Andrew Schwartzmeyer <>
Subject Review Request 61915: Enabled `DockerContainerizerProcess::usage` for all platforms.
Date Fri, 25 Aug 2017 18:10:18 GMT

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

Review request for mesos, Jie Yu, Joseph Wu, and Li Li.

Bugs: MESOS-7917

Repository: mesos


When originally implemented this code was enabled only for Linux.
However, this code should work anywhere Docker is being used, as it
obtains the information by shelling out to the `docker` executable.

This patch fixes the resources information provided by the `mesos-agent`
on Windows platforms for Docker containers by making the `statistics`
JSON object available.

Note that the `cgroupsStatistics` is Linux specific, and so the
pre-processor guard was not removed but reduced in scope for just it.


  src/slave/containerizer/docker.cpp cbcff39a5d8398f71570849615aeb4368130b433 



Built on Windows, started a Windows Docker container, observed the `statistics` field from
`<theagent>/container` being populated where it was not previously included.

Building and running `mesos-tests`, update pending.


Andrew Schwartzmeyer

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