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

JENKINS-45266: Support for setting shared memory size (--shm-size) for docker slaves #497

Closed
wants to merge 2 commits into from

Conversation

laenzlinger
Copy link

For our use case we would like to increase the shared memory size of the containers started by the docker-plugin. The setting is possible via the command line with the option --shm-size.

This PR allows to configure the shared memory sized in the jenkins plugin configuration (within the template config)

@laenzlinger laenzlinger changed the title Support for setting shared memory size (--shm-size) for docker slaves [JENKINS-45266] Support for setting shared memory size (--shm-size) for docker slaves Jul 3, 2017
@laenzlinger laenzlinger changed the title [JENKINS-45266] Support for setting shared memory size (--shm-size) for docker slaves JENKINS-45266: Support for setting shared memory size (--shm-size) for docker slaves Jul 3, 2017
@fboudra
Copy link

fboudra commented Aug 21, 2017

it's now possible in KostyaSha/yet-another-docker-plugin 0.1.0-rc38

@laenzlinger
Copy link
Author

We have moved to yet-another-docker-plugin

@cesarhernandezgt
Copy link

Is there any intention to merge this PR or the feature is a -1 ?

@pjdarton
Copy link
Member

I've looked through the changes and they seem ok to me.

...however the PR would need updating as the code has moved on since this PR was authored such that it won't merge as-is.

@zucher
Copy link

zucher commented Apr 15, 2018

Hi @laenzlinger could you retry to solve the merge request, some features in YADP like ssh login no well working.

Thank you

@pjdarton
Copy link
Member

pjdarton commented May 8, 2018

Rendered obsolete by #643. All further comments should be made there.

@pjdarton pjdarton closed this May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants