Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Add Journalbeat docker image #58

Merged
merged 4 commits into from
Nov 27, 2018
Merged

Add Journalbeat docker image #58

merged 4 commits into from
Nov 27, 2018

Conversation

kvch
Copy link
Contributor

@kvch kvch commented Nov 21, 2018

This PR adds Journalbeat to the available Beats.
Let me know if there is any problem with the PR.

@ph
Copy link

ph commented Nov 22, 2018

Took a small look at this, can you update the README.md to mention journalbeat?

@kvch kvch merged commit bd79a70 into elastic:master Nov 27, 2018
@kvch kvch mentioned this pull request Nov 27, 2018
23 tasks
@@ -16,6 +17,7 @@ Documentation can be found on the [Elastic website][elastic]:
[auditbeat]: https://www.elastic.co/guide/en/beats/auditbeat/current/running-on-docker.html
[filebeat]: https://www.elastic.co/guide/en/beats/filebeat/current/running-on-docker.html
[heartbeat]: https://www.elastic.co/guide/en/beats/heartbeat/current/running-on-docker.html
[journalbeat]: https://www.elastic.co/guide/en/beats/journalbeat/current/running-on-docker.html
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

404 :)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have not added that page yet. I can do that today or tomorrow, but I need to know which versions need to have the page. I have 6.6 in my notes, but I'm not sure that's right. @kvch Can you let me know?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've created a PR here that adds the docker content: elastic/beats#9386

Copy link

@PhilippHomann PhilippHomann Dec 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dedemorton
Are there any news on that?
Is there already a docker image available for journalbeat?
Tried to pull some versions, but no success and unfortunately there is no latest at any beat..

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dedemorton It is correct. Its release is scheduled for 6.6.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tardyp,

You can pull this Docker image from our registry:

docker pull docker.elastic.co/beats/journalbeat:6.6.0
6.6.0: Pulling from beats/journalbeat
a02a4930cb5d: Already exists
3e1d11318120: Pull complete
6a2daf26539e: Pull complete
3c8eb3bcd46e: Pull complete
3101fedd1308: Pull complete
18021bedd90a: Pull complete
941f3bf14a14: Pull complete
Digest: sha256:6288b53f985bf9e705ec87668629976596abf0d7714f6960f207fa6fa9ad068f
Status: Downloaded newer image for docker.elastic.co/beats/journalbeat:6.6.0

I'm doing the work to have this docker image available in Dockerhub and displayed on https://www.docker.elastic.co/
Sorry for the trouble.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://www.docker.elastic.co/ has been updated to include journalbeat and the image is available on docker hub now https://cloud.docker.com/u/elastic/repository/docker/elastic/journalbeat.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

super cool thanks! I have been able to deploy the 6.6.0 image and replace mheese/journalbeat with elastic/journatbeat !

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The doc additions have already been merged into master. I will cherry-pick them into the 6.6 branch today.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually I did backport it. It just needs to be reviewed so I can merge it: elastic/beats#10386

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants