You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Kevin Klues <kl...@gmail.com> on 2018/03/01 23:24:04 UTC
Re: Review Request 64211: Added options to build the Python CLI and
run unit tests.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64211/#review198482
-----------------------------------------------------------
Looks like this had errors on the Windows CI?
- Kevin Klues
On Feb. 21, 2018, 9:58 a.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64211/
> -----------------------------------------------------------
>
> (Updated Feb. 21, 2018, 9:58 a.m.)
>
>
> Review request for mesos, Benjamin Bannier and Kevin Klues.
>
>
> Bugs: MESOS-8240
> https://issues.apache.org/jira/browse/MESOS-8240
>
>
> Repository: mesos
>
>
> Description
> -------
>
> An update of the discarded review /r/52543.
>
> Works with Autotools and CMake.
>
>
> Diffs
> -----
>
> CMakeLists.txt 6702f02245e3867c06bbd9efbbf4e3b961a7d9aa
> cmake/CompilationConfigure.cmake 50cddf9476c8c5196c4824a7b060c2680a96b277
> configure.ac 30fbadc32d1d96f719d45fa8067f975283c25507
> docs/configuration/cmake.md 1e34657ea55fa324f65f865f7d0a67084c6719d9
> src/CMakeLists.txt 0c135034982a6a4844509cd0e9b2cc41804333ef
> src/Makefile.am a2ba1e6de0868c7a4fe58304ec479dce039bc288
> src/python/cli_new/CMakeLists.txt PRE-CREATION
> src/python/cli_new/tests/CMakeLists.txt PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/64211/diff/13/
>
>
> Testing
> -------
>
> Testing done on Fedora 25.
>
> For Autotools:
> ```
> $ ./bootstrap
> $ mkdir build
> $ cd build
> $ ../configure --enable-new-cli --disable-java --disable-python
> $ make check
> ```
>
> For CMake:
> ```
> $ ./bootstrap
> $ mkdir build
> $ cd build
> $ cmake .. -DENABLE_NEW_CLI=1
> $ cmake --build . -- -j16
> $ ./src/mesos
> Mesos CLI
>
> Usage:
> mesos (-h | --help)
> mesos --version
> mesos <command> [<args>...]
>
> Options:
> -h --help Show this screen.
> --version Show version info.
>
> Commands:
> agent Interacts with the Mesos agents
> config Interacts with the Mesos CLI configuration file
> task Interacts with the tasks running in a Mesos cluster
>
> See 'mesos help <command>' for more information on a specific command.
> $ cmake --build . --target tests -- -j16
> $ ctest -R CLI
> Test project /home/agrillet/apache-mesos/build
> Start 4: CLITests
> 1/1 Test #4: CLITests ......................... Passed 3.63 sec
>
> 100% tests passed, 0 tests failed out of 1
> ```
>
> Checked that the the CLI tests were run, that the content of the directory `build/src/cli` was as expected, and that `build/src/mesos` was correctly running.
>
>
> Thanks,
>
> Armand Grillet
>
>
Re: Review Request 64211: Added options to build the Python CLI and
run unit tests.
Posted by Armand Grillet <ag...@mesosphere.io>.
> On March 1, 2018, 11:24 p.m., Kevin Klues wrote:
> > Looks like this had errors on the Windows CI?
Following today's rebase, both CIs are successful.
- Armand
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64211/#review198482
-----------------------------------------------------------
On March 2, 2018, 8:12 a.m., Armand Grillet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64211/
> -----------------------------------------------------------
>
> (Updated March 2, 2018, 8:12 a.m.)
>
>
> Review request for mesos, Benjamin Bannier and Kevin Klues.
>
>
> Bugs: MESOS-8240
> https://issues.apache.org/jira/browse/MESOS-8240
>
>
> Repository: mesos
>
>
> Description
> -------
>
> An update of the discarded review /r/52543.
>
> Works with Autotools and CMake.
>
>
> Diffs
> -----
>
> CMakeLists.txt 6702f02245e3867c06bbd9efbbf4e3b961a7d9aa
> cmake/CompilationConfigure.cmake 50cddf9476c8c5196c4824a7b060c2680a96b277
> configure.ac 30fbadc32d1d96f719d45fa8067f975283c25507
> docs/configuration/cmake.md 1e34657ea55fa324f65f865f7d0a67084c6719d9
> src/CMakeLists.txt 0c135034982a6a4844509cd0e9b2cc41804333ef
> src/Makefile.am 84753794eca822ab251200cccb907b328c849fb7
> src/python/cli_new/CMakeLists.txt PRE-CREATION
> src/python/cli_new/tests/CMakeLists.txt PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/64211/diff/14/
>
>
> Testing
> -------
>
> Testing done on Fedora 25.
>
> For Autotools:
> ```
> $ ./bootstrap
> $ mkdir build
> $ cd build
> $ ../configure --enable-new-cli --disable-java --disable-python
> $ make check
> ```
>
> For CMake:
> ```
> $ ./bootstrap
> $ mkdir build
> $ cd build
> $ cmake .. -DENABLE_NEW_CLI=1
> $ cmake --build . -- -j16
> $ ./src/mesos
> Mesos CLI
>
> Usage:
> mesos (-h | --help)
> mesos --version
> mesos <command> [<args>...]
>
> Options:
> -h --help Show this screen.
> --version Show version info.
>
> Commands:
> agent Interacts with the Mesos agents
> config Interacts with the Mesos CLI configuration file
> task Interacts with the tasks running in a Mesos cluster
>
> See 'mesos help <command>' for more information on a specific command.
> $ cmake --build . --target tests -- -j16
> $ ctest -R CLI
> Test project /home/agrillet/apache-mesos/build
> Start 4: CLITests
> 1/1 Test #4: CLITests ......................... Passed 3.63 sec
>
> 100% tests passed, 0 tests failed out of 1
> ```
>
> Checked that the the CLI tests were run, that the content of the directory `build/src/cli` was as expected, and that `build/src/mesos` was correctly running.
>
>
> Thanks,
>
> Armand Grillet
>
>