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

Moved CSV from mount to copy #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MatthewJSalerno
Copy link

Since we are already creating an overlay image, we might as well move it here. In one of my builds it was build mounted as a directory. Probably just my docker syntax, but completely avoidable if we copy it from here.

Since we are already creating an overlay image, we might as well move it here.  In one of my builds it was build mounted as a directory.  Probably just my docker syntax, but completely avoidable if we copy it from here.
@derekslenk
Copy link

derekslenk commented Nov 26, 2019

Just an FYI, it can be mounted as a file, but it has to be the full path to the file and in quotes, eg:
"/home/drone/docker-scripts/service-names-port-numbers.csv:/etc/graylog/server/service-names-port-numbers.csv:ro"

You of course can have a variable in there:
"${DOCKER_HOME}/service-names-port-numbers.csv:/etc/graylog/server/service-names-port-numbers.csv:ro"

To get this to work you will have to remove the directory the old script created first.

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.

2 participants