8000 [Question] Loadable Plugins? · Issue #178 · google/mtail · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Question] Loadable Plugins? #178

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.

Sign up for GitHub

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

Jump to bottom
Closed
jnovack opened this issue Sep 21, 2018 · 2 comments
Closed

[Question] Loadable Plugins? #178

jnovack opened this issue Sep 21, 2018 · 2 comments
Labels
question This issue looks like a usage problem, not a bug or feature request

Comments

@jnovack
Copy link
Contributor
jnovack commented Sep 21, 2018

Question for the future, definitely not near future (considering you are close to a release).

With Go permitting plug-in architecture in recent builds, have you given thought to separating the tailing functions from the metrics processing function to be incorporated as a plugin?

You have a phenomenally structured and well-built tailing function. I wouldn't want to duplicate efforts for another project considering yours has been put through tests and real-life examples.

I wanted to write a "tail-to-syslog forwarder" (multiplatform, but specifically for windows clients, as there currently is none), but realized I'd be duplicating most of your tailing and filewatching code.

I realize the syslog forwarding is out of scope of this project, that's why I was curious about the loadable plugins potential.
@jaqx0r jaqx0r added the question This issue looks like a usage problem, not a bug or feature request label Nov 5, 2018
@jaqx0r
Copy link
Contributor
jaqx0r commented Nov 27, 2018

I don't know about making it a plugin, but it could be turned into it's own package to be imported into your own code...

@jnovack
Copy link
Contributor Author
jnovack commented Sep 9, 2019

This is the same ball-park of a question as #199.

Closing this as discussion is better over there.

@jnovack jnovack closed this as completed Sep 9, 2019
chengchuanpeng pushed a commit to chengchuanpeng/mtail that referenced this issue May 30, 2025
@jaqx0r
Merge pull request google#178 from jaqx0r/codeql
3408651 
ci: Remove manual codeql configs and revert to default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question This issue looks like a usage problem, not a bug or feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jnovack @jaqx0r
0