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

feat(splunk-hec): add plugin & fix dependencies #179

Merged
merged 1 commit into from
Mar 2, 2021

Conversation

jliao2011
Copy link
Contributor

Add the offical v1.2.4 splunk-hec plugin to rubygems
Removes fluent-plugin-logzio plugin due to dependency errors and deprecated plugin that hasn't been updated/maintained since March 2019
Updates fluent-plugin-loggly to v.1.0.pre to fix dependency conflict
Updates fluent-plugin-datadog to v0.12.1 to fix dependency conflict
Fixes #125

Signed-off-by: Jeff Liao jliao.2015@gmail.com

base-image/Gemfile Outdated Show resolved Hide resolved
@carpenterm
Copy link
Contributor

@jliao2011 Thanks for your contribution, there are teams using the Logz.io plugin in production so we cannot just remove this entirely. Looks like Logz.io are working on an update - logzio/fluent-plugin-logzio#33

@carpenterm
Copy link
Contributor

@jliao2011 a new version of fluent-plugin-logzio (0.0.21) is available, please could you update your PR accordingly?

Add the offical v1.2.4 splunk-hec plugin to rubygems
Removes fluent-plugin-logzio plugin due to dependency errors and deprecated plugin that hasn't been updated/maintained since March 2019
Updates fluent-plugin-loggly to v.1.0.pre to fix dependency conflict
Updates fluent-plugin-datadog to v0.12.1 to fix dependency conflict
Fixes vmware#125

Signed-off-by: Jeff Liao <jliao.2015@gmail.com>
@jliao2011
Copy link
Contributor Author

@jliao2011 a new version of fluent-plugin-logzio (0.0.21) is available, please could you update your PR accordingly?

Done.

Copy link
Contributor

@Cryptophobia Cryptophobia left a comment

Choose a reason for hiding this comment

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

lgtm

@Cryptophobia Cryptophobia merged commit 83941ee into vmware:master Mar 2, 2021
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.

Migrate to official splunk plugin GEM
4 participants