Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Create R script file in workspace #6

Merged
merged 1 commit into from May 19, 2017

Conversation

EmteZogaf
Copy link

Trying to use the r step inside a docker container step (like withDockerContainer) in a pipeline script failed because the R script file was created inside the /tmp folder of the jenkins node instead of the docker container. The /tmp folder is not bound to the docker container as a volume by default, but the project workspace is.

Trying to use the r step inside a docker container step (like
withDockerContainer) in a pipeline script failed because the R script
file was created inside the /tmp folder of the jenkins node instead of
the docker container. The /tmp folder is not bound to the docker
container as a volume by default, but the project workspace is.
@EmteZogaf EmteZogaf changed the title Create R script file in workspace [WIP] Create R script file in workspace May 19, 2017
@kinow
Copy link
Member

kinow commented May 19, 2017

Nice catch, using the inThisDirectory boolean flag as true seems harmless too.

@kinow kinow merged commit 5085280 into jenkinsci:master May 19, 2017
@kinow
Copy link
Member

kinow commented May 20, 2017

0.5 released with this change. Updating Wiki now. Sync in process, available from update center in the next hours.

@kinow
Copy link
Member

kinow commented May 20, 2017

The FreeStyle build is working, and I think the workflow steps should work too. The change seemed harmless, so merged & released regardless of the WIP. Feel free to update the ticket if we need to do any further work @EmteZogaf We can just cut a new release in the next days.

@EmteZogaf
Copy link
Author

OK, thanks. I marked it as WIP, to have a look at the BourneShellScriptclass where it stores the temporary script files. I believe it's storing it inside a subfolder of the workspace.
But this can be changed in another PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants