Re: Review Request 51746: Update e2e tests to verify running a docker image via the unified containerizer.

2016-09-08 Thread Stephan Erb

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


Ship it!




Ship It!

- Stephan Erb


On Sept. 8, 2016, 10:56 p.m., Joshua Cohen wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51746/
> ---
> 
> (Updated Sept. 8, 2016, 10:56 p.m.)
> 
> 
> Review request for Aurora, Stephan Erb and Zhitao Li.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Update e2e tests to verify running a docker image via the unified 
> containerizer.
> 
> 
> Diffs
> -
> 
>   examples/vagrant/mesos_config/etc_mesos-slave/docker_registry PRE-CREATION 
>   examples/vagrant/mesos_config/etc_mesos-slave/image_providers 
> 7168c0ca4a81e0a1ce2320534015c6692b804b54 
>   examples/vagrant/mesos_config/etc_mesos-slave/isolation 
> c36223028513ace26040620b813022accaf0edf3 
>   src/test/sh/org/apache/aurora/e2e/http/http_example.aurora 
> 042424d408a6b14be05c6e1c967215a53836d20e 
>   src/test/sh/org/apache/aurora/e2e/http/http_example_bad_healthcheck.aurora 
> f80f2b1fe8accfa575b2c3e626feb8d926455f68 
>   src/test/sh/org/apache/aurora/e2e/http/http_example_updated.aurora 
> 4a95bc3a3d1ce50ea143266884efc4d0a7c6e767 
>   src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh 
> d81abe1f974bfb13b54a60b2018700aaf28a8ee6 
> 
> Diff: https://reviews.apache.org/r/51746/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joshua Cohen
> 
>



Re: Review Request 51746: Update e2e tests to verify running a docker image via the unified containerizer.

2016-09-08 Thread Aurora ReviewBot

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


Ship it!




Master (8fca745) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On Sept. 8, 2016, 8:56 p.m., Joshua Cohen wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51746/
> ---
> 
> (Updated Sept. 8, 2016, 8:56 p.m.)
> 
> 
> Review request for Aurora, Stephan Erb and Zhitao Li.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Update e2e tests to verify running a docker image via the unified 
> containerizer.
> 
> 
> Diffs
> -
> 
>   examples/vagrant/mesos_config/etc_mesos-slave/docker_registry PRE-CREATION 
>   examples/vagrant/mesos_config/etc_mesos-slave/image_providers 
> 7168c0ca4a81e0a1ce2320534015c6692b804b54 
>   examples/vagrant/mesos_config/etc_mesos-slave/isolation 
> c36223028513ace26040620b813022accaf0edf3 
>   src/test/sh/org/apache/aurora/e2e/http/http_example.aurora 
> 042424d408a6b14be05c6e1c967215a53836d20e 
>   src/test/sh/org/apache/aurora/e2e/http/http_example_bad_healthcheck.aurora 
> f80f2b1fe8accfa575b2c3e626feb8d926455f68 
>   src/test/sh/org/apache/aurora/e2e/http/http_example_updated.aurora 
> 4a95bc3a3d1ce50ea143266884efc4d0a7c6e767 
>   src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh 
> d81abe1f974bfb13b54a60b2018700aaf28a8ee6 
> 
> Diff: https://reviews.apache.org/r/51746/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joshua Cohen
> 
>



Re: Review Request 51746: Update e2e tests to verify running a docker image via the unified containerizer.

2016-09-08 Thread Zhitao Li

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


Ship it!




Ship It!

- Zhitao Li


On Sept. 8, 2016, 8:56 p.m., Joshua Cohen wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51746/
> ---
> 
> (Updated Sept. 8, 2016, 8:56 p.m.)
> 
> 
> Review request for Aurora, Stephan Erb and Zhitao Li.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Update e2e tests to verify running a docker image via the unified 
> containerizer.
> 
> 
> Diffs
> -
> 
>   examples/vagrant/mesos_config/etc_mesos-slave/docker_registry PRE-CREATION 
>   examples/vagrant/mesos_config/etc_mesos-slave/image_providers 
> 7168c0ca4a81e0a1ce2320534015c6692b804b54 
>   examples/vagrant/mesos_config/etc_mesos-slave/isolation 
> c36223028513ace26040620b813022accaf0edf3 
>   src/test/sh/org/apache/aurora/e2e/http/http_example.aurora 
> 042424d408a6b14be05c6e1c967215a53836d20e 
>   src/test/sh/org/apache/aurora/e2e/http/http_example_bad_healthcheck.aurora 
> f80f2b1fe8accfa575b2c3e626feb8d926455f68 
>   src/test/sh/org/apache/aurora/e2e/http/http_example_updated.aurora 
> 4a95bc3a3d1ce50ea143266884efc4d0a7c6e767 
>   src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh 
> d81abe1f974bfb13b54a60b2018700aaf28a8ee6 
> 
> Diff: https://reviews.apache.org/r/51746/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Joshua Cohen
> 
>



Review Request 51746: Update e2e tests to verify running a docker image via the unified containerizer.

2016-09-08 Thread Joshua Cohen

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

Review request for Aurora, Stephan Erb and Zhitao Li.


Repository: aurora


Description
---

Update e2e tests to verify running a docker image via the unified containerizer.


Diffs
-

  examples/vagrant/mesos_config/etc_mesos-slave/docker_registry PRE-CREATION 
  examples/vagrant/mesos_config/etc_mesos-slave/image_providers 
7168c0ca4a81e0a1ce2320534015c6692b804b54 
  examples/vagrant/mesos_config/etc_mesos-slave/isolation 
c36223028513ace26040620b813022accaf0edf3 
  src/test/sh/org/apache/aurora/e2e/http/http_example.aurora 
042424d408a6b14be05c6e1c967215a53836d20e 
  src/test/sh/org/apache/aurora/e2e/http/http_example_bad_healthcheck.aurora 
f80f2b1fe8accfa575b2c3e626feb8d926455f68 
  src/test/sh/org/apache/aurora/e2e/http/http_example_updated.aurora 
4a95bc3a3d1ce50ea143266884efc4d0a7c6e767 
  src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh 
d81abe1f974bfb13b54a60b2018700aaf28a8ee6 

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


Testing
---


Thanks,

Joshua Cohen