Skip to content

Conversation

@ejona86
Copy link
Member

@ejona86 ejona86 commented Sep 30, 2025

io.netty.util.Version is unreliable, so we stop using it. grpc-netty and grpc-netty-shaded have their version.properties mix, and you can't tell which is which.

Changed the tests to use assume, so it is clear in the results that they weren't run.

io.netty.util.Version is unreliable, so we stop using it. grpc-netty and
grpc-netty-shaded have their version.properties mix, and you can't tell
which is which.

Changed the tests to use assume, so it is clear in the results that they
weren't run.
@ejona86 ejona86 requested a review from AgraVator September 30, 2025 18:01
@ejona86 ejona86 added the TODO:backport PR needs to be backported. Removed after backport complete label Sep 30, 2025
@AgraVator AgraVator merged commit 6b83959 into grpc:master Oct 1, 2025
16 of 17 checks passed
AgraVator pushed a commit to AgraVator/grpc-java that referenced this pull request Oct 1, 2025
io.netty.util.Version is unreliable, so we stop using it. grpc-netty and
grpc-netty-shaded have their version.properties mix, and you can't tell
which is which.

Changed the tests to use assume, so it is clear in the results that they
weren't run.
@ejona86 ejona86 deleted the netty-adaptive-disable branch October 1, 2025 16:39
ejona86 added a commit that referenced this pull request Oct 1, 2025
io.netty.util.Version is unreliable, so we stop using it. grpc-netty and
grpc-netty-shaded have their version.properties mix, and you can't tell
which is which.

Changed the tests to use assume, so it is clear in the results that they
weren't run.
@AgraVator AgraVator removed the TODO:backport PR needs to be backported. Removed after backport complete label Oct 3, 2025
@ejona86 ejona86 added TODO:backport PR needs to be backported. Removed after backport complete and removed TODO:backport PR needs to be backported. Removed after backport complete labels Oct 30, 2025
AgraVator pushed a commit to AgraVator/grpc-java that referenced this pull request Nov 3, 2025
io.netty.util.Version is unreliable, so we stop using it. grpc-netty and
grpc-netty-shaded have their version.properties mix, and you can't tell
which is which.

Changed the tests to use assume, so it is clear in the results that they
weren't run.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants