-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55951/#review165649
-----------------------------------------------------------
Master (9ea8979) is red with this patch.
./build-support/jenkins/build.sh
==================== FAILURES ====================
test_simple_process_filesystem_isolator_launch_info
args = (), kw = {}
[1m @wraps(f)[0m
[1m def _inner(*args, **kw):[0m
[1m self._patch_dict()[0m
[1m try:[0m
[1m> return f(*args, **kw)[0m
.pants.d/python-setup/chroots/22bbf744265cd5201eb0521a411354ed4e0941bf/.deps/mock-1.0.1-py2-none-any.whl/mock.py:1618:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
src/test/python/apache/thermos/core/test_process.py:190:
in test_simple_process_filesystem_isolator_launch_info
[1m '"working_directory": "%s", "user": "None",
"rootfs": "/some/path/taskfs"}\n' % (sandbox))[0m
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
taskpath = <apache.thermos.common.path.TaskPath object at
0x7f6b822fcd90>
log_name = 'stdout'
expected_content = 'launch --unshare_namespace_mnt
--launch_info={"command": {"shell": false, "arguments": ["/bin/bash", "-c",
"echo hell..."value": "/bin/bash"}, "working_directory":
"/tmp/tmpc0WFNT/sandbox", "user": "None", "rootfs": "/some/path/taskfs"}\n'
[1m def assert_log_content(taskpath, log_name,
expected_content):[0m
[1m log =
taskpath.with_filename(log_name).getpath('process_logdir')[0m
[1m assert os.path.exists(log)[0m
[1m with open(log, 'r') as fp:[0m
[1m> assert fp.read() == expected_content[0m
[1m[31mE assert 'launch --uns.../bin/bash"}\n' ==
'launch --unsh...th/taskfs"}\n'[0m
[1m[31mE - launch --unshare_namespace_mnt
--working_directory=/tmp/tmpc0WFNT/sandbox --rootfs=/some/path/taskfs
--user=None --command={"shell": false, "arguments": ["/bin/bash", "-c", "echo
hello world"], "value": "/bin/bash"}[0m
[1m[31mE + launch --unshare_namespace_mnt
--launch_info={"command": {"shell": false, "arguments": ["/bin/bash", "-c",
"echo hello world"], "value": "/bin/bash"}, "working_directory":
"/tmp/tmpc0WFNT/sandbox", "user": "None", "rootfs": "/some/path/taskfs"}[0m
src/test/python/apache/thermos/core/test_process.py:364:
AssertionError
generated xml file:
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/test-results/aaf4d108c31293299a0839bdc404a91802f80937.xml
[1m[31m 1 failed, 780 passed, 6 skipped, 1 warnings in
288.49 seconds [0m
FAILURE
05:52:42 05:26 [complete][31m
FAILURE[0m
I will refresh this build result if you post a review containing "@ReviewBot
retry"
- Aurora ReviewBot
On Feb. 15, 2017, 5:40 a.m., Santhosh Kumar Shanmugham wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/55951/
> -----------------------------------------------------------
>
> (Updated Feb. 15, 2017, 5:40 a.m.)
>
>
> Review request for Aurora, Joshua Cohen, Stephan Erb, and Karthik Anantha
> Padmanabhan.
>
>
> Bugs: AURORA-1882
> https://issues.apache.org/jira/browse/AURORA-1882
>
>
> Repository: aurora
>
>
> Description
> -------
>
> MesosContainerizer has updated the command line parameters in 1.2.0 and
> consolidated the individual arguments into a single ContainerLaunchInfo
> proto buf message. Update ThermosExecutor to use the new `--launch_info`
> parameter to be compatible with MesosContainerizer also check the
> containerizer binary interface to determine to be backward-compatible.
>
>
> Diffs
> -----
>
> src/main/python/apache/thermos/common/process_util.py
> 54e716b726fc02f3901f1b9143d3fa253511e29b
> src/test/python/apache/aurora/executor/common/test_health_checker.py
> a3b5a22a805753d42e97f8f1e4e3dab7d3c9c507
> src/test/python/apache/thermos/core/test_process.py
> 520390217f691b9136cb4d36262be3d372a16509
>
> Diff: https://reviews.apache.org/r/55951/diff/
>
>
> Testing
> -------
>
> build-support/jenkins/build.sh
> src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
>
>
> Thanks,
>
> Santhosh Kumar Shanmugham
>
>
