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

Open udp logstash port 5000 for logspout #463

Merged
merged 1 commit into from
Feb 10, 2020

Conversation

DoubleDi
Copy link
Contributor

Hey! Thanks for your great repo on elk!
Had an issue when installing the logspout extension. We use udp to send data to logstash, but the port is opened only for tcp connections. This PR fixes that

@antoineco
Copy link
Collaborator

antoineco commented Jan 16, 2020

Hey, thanks for the contribution! This only applies to ports exposed on the host, containers that you start via compose are in the same network and are not affected by those settings. May I ask how you started Logspout?

See also https://github.com/deviantony/docker-elk/blob/master/extensions/logspout/README.md

@antoineco
Copy link
Collaborator

ping @DoubleDi

@DoubleDi
Copy link
Contributor Author

DoubleDi commented Feb 9, 2020

Yes, i cloned this repo to 2 different machines.
One had containers, which logs i wanted to see on kibana - i started only logspout there.
Other was the main machine - i started everithing else there.
So i think that happened because i had 2 machines and i was sending logs between them.

@antoineco
Copy link
Collaborator

In that case the change is necessary indeed, thanks for the clarification!

The change looks sensible because several Logstash input plugins work over UDP, so this could save users some unnecessary headaches. Thanks for the contribution!

@antoineco antoineco merged commit 69abb4a into deviantony:master Feb 10, 2020
antoineco pushed a commit that referenced this pull request Feb 11, 2020
bleed2002 pushed a commit to bleed2002/docker-elk that referenced this pull request Feb 29, 2020
DanBrown47 pushed a commit to DanBrown47/docker-elk that referenced this pull request Jun 22, 2023
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

2 participants