Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Backport 1.2] Upgrading Netty to 4.1.72-Final #1890

Merged
merged 1 commit into from
Jan 12, 2022
Merged

[Backport 1.2] Upgrading Netty to 4.1.72-Final #1890

merged 1 commit into from
Jan 12, 2022

Conversation

saratvemulapalli
Copy link
Member

Description

Backporting to 1.2 #1831

Check List

  • 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.

@saratvemulapalli
Updatting Netty to 4.1.72.Final (#1831)
b657945 
Signed-off-by: Sarat Vemulapalli <vemulapallisarat@gmail.com>
@saratvemulapalli saratvemulapalli requested a review from a team as a code owner January 12, 2022 00:50
@saratvemulapalli saratvemulapalli changed the base branch from main to 1.2 January 12, 2022 00:50
@saratvemulapalli saratvemulapalli added >upgrade Label used when upgrading library dependencies (e.g., Lucene) backport PRs or issues specific to backporting features or enhancments backport 1.2 v1.2.4 CVE Fixes a CVE labels Jan 12, 2022
@opensearch-ci-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure b657945
Log 1868

Reports 1868

@saratvemulapalli
Copy link
Member Author

❌   Gradle Check failure b657945 Log 1868

Reports 1868

:distribution:tools:plugin-cli:test
REPRODUCE WITH: ./gradlew ':distribution:tools:plugin-cli:test' --tests "org.opensearch.plugins.InstallPluginCommandTests" -Dtests.seed=2BD6C73AFF7F8FA9 -Dtests.security.manager=false -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=ko-KR -Dtests.timezone=Asia/Tel_Aviv -Druntime.java=15

@saratvemulapalli
Copy link
Member Author

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure b657945
Log 1870

Reports 1870

@dblock
Copy link
Member

dblock commented Jan 12, 2022

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success b657945
Log 1876

Reports 1876

@saratvemulapalli saratvemulapalli merged commit c7e9b2d into opensearch-project:1.2 Jan 12, 2022
@saratvemulapalli saratvemulapalli deleted the 1.2-netty-upgrade branch January 12, 2022 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reta reta reta approved these changes

@dblock dblock dblock approved these changes

@tlfeng tlfeng tlfeng approved these changes

Assignees
No one assigned
Labels
backport PRs or issues specific to backporting features or enhancments backport 1.2 CVE Fixes a CVE >upgrade Label used when upgrading library dependencies (e.g., Lucene) v1.2.4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@saratvemulapalli @opensearch-ci-bot @dblock @reta @tlfeng