-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Fix read-only instructions for running Beats on Docker #41120
Conversation
|
libbeat/docs/shared-docker.asciidoc
Outdated
docker run \ | ||
--mount type=source=$(pwd}/data,destination=/usr/share/{beatname_lc}/data | ||
--mount type=source=$(pwd)/data,destination=/usr/share/{beatname_lc}/data | ||
--read-only | ||
{dockerimage} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I managed to get it working with type=bind
, I also noticed that some \
are missing making copy/paste not to work, here is my fixed version:
docker run --rm \
--mount type=bind,source=$(pwd)/data,destination=/usr/share/filebeat/data \
--read-only \
docker.elastic.co/beats/filebeat:8.15.2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome. Thanks @belimawr!
Here's how it looks now (I'm building the docs off of main
but once published this would show an 8.15.2 Docker image):
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
* Fix read-only instructions for running Beats on Docker * Fix up command (cherry picked from commit b7d7fec)
* Fix read-only instructions for running Beats on Docker * Fix up command (cherry picked from commit b7d7fec)
* Fix read-only instructions for running Beats on Docker * Fix up command (cherry picked from commit b7d7fec) Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com> Co-authored-by: Pierre HILBERT <pierre.hilbert@elastic.co>
@blakerouse, Bill noticed a couple of issues with the Run (beat) on a read-only file system instructions:
I believe:
$(pwd}
should be$(pwd)
type=....,source=
This PR fixes up the first issue, but can you please help with the second? I'm not sure what the change should be, if any. I just copied that command from your comment here.