Skip to content

[1.3] Bump netty to match version from core's buildSrc/version.properties and bump grpc to 1.56.1#546

Merged
cwperks merged 2 commits intoopensearch-project:1.3from
cwperks:bump-netty-and-grpc
Apr 22, 2024
Merged

[1.3] Bump netty to match version from core's buildSrc/version.properties and bump grpc to 1.56.1#546
cwperks merged 2 commits intoopensearch-project:1.3from
cwperks:bump-netty-and-grpc

Conversation

@cwperks
Copy link
Copy Markdown
Member

@cwperks cwperks commented Apr 20, 2024

Is your feature request related to a problem? Please provide an existing Issue # , or describe.

This PR will match the build.gradle file on main here: https://github.com/opensearch-project/performance-analyzer-rca/blob/main/build.gradle#L373-L377

The dependencies on main should be updated or dependabot added to this repo if its not enabled already.

Without this change performance-analyzer-rca is being bundled with netty-codec-http-4.1.87.Final.jar, netty-codec-http2-4.1.87.Final.jar and netty-handler-4.1.87.Final.jar which are affected by https://avd.aquasec.com/nvd/cve-2024-29025, GHSA-xpw8-rcwv-8f8p and https://avd.aquasec.com/nvd/cve-2023-34462 respectively.

Check List

  • Backport Labels added.
  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

cwperks added 2 commits April 19, 2024 20:19
…nd bump grpc to 1.56.1

Signed-off-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@kkhatua kkhatua requested a review from dzane17 April 22, 2024 18:31
@bbarani
Copy link
Copy Markdown
Member

bbarani commented Apr 22, 2024

@sgup432 can you please review it?

@sgup432
Copy link
Copy Markdown
Contributor

sgup432 commented Apr 22, 2024

@cwperks
I see main branch has implementation 'io.grpc:grpc-stub:1.52.1', do we need to update it to 1.56.1 there as well?

@cwperks
Copy link
Copy Markdown
Member Author

cwperks commented Apr 22, 2024

@sgup432 Yes, main needs to be updated as well. I opened up a PR to add dependabot to automatically monitor for outdated dependencies and create PRs.

In the previous commit I made this PR exactly match main and it gave the following error:

Execution failed for task ':updateShas'.
> Could not resolve all dependencies for configuration ':runtimeClasspath'.
   > Conflict(s) found for the following module(s):
       - io.grpc:grpc-api between versions 1.56.1 and 1.52.1
     Run with:
         --scan or
         :dependencyInsight --configuration runtimeClasspath --dependency io.grpc:grpc-api
     to get more insight on how to solve the conflict.

ref: https://github.com/opensearch-project/performance-analyzer-rca/actions/runs/8761123266/job/24047189857?pr=546

@cwperks cwperks merged commit 8bd64c0 into opensearch-project:1.3 Apr 22, 2024
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.

4 participants