----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52046/#review149521 -----------------------------------------------------------
Master (330e8ee) is red with this patch. ./build-support/jenkins/build.sh [1m # Create file stdout for capturing output. We can't use StringIO mock[0m [1m # because TestProcess is running fork.[0m [1m with open(os.path.join(td, 'sys_stdout'), 'w+') as stdout:[0m [1m with open(os.path.join(td, 'sys_stderr'), 'w+') as stderr:[0m [1m with mutable_sys():[0m [1m sys.stdout, sys.stderr = stdout, stderr[0m [1m [0m [1m p = TestProcess('process', 'echo hello world; echo >&2 hello stderr', 0,[0m [1m taskpath, sandbox, logger_destination=LoggerDestination.BOTH)[0m [1m p.start()[0m [1m rc = wait_for_rc(taskpath.getpath('process_checkpoint'))[0m [1m [0m [1m assert rc == 0[0m [1m # Check log files were created in std path with correct content[0m [1m> assert_log_content(taskpath, 'stdout', 'hello world\n')[0m src/test/python/apache/thermos/core/test_process.py:487: _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ taskpath = <apache.thermos.common.path.TaskPath object at 0x7f2df4b7db10> log_name = 'stdout' expected_content = 'hello world\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 '' == 'hello world\n'[0m [1m[31mE + hello world[0m src/test/python/apache/thermos/core/test_process.py:313: AssertionError generated xml file: /home/jenkins/jenkins-slave/workspace/AuroraBot/dist/test-results/415337499eb72578eab327a6487c1f5c9452b3d6.xml [1m[31m 1 failed, 710 passed, 6 skipped, 1 warnings in 398.69 seconds [0m FAILURE 18:54:17 07:27 [complete][31m FAILURE[0m I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Sept. 19, 2016, 6:28 p.m., Joshua Cohen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52046/ > ----------------------------------------------------------- > > (Updated Sept. 19, 2016, 6:28 p.m.) > > > Review request for Aurora and Jake Farrell. > > > Repository: aurora > > > Description > ------- > > Adding gpg key for jco...@apache.org > > > Diffs > ----- > > KEYS d9e5ec932aea33dc5c2cb69e64bbefd1a155cced > > Diff: https://reviews.apache.org/r/52046/diff/ > > > Testing > ------- > > > Thanks, > > Joshua Cohen > >