You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by David Forsythe <df...@gmail.com> on 2015/10/26 01:32:46 UTC

Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

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

Review request for mesos and Ian Downes.


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


Repository: mesos


Description
-------

FreeBSD: Enable mesos build and start fixing some tests


Diffs
-----

  configure.ac 35f2908b4fac8e29fc36d89894e447c46ca2e15c 
  src/Makefile.am 98cbafc134ec388a176d50172912fbfdf9f5bfa3 
  src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
  src/tests/resources_tests.cpp 6584fc6c39e6ffe9f8085576677dcc669f127697 
  src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 

Diff: https://reviews.apache.org/r/39634/diff/


Testing
-------

make check on ubuntu 14.04


Thanks,

David Forsythe


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by David Forsythe <df...@gmail.com>.

> On Nov. 2, 2015, 6:58 p.m., Alex Clemmer wrote:
> > configure.ac, lines 612-621
> > <https://reviews.apache.org/r/39634/diff/2/?file=1112829#file1112829line612>
> >
> >     If it's not too much trouble, it would be great to see this logic added also the `cmake/CompilationCOnfigure.cmake`. It should only be a couple lines of code.

Mind if that happens after/if this change lands? I'm not sure what the ettiquite is for filing tickets on unsupported platforms, but I have no problem bringing the cmake stuff up to date if this gets merged.


- David


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


On Oct. 30, 2015, 5:05 a.m., David Forsythe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39634/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2015, 5:05 a.m.)
> 
> 
> Review request for mesos and Ian Downes.
> 
> 
> Bugs: MESOS-1563
>     https://issues.apache.org/jira/browse/MESOS-1563
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> FreeBSD: Enable mesos build and start fixing some tests
> 
> 
> Diffs
> -----
> 
>   configure.ac 656766430e2720c6a407e282521102b54547fd2d 
>   src/Makefile.am d6eb302f0e812a777f51f421deef89140871a1db 
>   src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
>   src/tests/resources_tests.cpp 6584fc6c39e6ffe9f8085576677dcc669f127697 
>   src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 
> 
> Diff: https://reviews.apache.org/r/39634/diff/
> 
> 
> Testing
> -------
> 
> make check on ubuntu 14.04
> 
> 
> Thanks,
> 
> David Forsythe
> 
>


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by Alex Clemmer <cl...@gmail.com>.

> On Nov. 2, 2015, 6:58 p.m., Alex Clemmer wrote:
> > configure.ac, lines 612-621
> > <https://reviews.apache.org/r/39634/diff/2/?file=1112829#file1112829line612>
> >
> >     If it's not too much trouble, it would be great to see this logic added also the `cmake/CompilationCOnfigure.cmake`. It should only be a couple lines of code.
> 
> David Forsythe wrote:
>     Mind if that happens after/if this change lands? I'm not sure what the ettiquite is for filing tickets on unsupported platforms, but I have no problem bringing the cmake stuff up to date if this gets merged.

Sure, that sounds good, but could I ask you file a JIRA ticket so we can track the work? The CMake epic is here: https://issues.apache.org/jira/browse/MESOS-898


- Alex


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


On Oct. 30, 2015, 5:05 a.m., David Forsythe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39634/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2015, 5:05 a.m.)
> 
> 
> Review request for mesos and Ian Downes.
> 
> 
> Bugs: MESOS-1563
>     https://issues.apache.org/jira/browse/MESOS-1563
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> FreeBSD: Enable mesos build and start fixing some tests
> 
> 
> Diffs
> -----
> 
>   configure.ac 656766430e2720c6a407e282521102b54547fd2d 
>   src/Makefile.am d6eb302f0e812a777f51f421deef89140871a1db 
>   src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
>   src/tests/resources_tests.cpp 6584fc6c39e6ffe9f8085576677dcc669f127697 
>   src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 
> 
> Diff: https://reviews.apache.org/r/39634/diff/
> 
> 
> Testing
> -------
> 
> make check on ubuntu 14.04
> 
> 
> Thanks,
> 
> David Forsythe
> 
>


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by Alex Clemmer <cl...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/#review104756
-----------------------------------------------------------



configure.ac (lines 612 - 621)
<https://reviews.apache.org/r/39634/#comment162999>

    If it's not too much trouble, it would be great to see this logic added also the `cmake/CompilationCOnfigure.cmake`. It should only be a couple lines of code.


- Alex Clemmer


