-
Notifications
You must be signed in to change notification settings - Fork 23
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
Add an option to support plain TCP #55
Comments
@ph: I was looking at using the Lumberjack protocol over spiped. Do you agree the output plugin should have a similar configuration for symmetry? |
@ralphm +1 for symmetry, and it's just a matter of exposing the underlying configuration options. |
@suyograo Do we still want to add this to the lumberjack input since LSF wont support plain text connection and we have created |
+1 |
@ph as a user, my gut would say to keep behavior here as-is, and focus on making sure we have a clean migration path for users moving from LSF to FB, in preparation for full-on removal of this input at some future date. Any thoughts? |
Yeah, i would keep the current behavior as is and put our efforts on input-beats. Filebeats is already being deployed as a replacement for LSF, so there is really no need for this feature. I'd close this issue if you agree |
@suyograo I agree. Though it's worth noting, there was a report in IRC today of difficulties using LSF with 2.x. While the user is planning on adopting FB, they are also trying to modernize their LS install. Is there a recommended upgrade path in terms of versions, etc, for a user to move from LSF to FB incrementally? |
I am confused, maybe by the many acronyms. I currently use this to receive events shipped by a Logstash instance in another data center like so: Kafka -> Logstash -------------> Logstash -> Kafka The receiving instance does some additional filtering and selects the destination Kafka topic based on data in the event. How do beats come into play here? Does it use the same protocol? |
@ralphm My understanding is that that use case is satisfied by moving the output/input combination from lumberjack/lumberjack to beats/beats. Again, my understanding here is that beats is an advancement of the custom protocol that this plugin piloted, but I welcome any corrections from the team. |
@torrancew You are correct, Its an evolution of the protocol of LSF and also an updated light agent. |
Since we are moving our effort to inputs beats I will be closing this issue. |
The current master of ruby-lumberjack include an option to disable SSL for the socket connection.
We have to expose this feature to the input, I suggest we use the same naming that we use in the elasticsearch output (https://github.com/logstash-plugins/logstash-output-elasticsearch/blob/master/lib/logstash/outputs/elasticsearch.rb#L161), keep in mind that it will be turn on by default and people need to explicitly turn it off.
Ref: elastic/ruby-lumberjack#14 elastic/ruby-lumberjack#12
The text was updated successfully, but these errors were encountered: