Daijiro Fukuda

Results 519 comments of Daijiro Fukuda

If an external process was opening the file, the update of the storage file would inevitably fail in this way. It is not nice that the tmp files are left,...

Is this issue resolved? Should we update some dependencies? Looks like the gemspec of fluent-plugin-gelf-hs has no problem.

Oh, I see. #219 fixed the version of `gelf` to `3.0.0`, so the image installs `gelf 3.0.0`, currently. Do we not need #219 anymore? If so, we should revert #219.

> Do we not need #219 anymore? Until Fluentd v1.8.0, `gelf-rb 3.1.0` was causing a severe error that Fluentd could not start. The problem was partially fixed by https://github.com/fluent/fluentd/pull/2709 (Fluentd...

`gelf-rb` is no longer maintained. https://github.com/graylog-labs/gelf-rb/issues/93#issuecomment-750307376 It would be better to use [gelf_redux](https://github.com/manet-marketing/gelf_redux). (https://github.com/graylog-labs/gelf-rb/issues/93#issuecomment-1603109468)

> Hmm, CI is failed due to a change of dependent gem. It seems that console gem is needed pinned. I have made the issue for this: * #4487 We...

For `console` v1.25, we need to consider #4487 For `console` v1.24, we need to backport #4477.

@yugeeklab Thanks for this fix! CI is currently unstable because of #4487. We will fix it. Sorry for the trouble. I see the intent of this fix as follows. *...

@yugeeklab > I've just published an issue #4491 for more information. Thanks so much! > as-is: max_line_size helps you avoid buffer overflow configuring via buffer section. to-be: max_line_size helps prevent...

About CI failures, although #4493 has been resolved, we still need to resolve #4487.