On Oct. 30, 2015, 5:05 a.m., David Forsythe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39634/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2015, 5:05 a.m.)
> 
> 
> Review request for mesos and Ian Downes.
> 
> 
> Bugs: MESOS-1563
>     https://issues.apache.org/jira/browse/MESOS-1563
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> FreeBSD: Enable mesos build and start fixing some tests
> 
> 
> Diffs
> -----
> 
>   configure.ac 656766430e2720c6a407e282521102b54547fd2d 
>   src/Makefile.am d6eb302f0e812a777f51f421deef89140871a1db 
>   src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
>   src/tests/resources_tests.cpp 6584fc6c39e6ffe9f8085576677dcc669f127697 
>   src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 
> 
> Diff: https://reviews.apache.org/r/39634/diff/
> 
> 
> Testing
> -------
> 
> make check on ubuntu 14.04
> 
> 
> Thanks,
> 
> David Forsythe
> 
>


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by Ian Downes <ia...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/#review108377
-----------------------------------------------------------

Ship it!


Very minor things. I'll just make the changes when I commit the patch. Thanks!


configure.ac (line 619)
<https://reviews.apache.org/r/39634/#comment167814>

    End with fullstop.



src/Makefile.am (line 1756)
<https://reviews.apache.org/r/39634/#comment167816>

    Can you add a comment that FreeBSD includes the  necessary functionality in libc?



src/tests/attributes_tests.cpp (line 38)
<https://reviews.apache.org/r/39634/#comment167813>

    These are unrelated to the FreeBSD support so can you split them out? Else, I'll do it when I commit this patch.


- Ian Downes


On Nov. 27, 2015, 8:26 p.m., David Forsythe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39634/
> -----------------------------------------------------------
> 
> (Updated Nov. 27, 2015, 8:26 p.m.)
> 
> 
> Review request for mesos and Ian Downes.
> 
> 
> Bugs: MESOS-1563
>     https://issues.apache.org/jira/browse/MESOS-1563
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> FreeBSD: Enable mesos build and start fixing some tests
> 
> 
> Diffs
> -----
> 
>   configure.ac b30a8d30076f3068fd7d5fc8ccea1982639e998a 
>   src/Makefile.am fd38cfa73d81a98c819378f99a766e2ddb7e1a04 
>   src/tests/attributes_tests.cpp d8c84d25a37f9cf1b38a97193d5b3b3001fd54ff 
>   src/tests/resources_tests.cpp dbd39cd5a6786682a7b528b6fea37ab78904cf12 
>   src/tests/values_tests.cpp fb7f982a50df8274ad29dab9e55157c39acdc104 
> 
> Diff: https://reviews.apache.org/r/39634/diff/
> 
> 
> Testing
> -------
> 
> make check on ubuntu 14.04
> 
> 
> Thanks,
> 
> David Forsythe
> 
>


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by David Forsythe <df...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/
-----------------------------------------------------------

(Updated Nov. 28, 2015, 4:26 a.m.)


Review request for mesos and Ian Downes.


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


Repository: mesos


Description
-------

FreeBSD: Enable mesos build and start fixing some tests


Diffs (updated)
-----

  configure.ac b30a8d30076f3068fd7d5fc8ccea1982639e998a 
  src/Makefile.am fd38cfa73d81a98c819378f99a766e2ddb7e1a04 
  src/tests/attributes_tests.cpp d8c84d25a37f9cf1b38a97193d5b3b3001fd54ff 
  src/tests/resources_tests.cpp dbd39cd5a6786682a7b528b6fea37ab78904cf12 
  src/tests/values_tests.cpp fb7f982a50df8274ad29dab9e55157c39acdc104 

Diff: https://reviews.apache.org/r/39634/diff/


Testing
-------

make check on ubuntu 14.04


Thanks,

David Forsythe


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by David Forsythe <df...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/
-----------------------------------------------------------

(Updated Nov. 14, 2015, 9:02 a.m.)


Review request for mesos and Ian Downes.


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


Repository: mesos


Description
-------

FreeBSD: Enable mesos build and start fixing some tests


Diffs (updated)
-----

  configure.ac 8b28ac78eeb3e3b5905b411b4bc0db3ccf0f543f 
  src/Makefile.am 92d07c3da897c7664c63e2af91d45921d66f64aa 
  src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
  src/tests/resources_tests.cpp 0d084fd97ec108d5ec2d050eddc2e80ea81ffac0 
  src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 

Diff: https://reviews.apache.org/r/39634/diff/


Testing
-------

