Skip to content

Conversation

@Watson1978
Copy link
Contributor

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

What this PR does / why we need it:
Fixes an issue where an inappropriate protocol was specified in the unused_port method.
See #4674 (comment)

Docs Changes:

Release Note:

@daipom daipom self-requested a review October 28, 2024 07:02
@daipom
Copy link
Contributor

daipom commented Oct 28, 2024

Thanks, but it should be tcp.
There must be another reason why this test fails.

ref: #4674 (comment)

@daipom daipom closed this Oct 28, 2024
@daipom daipom reopened this Oct 28, 2024
@daipom
Copy link
Contributor

daipom commented Oct 28, 2024

#4674 (comment)

If there is a possibility of using TCP, it will need to take care of both.

it may be a good idea to take care of both!

Watson1978 and others added 2 commits October 28, 2024 16:44
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.

LGTM. Thanks!

@daipom daipom merged commit add96bb into fluent:master Oct 28, 2024
@Watson1978
Copy link
Contributor Author

Thanks!

@Watson1978 Watson1978 deleted the fix-unused_port branch October 28, 2024 11:03
@kenhys kenhys mentioned this pull request Nov 28, 2024
@daipom daipom added the backport to v1.16 We will backport this fix to the LTS branch label Nov 29, 2024
@daipom daipom added the CI Test/CI issues label Jan 28, 2025
@daipom daipom added this to the v1.18.0 milestone Jan 28, 2025
kenhys pushed a commit that referenced this pull request Jan 29, 2025
kenhys pushed a commit that referenced this pull request Jan 29, 2025
Signed-off-by: Watson <[email protected]>
Co-authored-by: Daijiro Fukuda <[email protected]>
Signed-off-by: Kentaro Hayashi <[email protected]>
daipom added a commit that referenced this pull request Jan 29, 2025
**Which issue(s) this PR fixes**: 

Backport #4686

**What this PR does / why we need it**: 

Fixes an issue where an inappropriate protocol was specified in the
unused_port method.
See
#4674 (comment)

**Docs Changes**:

**Release Note**:

Signed-off-by: Watson <[email protected]>
Signed-off-by: Kentaro Hayashi <[email protected]>
Co-authored-by: Watson <[email protected]>
Co-authored-by: Daijiro Fukuda <[email protected]>
@kenhys kenhys added the backported "backport to LTS" is done label Apr 23, 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 CI Test/CI issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Fails with "Errno::EACCES: Permission denied - bind(2)" on Windows

3 participants