Skip to content

Conversation

@Watson1978
Copy link
Contributor

Which issue(s) this PR fixes:
Fixes #4742

What this PR does / why we need it:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

Docs Changes:

Release Note:

This reverts commit b193dc4.

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Copy link
Contributor

@daipom daipom left a comment

Choose a reason for hiding this comment

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

Thanks!!

@daipom daipom merged commit c19032a into fluent:master Feb 17, 2025
10 checks passed
@Watson1978 Watson1978 deleted the logger branch February 17, 2025 02:33
@daipom daipom added this to the v1.19.0 milestone Apr 23, 2025
@daipom daipom added the backport to v1.16 We will backport this fix to the LTS branch label Apr 23, 2025
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
kenhys pushed a commit to kenhys/fluentd that referenced this pull request Apr 23, 2025
…" (fluent#4833)

**Which issue(s) this PR fixes**:
Fixes fluent#4742

**What this PR does / why we need it**:
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
daipom pushed a commit that referenced this pull request Apr 24, 2025
…tion (#4747)" (#4833) (#4923)

**Which issue(s) this PR fixes**: 
Backport #4833
Fixes #4742

**What this PR does / why we need it**: 
This reverts commit b193dc4.
Because ruby/logger#107 was fixed.

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
@daipom daipom added the backported "backport to LTS" is done label Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to v1.16 We will backport this fix to the LTS branch backported "backport to LTS" is done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Fails "test_reopen(LogTest)" test with "TypeError: no implicit conversion of Pathname into String" since logger v1.6.3 on Windows

2 participants