mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Peach <jpe...@apache.org>
Subject Review Request 64876: Fixed `getExecutorRunPath` to handle nested ContainerIDs.
Date Sat, 30 Dec 2017 20:05:49 GMT

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

Review request for mesos, Alexander Rukletsov, Anand Mazumdar, and Jie Yu.


Bugs: MESOS-8366
    https://issues.apache.org/jira/browse/MESOS-8366


Repository: mesos


Description
-------

If `getExecutorRunPath` is passed a nested ContainerID, it
naively stringifies it, producing an invalid executor path.
Since the executor is always the root container, we should
stringify only the root ContainerID.


Diffs
-----

  src/slave/paths.cpp 9f8ee391dec0472b6db1840bad691b4ef024ce9c 


Diff: https://reviews.apache.org/r/64876/diff/1/


Testing
-------

make check (Fedora 27)


Thanks,

James Peach


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