> On Feb. 14, 2017, 9:52 p.m., Aurora ReviewBot wrote: > > 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"
@ReviewBot retry - Santhosh Kumar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55951/#review165649 ----------------------------------------------------------- On Feb. 14, 2017, 9:40 p.m., Santhosh Kumar Shanmugham wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55951/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2017, 9:40 p.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 > >