Support Ruby 3.1 on Windows #3766
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For avoiding the following error on Ruby-3.1 on Windows: ``` 2022-02-18T01:36:00.3691879Z =============================================================================== 2022-02-18T01:36:00.3692453Z Failure: test: send tags in str (utf-8 strings)(ForwardOutputTest): 2022-02-18T01:36:00.3692799Z On subject , 2022-02-18T01:36:00.3693302Z unexpected method invocation: 2022-02-18T01:36:00.3693595Z process_wait(2088, 0) 2022-02-18T01:36:00.3702794Z expected invocations: 2022-02-18T01:36:00.3703495Z C:/hostedtoolcache/windows/Ruby/3.1.0/x64/lib/ruby/gems/3.1.0/gems/serverengine-2.2.5/lib/serverengine/socket_manager.rb:79:in ``' 2022-02-18T01:36:00.3704338Z C:/hostedtoolcache/windows/Ruby/3.1.0/x64/lib/ruby/gems/3.1.0/gems/serverengine-2.2.5/lib/serverengine/socket_manager.rb:79:in `block in generate_path' 2022-02-18T01:36:00.3705025Z C:/hostedtoolcache/windows/Ruby/3.1.0/x64/lib/ruby/gems/3.1.0/gems/serverengine-2.2.5/lib/serverengine/socket_manager.rb:78:in `each' 2022-02-18T01:36:00.3705695Z C:/hostedtoolcache/windows/Ruby/3.1.0/x64/lib/ruby/gems/3.1.0/gems/serverengine-2.2.5/lib/serverengine/socket_manager.rb:78:in `generate_path' 2022-02-18T01:36:00.3706156Z D:/a/fluentd/fluentd/lib/fluent/test/driver/base.rb:105:in `instance_start' 2022-02-18T01:36:00.3706550Z D:/a/fluentd/fluentd/lib/fluent/test/driver/base.rb:77:in `run' 2022-02-18T01:36:00.3706945Z D:/a/fluentd/fluentd/lib/fluent/test/driver/base_owner.rb:130:in `run' 2022-02-18T01:36:00.3707422Z D:/a/fluentd/fluentd/test/plugin/test_out_forward.rb:434:in `block in <class:ForwardOutputTest>' 2022-02-18T01:36:00.3707780Z 431: ] 2022-02-18T01:36:00.3708021Z 432: 2022-02-18T01:36:00.3708366Z 433: stub(d.instance.ack_handler).read_ack_from_sock(anything).never 2022-02-18T01:36:00.3708758Z => 434: target_input_driver.run(expect_records: 2) do 2022-02-18T01:36:00.3709053Z 435: d.run do 2022-02-18T01:36:00.3709364Z 436: emit_events.each do |tag, t, record| 2022-02-18T01:36:00.3709681Z 437: d.feed(tag, t, record) 2022-02-18T01:36:00.3709997Z =============================================================================== ``` Signed-off-by: Takuro Ashie <ashie@clear-code.com>
Although we have used WindowsFile as altenertive of File mainly for specifying FILE_SHARE_DELETE to CreateFile, WindowsFile#io always raises Errno::EBADF on Ruby 3.1 on UCRT. Instead we can use File.open with File::Constants::SHARE_DELETE to avoid it. It was introduced for Fluentd, so we should use it :D https://bugs.ruby-lang.org/issues/11218 Signed-off-by: Takuro Ashie <ashie@clear-code.com>
Signed-off-by: Takuro Ashie <ashie@clear-code.com>
|
Although I replaced
|
It seems work fine than WindowsFile, so remove old code to simplify the code. Signed-off-by: Takuro Ashie <ashie@clear-code.com>
cosmo0920
reviewed
May 31, 2022
There was a problem hiding this comment.
From the comment and the CI result on Ruby 3.1 on Windows, it looks good to me! 💪
cosmo0920
approved these changes
May 31, 2022
fujimotos
reviewed
May 31, 2022
fujimotos
approved these changes
May 31, 2022
|
Thanks for your review! |
|
CI on Ruby 2.7 Windows always fails, but it's not concerned with this pull request. |
Signed-off-by: Takuro Ashie <ashie@clear-code.com>
kavirajk
pushed a commit
to grafana/loki
that referenced
this pull request
Oct 10, 2023
…10839) [](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [fluentd](https://www.fluentd.org/) ([source](https://togithub.com/fluent/fluentd)) | `'1.14.2'` -> `'1.15.3'` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | ### GitHub Vulnerability Alerts #### [CVE-2022-39379](https://togithub.com/fluent/fluentd/security/advisories/GHSA-fppq-mj76-fpj2) ### Impact A remote code execution (RCE) vulnerability in non-default configurations of Fluentd allows unauthenticated attackers to execute arbitrary code via specially crafted JSON payloads. Fluentd setups are only affected if the environment variable `FLUENT_OJ_OPTION_MODE` is explicitly set to `object`. Please note: The option FLUENT_OJ_OPTION_MODE was introduced in Fluentd version 1.13.2. Earlier versions of Fluentd are not affected by this vulnerability. ### Patches v1.15.3 ### Workarounds Do not use `FLUENT_OJ_OPTION_MODE=object`. ### References * GHSL-2022-067 --- ### Release Notes <details> <summary>fluent/fluentd (fluentd)</summary> ### [`v1.15.3`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1153---20221102) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.15.2...v1.15.3) ##### Bug Fix - Support glob for `!include` directive in YAML config format [fluent/fluentd#3917 - Remove meaningless oj options [fluent/fluentd#3929 - Fix log initializer to correctly create per-process files on Windows [fluent/fluentd#3939 - out_file: Fix the multi-worker check with `<worker 0-N>` directive [fluent/fluentd#3942 ##### Misc - Fix broken tests on Ruby 3.2 [fluent/fluentd#3883 ### [`v1.15.2`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1152---20220822) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.15.1...v1.15.2) ##### Enhancement - Add a new system configuration `enable_jit` [fluent/fluentd#3857 ##### Bug Fix - out_file: Fix append mode with `--daemon` flag [fluent/fluentd#3864 - child_process: Plug file descriptor leak [fluent/fluentd#3844 ##### Misc - Drop win32-api gem to support Ruby 3.2 [fluent/fluentd#3849 ### [`v1.15.1`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1151---20220727) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.15.0...v1.15.1) ##### Bug Fix - Add support for concurrent append in out_file [fluent/fluentd#3808 ##### Misc - in_tail: Show more information on skipping update_watcher [fluent/fluentd#3829 ### [`v1.15.0`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1150---20220629) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.6...v1.15.0) ##### Enhancement - in_tail: Add log throttling in files based on group rules [fluent/fluentd#3535 - Add `dump` command to fluent-ctl [fluent/fluentd#3680 - Handle YAML configuration format on configuration file [fluent/fluentd#3712 - Add `restart_worker_interval` parameter in `<system>` directive to set interval to restart workers that has stopped for some reas[fluent/fluentd#3768 ##### Bug fixes - out_forward: Fix to update timeout of cached sockets [fluent/fluentd#3711 - in_tail: Fix a possible crash on file rotation when `follow_inodes true` [fluent/fluentd#3754 - output: Fix a possible crash of flush thread [fluent/fluentd#3755 - in_tail: Fix crash bugs on Ruby 3.1 on Windows [fluent/fluentd#3766 - in_tail: Fix a bug that in_tail cannot open non-ascii path on Windows [fluent/fluentd#3774 - Fix a bug that fluentd doesn't release its own log file even after rotated by external to[fluent/fluentd#3782 ##### Misc - in_tail: Simplify TargetInfo related code [fluent/fluentd#3489 - Fix a wrong issue number in CHANGELOG [fluent/fluentd#3700 - server helper: Add comments to linger_timeout behavior about Windows [fluent/fluentd#3701 - service_discovery: Fix typo [fluent/fluentd#3724 - test: Fix unstable tests and warnings [fluent/fluentd#3745 ### [`v1.14.6`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1146---20220331) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.5...v1.14.6) ##### Enhancement - Enable server plugins to specify socket-option `SO_LINGER` [fluent/fluentd#3644 - Add `--umask` command line parameter [fluent/fluentd#3671 ##### Bug fixes - Fix metric name typo [fluent/fluentd#3630 - Apply modifications in pipeline to the records being passed to `@ERROR` label [fluent/fluentd#3631 - Fix wrong calculation of retry interval [fluent/fluentd#3640 - Support IPv6 address for `rpc_endpoint` in `system` config [fluent/fluentd#3641 ##### Misc - CI: Support Ruby 3.1 except Windows [fluent/fluentd#3619 - Switch to GitHub Discussions [fluent/fluentd#3654 - Fix CHANGELOG.md heading styles [fluent/fluentd#3648 - Declare `null_value_pattern` as `regexp` [fluent/fluentd#3650 ### [`v1.14.5`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1145---20220209) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.4...v1.14.5) ##### Enhancement - Add support for "application/x-ndjson" to `in_http` [fluent/fluentd#3616 - Add support for ucrt binary for Windows [fluent/fluentd#3613 ##### Bug fixes - Don't retry when `retry_max_times == 0` [fluent/fluentd#3608 - Fix hang-up issue during TLS handshake in `out_forward` [fluent/fluentd#3601 - Bump up required ServerEngine to v2.2.5 [fluent/fluentd#3599 - Fix "invalid byte sequence is replaced" warning on Kubernetes [fluent/fluentd#3596 - Fix "ArgumentError: unknown keyword: :logger" on Windows with Ruby 3.1 [fluent/fluentd#3592 ### [`v1.14.4`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1144---20220106) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.3...v1.14.4) ##### Enhancement - `in_tail`: Add option to skip long lines (`max_line_size`) [fluent/fluentd#3565 ##### Bug fix - Incorrect BufferChunkOverflowError when each event size is < `chunk_limit_size` [fluent/fluentd#3560 - On macOS with Ruby 2.7/3.0, `out_file` fails to write events if `append` is true. [fluent/fluentd#3579 - test: Fix unstable test cases [fluent/fluentd#3574 ### [`v1.14.3`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1143---20211126) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.2...v1.14.3) ##### Enhancement - Changed to accept `http_parser.rb` 0.8.0. `http_parser.rb` 0.8.0 is ready for Ractor. [fluent/fluentd#3544 ##### Bug fix - in_tail: Fixed a bug that no new logs are read when `enable_stat_watcher true` and `enable_watch_timer false` is set. [fluent/fluentd#3541 - in_tail: Fixed a bug that the beginning and initial lines are lost after startup when `read_from_head false` and path includes wildcard '\*'.[fluent/fluentd#3542 - Fixed a bug that processing messages were lost when BufferChunkOverflowError was thrown even though only a specific message size exceeds chunk_limi[fluent/fluentd#3553 ##### Misc - Bump up required version of `win32-service` gem. newer version is required to implement additional `fluent-ctl` commands. [fluent/fluentd#3556 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/grafana/loki). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44LjEiLCJ1cGRhdGVkSW5WZXIiOiIzNy44LjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
rhnasc
pushed a commit
to inloco/loki
that referenced
this pull request
Apr 12, 2024
](https://renovatebot.com){kind=link}
…rafana#10839) [](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [fluentd](https://www.fluentd.org/) ([source](https://togithub.com/fluent/fluentd)) | `'1.14.2'` -> `'1.15.3'` | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | ### GitHub Vulnerability Alerts #### [CVE-2022-39379](https://togithub.com/fluent/fluentd/security/advisories/GHSA-fppq-mj76-fpj2) ### Impact A remote code execution (RCE) vulnerability in non-default configurations of Fluentd allows unauthenticated attackers to execute arbitrary code via specially crafted JSON payloads. Fluentd setups are only affected if the environment variable `FLUENT_OJ_OPTION_MODE` is explicitly set to `object`. Please note: The option FLUENT_OJ_OPTION_MODE was introduced in Fluentd version 1.13.2. Earlier versions of Fluentd are not affected by this vulnerability. ### Patches v1.15.3 ### Workarounds Do not use `FLUENT_OJ_OPTION_MODE=object`. ### References * GHSL-2022-067 --- ### Release Notes <details> <summary>fluent/fluentd (fluentd)</summary> ### [`v1.15.3`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1153---20221102) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.15.2...v1.15.3) ##### Bug Fix - Support glob for `!include` directive in YAML config format [fluent/fluentd#3917 - Remove meaningless oj options [fluent/fluentd#3929 - Fix log initializer to correctly create per-process files on Windows [fluent/fluentd#3939 - out_file: Fix the multi-worker check with `<worker 0-N>` directive [fluent/fluentd#3942 ##### Misc - Fix broken tests on Ruby 3.2 [fluent/fluentd#3883 ### [`v1.15.2`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1152---20220822) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.15.1...v1.15.2) ##### Enhancement - Add a new system configuration `enable_jit` [fluent/fluentd#3857 ##### Bug Fix - out_file: Fix append mode with `--daemon` flag [fluent/fluentd#3864 - child_process: Plug file descriptor leak [fluent/fluentd#3844 ##### Misc - Drop win32-api gem to support Ruby 3.2 [fluent/fluentd#3849 ### [`v1.15.1`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1151---20220727) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.15.0...v1.15.1) ##### Bug Fix - Add support for concurrent append in out_file [fluent/fluentd#3808 ##### Misc - in_tail: Show more information on skipping update_watcher [fluent/fluentd#3829 ### [`v1.15.0`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1150---20220629) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.6...v1.15.0) ##### Enhancement - in_tail: Add log throttling in files based on group rules [fluent/fluentd#3535 - Add `dump` command to fluent-ctl [fluent/fluentd#3680 - Handle YAML configuration format on configuration file [fluent/fluentd#3712 - Add `restart_worker_interval` parameter in `<system>` directive to set interval to restart workers that has stopped for some reas[fluent/fluentd#3768 ##### Bug fixes - out_forward: Fix to update timeout of cached sockets [fluent/fluentd#3711 - in_tail: Fix a possible crash on file rotation when `follow_inodes true` [fluent/fluentd#3754 - output: Fix a possible crash of flush thread [fluent/fluentd#3755 - in_tail: Fix crash bugs on Ruby 3.1 on Windows [fluent/fluentd#3766 - in_tail: Fix a bug that in_tail cannot open non-ascii path on Windows [fluent/fluentd#3774 - Fix a bug that fluentd doesn't release its own log file even after rotated by external to[fluent/fluentd#3782 ##### Misc - in_tail: Simplify TargetInfo related code [fluent/fluentd#3489 - Fix a wrong issue number in CHANGELOG [fluent/fluentd#3700 - server helper: Add comments to linger_timeout behavior about Windows [fluent/fluentd#3701 - service_discovery: Fix typo [fluent/fluentd#3724 - test: Fix unstable tests and warnings [fluent/fluentd#3745 ### [`v1.14.6`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1146---20220331) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.5...v1.14.6) ##### Enhancement - Enable server plugins to specify socket-option `SO_LINGER` [fluent/fluentd#3644 - Add `--umask` command line parameter [fluent/fluentd#3671 ##### Bug fixes - Fix metric name typo [fluent/fluentd#3630 - Apply modifications in pipeline to the records being passed to `@ERROR` label [fluent/fluentd#3631 - Fix wrong calculation of retry interval [fluent/fluentd#3640 - Support IPv6 address for `rpc_endpoint` in `system` config [fluent/fluentd#3641 ##### Misc - CI: Support Ruby 3.1 except Windows [fluent/fluentd#3619 - Switch to GitHub Discussions [fluent/fluentd#3654 - Fix CHANGELOG.md heading styles [fluent/fluentd#3648 - Declare `null_value_pattern` as `regexp` [fluent/fluentd#3650 ### [`v1.14.5`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1145---20220209) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.4...v1.14.5) ##### Enhancement - Add support for "application/x-ndjson" to `in_http` [fluent/fluentd#3616 - Add support for ucrt binary for Windows [fluent/fluentd#3613 ##### Bug fixes - Don't retry when `retry_max_times == 0` [fluent/fluentd#3608 - Fix hang-up issue during TLS handshake in `out_forward` [fluent/fluentd#3601 - Bump up required ServerEngine to v2.2.5 [fluent/fluentd#3599 - Fix "invalid byte sequence is replaced" warning on Kubernetes [fluent/fluentd#3596 - Fix "ArgumentError: unknown keyword: :logger" on Windows with Ruby 3.1 [fluent/fluentd#3592 ### [`v1.14.4`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1144---20220106) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.3...v1.14.4) ##### Enhancement - `in_tail`: Add option to skip long lines (`max_line_size`) [fluent/fluentd#3565 ##### Bug fix - Incorrect BufferChunkOverflowError when each event size is < `chunk_limit_size` [fluent/fluentd#3560 - On macOS with Ruby 2.7/3.0, `out_file` fails to write events if `append` is true. [fluent/fluentd#3579 - test: Fix unstable test cases [fluent/fluentd#3574 ### [`v1.14.3`](https://togithub.com/fluent/fluentd/blob/HEAD/CHANGELOG.md#Release-v1143---20211126) [Compare Source](https://togithub.com/fluent/fluentd/compare/v1.14.2...v1.14.3) ##### Enhancement - Changed to accept `http_parser.rb` 0.8.0. `http_parser.rb` 0.8.0 is ready for Ractor. [fluent/fluentd#3544 ##### Bug fix - in_tail: Fixed a bug that no new logs are read when `enable_stat_watcher true` and `enable_watch_timer false` is set. [fluent/fluentd#3541 - in_tail: Fixed a bug that the beginning and initial lines are lost after startup when `read_from_head false` and path includes wildcard '\*'.[fluent/fluentd#3542 - Fixed a bug that processing messages were lost when BufferChunkOverflowError was thrown even though only a specific message size exceeds chunk_limi[fluent/fluentd#3553 ##### Misc - Bump up required version of `win32-service` gem. newer version is required to implement additional `fluent-ctl` commands. [fluent/fluentd#3556 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/grafana/loki). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44LjEiLCJ1cGRhdGVkSW5WZXIiOiIzNy44LjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Which issue(s) this PR fixes:
Fixes #3585
What this PR does / why we need it:
Currently in_tail is broken on Ruby 3.1 on Windows.
In addition, some out_forward tests are broken too.
This PR fixes these issues.
In addition, this PR add CI for Ruby 3.1 and head on Windows
Docs Changes:
None.
Release Note:
Same with the title.