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

chart: add udp_transport_type for gelf output plugin #1988

Merged
merged 4 commits into from
Mar 27, 2025

Conversation

bmichalkiewicz
Copy link
Contributor

This pull request adds the missing add udp_transport_type flag (introduced in version 1.4.1 of fluent-plugin-gelf-best).

we need to update the gem to version 1.4.1, any help how to do it ? just make a change here:

gem 'fluent-plugin-gelf-best', '1.3.4'
?

@pepov
Copy link
Member

pepov commented Mar 20, 2025

yes, please make a change in the Gemfile

@bmichalkiewicz
Copy link
Contributor Author

done

@bmichalkiewicz
Copy link
Contributor Author

fluent-plugin-gelf-best (= 1.3.4)

change here also?

@csatib02 csatib02 added the dependencies Pull requests that update a dependency file label Mar 23, 2025
Signed-off-by: Bartosz Michałkiewicz <[email protected]>
Signed-off-by: Bartosz Michałkiewicz <[email protected]>
Signed-off-by: Bartosz Michałkiewicz <[email protected]>
Copy link
Member

@csatib02 csatib02 left a comment

Choose a reason for hiding this comment

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

LGTM!
Thanks for your contribution!

@bmichalkiewicz
Copy link
Contributor Author

once it's merged, could you create a dev tag to create the pre-release chart and image?

@pepov pepov merged commit baf2f38 into kube-logging:master Mar 27, 2025
25 checks passed
@pepov
Copy link
Member

pepov commented Mar 27, 2025

5.3.0-dev.0 is building

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants