mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <jos...@mesosphere.io>
Subject Review Request 70740: Windows: Removed multiple categories of sources from the build.
Date Tue, 28 May 2019 23:11:48 GMT

-----------------------------------------------------------
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


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