make check on ubuntu 14.04


Thanks,

David Forsythe


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by David Forsythe <df...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/
-----------------------------------------------------------

(Updated Nov. 10, 2015, 8:33 a.m.)


Review request for mesos and Ian Downes.


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


Repository: mesos


Description
-------

FreeBSD: Enable mesos build and start fixing some tests


Diffs (updated)
-----

  configure.ac 8b28ac78eeb3e3b5905b411b4bc0db3ccf0f543f 
  src/CMakeLists.txt c235209743de6e84deb44df31c61948f4dc8b8eb 
  src/Makefile.am ae2740a5b56351d9fd82ae3bd5c733d10a90bf2f 
  src/slave/containerizer/mesos/provisioner/docker/local_puller.hpp 103fe4b435e115857e2ebb071b6d88e2776736b5 
  src/slave/containerizer/mesos/provisioner/docker/local_puller.cpp 9b2b8126f39d2b4014cfb637202670e6a06cdf56 
  src/slave/containerizer/mesos/provisioner/docker/puller.hpp e4202550af4deddddb0d355ca9ae6d71b40623f9 
  src/slave/containerizer/mesos/provisioner/docker/puller.cpp 70501f1811fde5b636e5dcf751ef9356b8c7caef 
  src/slave/containerizer/mesos/provisioner/docker/registry_client.hpp 3fd84e380f9393a6d46544fb7fc7d24bcb6256ac 
  src/slave/containerizer/mesos/provisioner/docker/registry_client.cpp f6ae9e8919fbd010c16918ddc78aa126b9da0062 
  src/slave/containerizer/mesos/provisioner/docker/registry_puller.hpp 2445c8132ee4af07482422f8563b408459c676b2 
  src/slave/containerizer/mesos/provisioner/docker/registry_puller.cpp e883b402c32df4d7ed5edb906231500615a00da5 
  src/slave/containerizer/mesos/provisioner/docker/store.cpp 6fdb85b3d55339556b0982598d2e5258f6159466 
  src/slave/flags.hpp 6ae7c94d2e05d81c8b970e7dcaa82d8aa4de7936 
  src/slave/flags.cpp 4bc68429679a4605056aeb120e9deee51af2dde6 
  src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
  src/tests/containerizer/provisioner_docker_tests.cpp 9227666868801335003aeb0fa21e6b8f0e94f2cb 
  src/tests/resources_tests.cpp e663f5ec8ed2723ae93f85a1fc3335b4a8cbe9ed 
  src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 

Diff: https://reviews.apache.org/r/39634/diff/


Testing
-------

make check on ubuntu 14.04


Thanks,

David Forsythe


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by David Forsythe <df...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/
-----------------------------------------------------------

(Updated Nov. 10, 2015, 6:35 a.m.)


Review request for mesos and Ian Downes.


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


Repository: mesos


Description
-------

FreeBSD: Enable mesos build and start fixing some tests


Diffs (updated)
-----

  configure.ac 8b28ac78eeb3e3b5905b411b4bc0db3ccf0f543f 
  src/Makefile.am 6ec0488027d6cfccc63ac3a6a8b0c3d8eb6c3330 
  src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
  src/tests/resources_tests.cpp e663f5ec8ed2723ae93f85a1fc3335b4a8cbe9ed 
  src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 

Diff: https://reviews.apache.org/r/39634/diff/


Testing
-------

make check on ubuntu 14.04


Thanks,

David Forsythe


Re: Review Request 39634: FreeBSD: Enable mesos build and start fixing some tests

Posted by David Forsythe <df...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39634/
-----------------------------------------------------------

(Updated Oct. 30, 2015, 5:05 a.m.)


Review request for mesos and Ian Downes.


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


Repository: mesos


Description
-------

FreeBSD: Enable mesos build and start fixing some tests


Diffs (updated)
-----

  configure.ac 656766430e2720c6a407e282521102b54547fd2d 
  src/Makefile.am d6eb302f0e812a777f51f421deef89140871a1db 
  src/tests/attributes_tests.cpp 4fc0c31c3b2eb745432818c99746a097fde65df3 
  src/tests/resources_tests.cpp 6584fc6c39e6ffe9f8085576677dcc669f127697 
  src/tests/values_tests.cpp e9b1079bbadf05390b39bedd5ad5677f3d4ec0d8 

Diff: https://reviews.apache.org/r/39634/diff/


Testing
-------

make check on ubuntu 14.04


Thanks,

David Forsythe