Skip to content

Bumped fluentd from 0.12.32 to 1.2.5#148

Open
yliaog wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
yliaog:master
Open

Bumped fluentd from 0.12.32 to 1.2.5#148
yliaog wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
yliaog:master

Conversation

@yliaog
Copy link

@yliaog yliaog commented Apr 20, 2018

1.2.5 is stable, tested, and has the feature to support nanosecond timestamp.

gem 'fluentd', '~>0.12.32'
gem 'fluentd', '~>0.14.25'
gem 'fluent-plugin-record-reformer', '~>0.8.3'
gem 'fluent-plugin-systemd', '~>0.0.8'
Copy link
Contributor

Choose a reason for hiding this comment

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

isn't fluent-plugin-systemd > 0.1.0 required for fluentd >= 0.14.11 as per https://github.com/reevoo/fluent-plugin-systemd#requirements ?

@yliaog
Copy link
Author

yliaog commented Apr 24, 2018

Yes, Bumped fluent-plugin-systemd to 0.3.1 to be compatible with fluentd 0.14.x. Please take another look.

@yliaog
Copy link
Author

yliaog commented Apr 25, 2018

/retest

@yliaog yliaog closed this Apr 25, 2018
@yliaog yliaog reopened this Apr 25, 2018
@yliaog
Copy link
Author

yliaog commented Apr 25, 2018

@deejay1 please take another look

@yliaog
Copy link
Author

yliaog commented May 2, 2018

@x13n @loburm could you this a look at this PR?

@loburm
Copy link
Member

loburm commented May 8, 2018

Hi Yu,
I'm interested in how have you tested this version of fluentd. Could you add more context to your PR. And maybe description which bugs have been fixed, or which features are introduced would be very helpful.

Thanks

@evanj
Copy link

evanj commented May 25, 2018

The big issue that I think this will fix is the fact that Google Kubernetes Engine truncates logs to second resolution. I think the underlying cause is this fluent-plugin-google-cloud bug, which was fixed in v0.6.9 (already included in this image I think): GoogleCloudPlatform/fluent-plugin-google-cloud#131

However, that will only work with fluentd >= 0.14 since before that it represents time as seconds: https://www.fluentd.org/blog/fluentd-v0.14.0-has-been-released

@evanj
Copy link

evanj commented May 25, 2018

See the GKE issue for this also: https://issuetracker.google.com/issues/35903605

Bumped fluent-plugin-systemd to 0.3.1 to be compatible with fluentd
1.2.5.
@yliaog
Copy link
Author

yliaog commented Oct 17, 2018

@loburm it seems that fluentd 1.2.5 is stable, and qualified, updated this PR to upgrade to v1.2.5. PTAL

@yliaog yliaog changed the title Bumped fluentd from 0.12.32 to 0.14.25 Bumped fluentd from 0.12.32 to 1.2.5 Oct 17, 2018
source 'https://rubygems.org'

gem 'fluentd', '~>0.12.32'
gem 'fluentd', '~>1.2.5'

Choose a reason for hiding this comment

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

Note: we have only qualified this version of fluentd as it pertains to the Docker image we distribute (i.e., the one built from https://github.com/GoogleCloudPlatform/google-fluentd/blob/master/docker/Dockerfile). There is no evidence that it will work properly for this Docker image. Should this not be obsoleted now that we are providing one?
@qingling128 @bmoyles0117 FYI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants