Skip to content

dep: bump netty to 4.1.69.Final and tcnative to 2.0.44.Final#8593

Closed
ran-su wants to merge 2 commits intogrpc:masterfrom
ran-su:netty.4.1.69
Closed

dep: bump netty to 4.1.69.Final and tcnative to 2.0.44.Final#8593
ran-su wants to merge 2 commits intogrpc:masterfrom
ran-su:netty.4.1.69

Conversation

@ran-su
Copy link
Copy Markdown
Member

@ran-su ran-su commented Oct 12, 2021

No description provided.

@dapengzhang0
Copy link
Copy Markdown
Contributor

dapengzhang0 commented Oct 14, 2021

I filed #8605 to track the issue of current failure. cc @sanjaypujare

@dapengzhang0 dapengzhang0 mentioned this pull request Oct 20, 2021
Closed
@dapengzhang0
Copy link
Copy Markdown
Contributor

dapengzhang0 commented Dec 1, 2021

The issue should be fixed in Netty 4.1.71.Final. Let's wait for that release.

@sergiitk
Copy link
Copy Markdown
Member

sergiitk commented Jan 7, 2022

Temporary re-opening this to build artifacts for related investigation.

@sergiitk sergiitk reopened this Jan 7, 2022
@sergiitk sergiitk added the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Jan 7, 2022
@grpc-kokoro grpc-kokoro removed the kokoro:force-run Add this label to a PR to tell Kokoro to re-run all tests. Not generally necessary label Jan 7, 2022
@sergiitk
Copy link
Copy Markdown
Member

sergiitk commented Jan 7, 2022 •
edited
Loading

For some reason, build artifacts were not saved. Closing this to try from my branch #8815

@sergiitk sergiitk closed this Jan 7, 2022
@sanjaypujare
Copy link
Copy Markdown
Contributor

sanjaypujare commented Jan 7, 2022

For some reason, build artifacts were not saved. Closing this to try from my branch #8815

You mean the test reports? You may need the changes I made in https://github.com/grpc/grpc-java/pull/8813/files#diff-a515351cde89a1974efa33a43ab0012a052e0f14deff85710f296934e81cabdbR18 and https://github.com/grpc/grpc-java/pull/8813/files#diff-79ced7352cff631f4849c3a808ab5c28f76a38c8ce8bc0f8f018168e5b79d96bR11

@sergiitk
Copy link
Copy Markdown
Member

sergiitk commented Jan 7, 2022

Yes, this. I thought we already had it at least for linux_artifacts.

@sanjaypujare
Copy link
Copy Markdown
Contributor

sanjaypujare commented Jan 7, 2022

Yes, this. I thought we already had it at least for linux_artifacts.

@dapengzhang0 made this change #8801 recently for linux tests. However the artifacts are accessed differently.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ran-su @dapengzhang0 @sergiitk @sanjaypujare @grpc-kokoro