Skip to content

[Re-opened for testing] Bump io.grpc:grpc-context from 1.56.1 to 1.57.1 in /plugins/repository-gcs #9151

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
Closed
wants to merge 40 commits into from
Closed

[Re-opened for testing] Bump io.grpc:grpc-context from 1.56.1 to 1.57.1 in /plugins/repository-gcs #9151

wants to merge 40 commits into from

Conversation

@cwperks
Copy link
Member

@cwperks cwperks commented Aug 7, 2023 •
edited
Loading

Description

Creating manual PR of dependabot version bump since it was failing due to missing classes. This updates the ignoreMissingClasses portion of the repository-gcs/build.gradle file to resolve the issue

Testing out fixes for ./gradlew :modules:transport-netty4:test --tests SimpleSecureNetty4TransportTests.testFailToSend that is failing on PR#14912 and cannot be reproduced locally or on a dev desktop

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
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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
Bump io.grpc:grpc-context from 1.56.1 to 1.57.1 in /plugins/repositor…
7a352cd 
…y-gcs

Signed-off-by: Craig Perkins <cwperx@amazon.com>
cwperks added 2 commits August 7, 2023 12:44
@cwperks
Update CHANGELOG
22bf841 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks
Remove paren
04c9e1e 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2023

Gradle Check (Jenkins) Run Completed with:

@opensearch-trigger-bot
Copy link
Contributor

Compatibility status:



> Task :checkCompatibility
Incompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/security-analytics.git]
Compatible components: [https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git]

BUILD SUCCESSFUL in 27m 8s

@opensearch-trigger-bot
Copy link
Contributor

Compatibility status:



> Task :checkCompatibility
Incompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/security-analytics.git]
Compatible components: [https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git]

BUILD SUCCESSFUL in 26m 55s

@github-actions
Copy link
Contributor

github-actions bot commented Aug 7, 2023

Gradle Check (Jenkins) Run Completed with:

@cwperks
Explicitly close and allow 5s linger
c86e827 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for c86e827: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@cwperks
Explicitly close connection
4548fc9 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for 4548fc9: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 12, 2025
Open
@cwperks
Merge branch 'main' into bump-grpc
5ff2232
@cwperks
Revert "Explicitly close connection"
10d8a81 
This reverts commit 4548fc9.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks
Revert "Explicitly close and allow 5s linger"
0b1601c 
This reverts commit c86e827.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for 0b1601c: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@cwperks
Add debug statement
d96f070 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for d96f070: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@cwperks
Increase timeout to 25
e965361 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for e965361: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@cwperks
Explicitly close TS_C and TS_D
830a92b 
Signed-off-by: Craig Perkins <cwperx@amazon.com>
@github-actions
Copy link
Contributor

❌ Gradle check result for 830a92b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for 830a92b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for 830a92b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 13, 2025
Open
@github-actions
Copy link
Contributor

❌ Gradle check result for 830a92b: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for 7b9e44a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for 7b9e44a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 14, 2025
Open
@github-actions
Copy link
Contributor

✅ Gradle check result for 7b9e44a: SUCCESS

@codecov
Copy link

codecov bot commented Feb 15, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 71.15385% with 60 lines in your changes missing coverage. Please review.

Project coverage is 72.31%. Comparing base (e34422b) to head (7b9e44a).
Report is 27 commits behind head on main.

Files with missing lines Patch % Lines
...opensearch/gradle/testclusters/OpenSearchNode.java 0.00% 9 Missing ⚠️
...ava/org/opensearch/common/ssl/KeyStoreFactory.java 50.00% 7 Missing and 1 partial ⚠️
.../main/java/org/opensearch/bootstrap/Bootstrap.java 0.00% 5 Missing ⚠️
.../opensearch/bootstrap/SecureRandomInitializer.java 37.50% 4 Missing and 1 partial ⚠️
...arch/gradle/precommit/TestingConventionsTasks.java 0.00% 4 Missing ⚠️
...g/opensearch/tools/launchers/SystemJvmOptions.java 71.42% 0 Missing and 4 partials ⚠️
...g/opensearch/common/ssl/TrustEverythingConfig.java 0.00% 3 Missing and 1 partial ⚠️
...rg/opensearch/gradle/OpenSearchTestBasePlugin.java 0.00% 3 Missing ⚠️
.../opensearch/gradle/info/GlobalBuildInfoPlugin.java 40.00% 3 Missing ⚠️
...va/org/opensearch/common/ssl/SslConfiguration.java 25.00% 2 Missing and 1 partial ⚠️
... and 10 more
Additional details and impacted files 
@@             Coverage Diff              @@
##               main    #9151      +/-   ##
============================================
- Coverage     72.40%   72.31%   -0.10%     
+ Complexity    65524    65475      -49     
============================================
  Files          5291     5296       +5     
  Lines        304336   304203     -133     
  Branches      44181    44147      -34     
============================================
- Hits         220360   219988     -372     
- Misses        65954    66187     +233     
- Partials      18022    18028       +6

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 25, 2025
Closed
@cwperks cwperks closed this Feb 26, 2025
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Feb 28, 2025
Open
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Mar 7, 2025
Open
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Apr 24, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@reta reta Awaiting requested review from reta

@anasalkouz anasalkouz Awaiting requested review from anasalkouz

@andrross andrross Awaiting requested review from andrross

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE

@dblock dblock Awaiting requested review from dblock

@gbbafna gbbafna Awaiting requested review from gbbafna

@setiah setiah Awaiting requested review from setiah

@kartg kartg Awaiting requested review from kartg

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal

@mch2 mch2 Awaiting requested review from mch2

@nknize nknize Awaiting requested review from nknize

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159

@ryanbogan ryanbogan Awaiting requested review from ryanbogan

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli

@shwetathareja shwetathareja Awaiting requested review from shwetathareja

@dreamer-89 dreamer-89 Awaiting requested review from dreamer-89

@tlfeng tlfeng Awaiting requested review from tlfeng

@VachaShah VachaShah Awaiting requested review from VachaShah

@dbwiddis dbwiddis Awaiting requested review from dbwiddis

@sachinpkale sachinpkale Awaiting requested review from sachinpkale

@sohami sohami Awaiting requested review from sohami

@peternied peternied Awaiting requested review from peternied peternied is a code owner

@ashking94 ashking94 Awaiting requested review from ashking94

@jainankitk jainankitk Awaiting requested review from jainankitk

@linuxpi linuxpi Awaiting requested review from linuxpi

@msfroh msfroh Awaiting requested review from msfroh

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cwperks @iigonin