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

deps: regenerating with new Protobuf (1.111.0-sp) #1382

Merged
merged 3 commits into from
Nov 3, 2022

Conversation

suztomo
Copy link
Member

@suztomo suztomo commented Nov 2, 2022

To merge into 1.111.0-sp branch. This PR was generated using Protobuf 3.16.3. Source-Link: googleapis/googleapis@41d8fbf

GraalVM tests are expected to fail because we didn't have GraalVM configuration file when 6.4.4 was released.

dependencies check is expected to fail because there's opencenssu-api version not consistent. As long as the tests pass, it's not a problem.

@suztomo suztomo requested review from a team as code owners November 2, 2022 15:34
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. api: pubsub Issues related to the googleapis/java-pubsub API. labels Nov 2, 2022
@suztomo
Copy link
Member Author

suztomo commented Nov 2, 2022

dependencies (8) is failing:

Warning:  Rule 2: org.apache.maven.plugins.enforcer.RequireUpperBoundDeps failed with message:
Failed while enforcing RequireUpperBoundDeps. The error(s) are [
Require upper bound dependencies error for io.opencensus:opencensus-api:0.24.0 paths to dependency are:
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-io.opencensus:opencensus-api:0.24.0
and
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-com.google.api:gax:1.60.1
    +-io.opencensus:opencensus-api:0.24.0 (managed) <-- io.opencensus:opencensus-api:0.28.0
and
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-com.google.http-client:google-http-client:1.38.0
    +-io.opencensus:opencensus-api:0.24.0 (managed) <-- io.opencensus:opencensus-api:0.28.0
and
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-io.opencensus:opencensus-impl:0.24.0
    +-io.opencensus:opencensus-api:0.24.0 (managed) <-- io.opencensus:opencensus-api:0.28.0
and
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-com.google.api:gax-grpc:1.60.1
    +-io.grpc:grpc-auth:1.36.3 (managed) <-- io.grpc:grpc-auth:1.32.2
      +-io.opencensus:opencensus-api:0.24.0 (managed) <-- io.opencensus:opencensus-api:0.28.0
and
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-com.google.http-client:google-http-client:1.38.0
    +-io.opencensus:opencensus-contrib-http-util:0.24.0
      +-io.opencensus:opencensus-api:0.24.0 (managed) <-- io.opencensus:opencensus-api:0.28.0
and
+-com.google.cloud:google-cloud-pubsub:1.111.0-sp.3-SNAPSHOT
  +-io.opencensus:opencensus-impl:0.24.0
    +-io.opencensus:opencensus-impl-core:0.24.0
      +-io.opencensus:opencensus-api:0.24.0 (managed) <-- io.opencensus:opencensus-api:0.28.0

It's not related to gRPC or Protobuf.

Linakge Monitor compares the library with the latest version of
other Cloud libraries. It does not work for old branches.
@mmicatka mmicatka added the kokoro:run Add this label to force Kokoro to re-run the tests. label Nov 3, 2022
Copy link
Contributor

@mmicatka mmicatka left a comment

Choose a reason for hiding this comment

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

LGTM pending build passing

@suztomo
Copy link
Member Author

suztomo commented Nov 3, 2022

Removed dependencies from required checks.

@suztomo suztomo merged commit 58d55aa into 1.111.0-sp Nov 3, 2022
@suztomo suztomo deleted the 1.111.0-sp_patch branch November 3, 2022 16:10
gcf-merge-on-green bot pushed a commit that referenced this pull request Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the googleapis/java-pubsub API. kokoro:run Add this label to force Kokoro to re-run the tests. size: xl Pull request size is extra large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants