[MESOS-1423] MesosContainerizerExecuteTest.IoRedirection doesn't pass on OSX

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Component/s: test
• Assignee: Unassigned
• Reporter: Yan Xu
• Created: 27/May/14 21:41
• Updated: 02/Jul/15 18:43
• Resolved: 27/May/14 21:43

Description

[ RUN      ] MesosContainerizerExecuteTest.IoRedirection
Using temporary directory '/tmp/MesosContainerizerExecuteTest_IoRedirection_d2HaEd'
I0527 14:00:06.949267 2123838224 mesos_containerizer.cpp:124] Using isolation: posix/cpu,posix/mem
I0527 14:00:06.949887 293957632 mesos_containerizer.cpp:537] Starting container 'test_container' for executor 'executor' of framework ''
I0527 14:00:06.950804 292884480 launcher.cpp:117] Forked child with pid '29433' for container 'test_container'
I0527 14:00:06.951549 292884480 mesos_containerizer.cpp:647] Fetching URIs for container 'test_container' using command '/Users/jyx/Dropbox/MyCode/twitter/mesos/build/src/mesos-fetcher'
I0527 14:00:08.952647 292347904 mesos_containerizer.cpp:1103] Executor for container 'test_container' has exited
I0527 14:00:08.952719 292347904 mesos_containerizer.cpp:987] Destroying container 'test_container'
../../src/tests/containerizer_tests.cpp:486: Failure
Value of: (os::read(path::join(directory, "stderr"))).get()
  Actual: "-n this is stderr\n"
Expected: errMsg
Which is: "this is stderr"
../../src/tests/containerizer_tests.cpp:487: Failure
Value of: (os::read(path::join(directory, "stdout"))).get()
  Actual: "-n this is stderr\n"
Expected: outMsg
Which is: "this is stderr"
[  FAILED  ] MesosContainerizerExecuteTest.IoRedirection (2022 ms)
[----------] 1 test from MesosContainerizerExecuteTest (2022 ms total)

The reason seems to be

Some shells may provide a builtin echo command which is similar or identical to this utility.  Most notably, the builtin echo in sh(1) does not accept the -n option.  Consult the builtin(1) manual page.

The fix could be to spell out the full path /bin/echo or add newline character in the string variable errMsg.

Issue Links

• duplicates: MESOS-1413 MesosContainerizerExecuteTest.IoRedirection fails on OSX