You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Jojy Varghese <jo...@mesosphere.io> on 2016/04/19 19:03:36 UTC
Review Request 46371: Added basic tests for capabilities API.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46371/
-----------------------------------------------------------
Review request for mesos and Jie Yu.
Repository: mesos
Description
-------
Added basic tests for capabilities API.
Diffs
-----
src/Makefile.am a8f68316c5d4d4c82d99363535a97fa5b1caafc5
src/tests/capabilities_tests.cpp PRE-CREATION
Diff: https://reviews.apache.org/r/46371/diff/
Testing
-------
make check.
Thanks,
Jojy Varghese
Re: Review Request 46371: Added basic tests for capabilities API.
Posted by Mesos ReviewBot <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46371/#review129649
-----------------------------------------------------------
Bad patch!
Reviews applied: [46371, 46370, 46369]
Failed command: ./support/apply-review.sh -n -r 46370
Error:
2016-04-19 23:35:36 URL:https://reviews.apache.org/r/46370/diff/raw/ [17675/17675] -> "46370.patch" [1]
error: patch failed: src/Makefile.am:858
error: src/Makefile.am: patch does not apply
Full log: https://builds.apache.org/job/mesos-reviewbot/12619/console
- Mesos ReviewBot
On April 19, 2016, 5:03 p.m., Jojy Varghese wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46371/
> -----------------------------------------------------------
>
> (Updated April 19, 2016, 5:03 p.m.)
>
>
> Review request for mesos and Jie Yu.
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added basic tests for capabilities API.
>
>
> Diffs
> -----
>
> src/Makefile.am a8f68316c5d4d4c82d99363535a97fa5b1caafc5
> src/tests/capabilities_tests.cpp PRE-CREATION
>
> Diff: https://reviews.apache.org/r/46371/diff/
>
>
> Testing
> -------
>
> make check.
>
>
> Thanks,
>
> Jojy Varghese
>
>
Re: Review Request 46371: Added basic tests for capabilities API.
Posted by Mesos ReviewBot <re...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46371/#review129827
-----------------------------------------------------------
Patch looks great!
Reviews applied: [46369, 46370, 46371]
Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh
- Mesos ReviewBot
On April 20, 2016, 7:12 p.m., Jojy Varghese wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46371/
> -----------------------------------------------------------
>
> (Updated April 20, 2016, 7:12 p.m.)
>
>
> Review request for mesos and Jie Yu.
>
>
> Repository: mesos
>
>
> Description
> -------
>
> Added basic tests for capabilities API.
>
>
> Diffs
> -----
>
> src/Makefile.am 08d5279bbc0c763255fda3c3360d8f24f41bccda
> src/tests/capabilities_test_helper.hpp PRE-CREATION
> src/tests/capabilities_test_helper.cpp PRE-CREATION
> src/tests/capabilities_test_helper_main.cpp PRE-CREATION
> src/tests/capabilities_tests.cpp PRE-CREATION
>
> Diff: https://reviews.apache.org/r/46371/diff/
>
>
> Testing
> -------
>
> make check.
>
>
> Thanks,
>
> Jojy Varghese
>
>
Re: Review Request 46371: Added basic tests for capabilities API.
Posted by Jojy Varghese <jo...@mesosphere.io>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46371/
-----------------------------------------------------------
(Updated April 20, 2016, 7:12 p.m.)
Review request for mesos and Jie Yu.
Repository: mesos
Description
-------
Added basic tests for capabilities API.
Diffs (updated)
-----
src/Makefile.am 08d5279bbc0c763255fda3c3360d8f24f41bccda
src/tests/capabilities_test_helper.hpp PRE-CREATION
src/tests/capabilities_test_helper.cpp PRE-CREATION
src/tests/capabilities_test_helper_main.cpp PRE-CREATION
src/tests/capabilities_tests.cpp PRE-CREATION
Diff: https://reviews.apache.org/r/46371/diff/
Testing
-------
make check.
Thanks,
Jojy Varghese