[jira] [Commented] (SLIDER-1115) stdout/stderr files have errors

["Billie Rinaldi (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SLIDER-1115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15310737#comment-15310737 ] 

Billie Rinaldi commented on SLIDER-1115:
----------------------------------------

+1, it looks like this patch fixes the test failures.  I've moved the log directory issue to a separate ticket, so we can commit this and resolve this part of the problem.

> stdout/stderr files have errors
> -------------------------------
>
>                 Key: SLIDER-1115
>                 URL: https://issues.apache.org/jira/browse/SLIDER-1115
>             Project: Slider
>          Issue Type: Bug
>            Reporter: Billie Rinaldi
>            Assignee: thomas liu
>             Fix For: Slider 0.91
>
>         Attachments: slider-1115-001.patch
>
>
> Some IT tests are failing due to the following errors.
> In shell.py and windows/system.py, the files are created based on the pid file name, but this doesn't work because when the pid file is specified it has an absolute path.  The file names could just be changed to application.log and application.err.
> Also in shell.py and windows/system.py, the stdout and stderr files are opened with 'w' mode, but each container may run multiple Execute commands, so the files will be overwritten with the latest output.  The mode should be changed to 'w+' so all the output is captured.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)