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.

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

Remove GRPC dependency #54

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Remove GRPC dependency #54

wants to merge 2 commits into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Feb 1, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
io.grpc:grpc-stub 1.45.0 -> 1.45.4 age adoption passing confidence
io.grpc:grpc-protobuf 1.45.0 -> 1.45.4 age adoption passing confidence

⚠ Dependency Lookup Warnings ⚠

Warnings were logged while processing this repo. Please check the Dependency Dashboard for more information.


Release Notes

grpc/grpc-java

v1.45.4

Bug Fixes

v1.45.3

Bug Fixes
  • core: Fix retry causing memory leak for canceled RPCs. (#​9360)

v1.45.2

Bug Fixes

  • xds: fix bugs in ring-hash load balancer picking subchannel behavior per gRFC. The bug may cause connection not failing over from TRANSIENT_FAILURE status. (#​9085)
  • xds: Protect xdstp processing with federation env var. If the xds server uses xdstp:// resource names it was possible for federation code paths to be entered even without enabling the experimental federation support. This is now fixed and it is safe for xds servers to use xdstp:// resource names. (#​9190)

Behavior Changes

  • xds: change ring_hash LB aggregation rule to better handle transient_failure channel status (#​9084)

Dependencies

  • Bump protobuf to 3.19.6
  • bom: Exclude grpc-observability. The module does not exist in 1.45.x. Should be a noop (#​9122)

v1.45.1

Bug Fixes

  • netty: Fixed incompatibility with Netty 4.1.75.Final that caused COMPRESSION_ERROR (#​9004)
  • xds: Fix LBs blindly propagating control plane errors (#​9012). This change forces the use of UNAVAILABLE for any xDS communication failures, which otherwise could greatly confuse an application. This is essentially a continuation of the fix in 1.45.0 for XdsNameResolver, but for other similar cases

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the dependencies An update to project dependencies label Feb 1, 2023
@github-actions
Copy link

github-actions bot commented Feb 1, 2023

Test Results

  12 files  ±0    12 suites  ±0   5m 57s ⏱️ + 1m 24s
  51 tests ±0    45 ✔️ ±0    6 💤 ±0  0 ±0 
127 runs  ±0  116 ✔️ ±0  11 💤 ±0  0 ±0 

Results for commit cc889f1. ± Comparison against base commit 1a90def.

♻️ This comment has been updated with latest results.

@renovate
Copy link
Contributor Author

renovate bot commented Feb 1, 2023

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@rpatel-figure rpatel-figure changed the title Update GRPC from 1.45.0 to 1.45.4 (patch) Remove GRPC dependency Feb 1, 2023
@rpatel-figure rpatel-figure added the patch Bug fixes or other non-breaking configuration changes label Feb 1, 2023
@rpatel-figure
Copy link
Collaborator

AFAIK the GRPC dependency was never actually necessary for the usage of p8e-scope-sdk

dkneisly-figure
dkneisly-figure previously approved these changes Feb 1, 2023
Copy link
Contributor

@dkneisly-figure dkneisly-figure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's give this a shot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies An update to project dependencies patch Bug fixes or other non-breaking configuration changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants