Skip to content

Docker::Services: Add 'mounts' parameter #421

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

Merged
merged 1 commit into from
Feb 10, 2019

Conversation

jacksgt
Copy link
Contributor

@jacksgt jacksgt commented Feb 7, 2019

The --mount parameter for Docker services is the equivalent to
-v/--volume for standalone Docker containers.
This patch allows to directly specify one or multile values for
this argument directly as a Puppet parameter.
This avoids the hassle of having to go through 'extra_params', since
this is a commonly used option.

https://docs.docker.com/storage/volumes/

Please let me know if I forgot to add/edit anything (tests/...)

The --mount parameter for Docker services is the equivalent to
-v/--volume for standalone Docker containers.
This patch allows to directly specify one or multile values for
this argument directly as a Puppet parameter.
This avoids the hassle of having to go through 'extra_params', since
this is a commonly used option.

https://docs.docker.com/storage/volumes/
@davejrt
Copy link
Contributor

davejrt commented Feb 10, 2019

LGTM, thanks for your contribution

@davejrt davejrt merged commit c1283da into puppetlabs:master Feb 10, 2019
@davejrt davejrt added this to the 3.2.1 milestone Feb 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants