----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70740/ -----------------------------------------------------------
Review request for mesos, Chun-Hung Hsiao and Gilbert Song. Repository: mesos Description ------- This removes some categories of sources from the Windows build, where it is possible to do so with minimal ifdef-ing. The features removed are all Linux-specific features that cannot be feasibly ported to Windows, including: * Container Storage Interface (CSI) * Docker image provisioning, specifically related to V2 * Open Container Interface * Volume GID Manager Protobufs are excluded where possible, but many of the above categories of protobufs are interleaved with other protobufs or source code, which makes exclusion non-trivial. For example, CSI V0 protobufs cannot be excluded without a large change; or libseccomp is a Linux-only feature, but its protobufs are now required to build the Mesos containerizer's protobufs. Docker image provisioning was semi-trivial to exclude, because the related components (provisioner & URI fetcher) are somewhat modularized. Diffs ----- src/CMakeLists.txt 1d4f541b73c07a307a8b61f217e0cfad5dc095e4 src/slave/containerizer/mesos/containerizer.cpp 043244841a73fa3f5f7119bc38f6d3a04be8990b src/slave/containerizer/mesos/provisioner/store.cpp 11fce0eb47e9e6dfce6289afe04a1d58a0c4461a src/tests/CMakeLists.txt e6b1d8a097246f0921467d7f778034d6754fcff6 src/tests/uri_fetcher_tests.cpp c727cc52e82a396fe187a00c8cc3c9e78a919c5d src/uri/fetcher.hpp cc4bd93b3b8bcb7803f8f912f4ad9d3cf45a58a9 src/uri/fetcher.cpp 8db43eb9763f1cf8040db93a1f03aae0fe9ab3c7 Diff: https://reviews.apache.org/r/70740/diff/1/ Testing ------- cmake --build . --target check This slightly decreases the memory footprint of the build, and allowed my build instance (4GB mem) to proceed beyond some agent files (which is where the Windows CI is also running out of memory). It still ran out of memory when compiling tests however. After giving the instance more memory (8GB), the build succeeds. Thanks, Joseph Wu
