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

fix(titus/build): use version 1.37.1 of io.grpc.* to fix compiler errors (#5487) #5517

Merged
merged 2 commits into from
Sep 9, 2021

Conversation

link108
Copy link
Member

@link108 link108 commented Sep 9, 2021

Fix in favor of #5514. The guava version being pulled in was causing issues.

dbyron-sf and others added 2 commits September 8, 2021 18:48
…ors (spinnaker#5487)

* chore(dependencies): Autobump korkVersion

* fix(titus/build): use version 1.37.1 of io.grpc.* to fix compiler errors

> Task :clouddriver-titus:compileJava FAILED
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/LoadBalancerServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/JobManagementServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/EvictionServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/SupervisorServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/TaskRelocationServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/AutoScalingServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/v4/ResourceConsumptionServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/v4/ResourceAllocationServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/v4/ResourceAllocationArchiveServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/v4/TaskManagementServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/v4/MachineServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/v4/TaskManagementArchiveServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/AgentManagementServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/ClusterMembershipServiceGrpc.java:14: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/SchedulerServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/grpc/protogen/HealthGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/UserIPServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
/home/runner/work/clouddriver/clouddriver/clouddriver-titus/build/generated/source/proto/main/grpc/com/netflix/titus/ValidatorIPServiceGrpc.java:10: error: cannot find symbol
@io.grpc.stub.annotations.GrpcGenerated
                         ^
  symbol:   class GrpcGenerated
  location: package io.grpc.stub.annotations
18 errors

Note that before this PR, gradle used version 1.27.2 of io.grpc.*:

$ ./gradlew :clouddriver-titus:dI --dependency io.grpc:grpc-stub

> Task :clouddriver-titus:dependencyInsight
io.grpc:grpc-stub:1.27.2 (by constraint)
   variant "compile" [
      org.gradle.status                  = release (not requested)
      org.gradle.usage                   = java-api
      org.gradle.libraryelements         = jar (compatible with: classes+resources)
      org.gradle.category                = library

      Requested attributes not found in the selected variant:
         org.gradle.dependency.bundling     = external
         org.jetbrains.kotlin.platform.type = jvm
         org.gradle.jvm.version             = 11
   ]

io.grpc:grpc-stub:1.27.2
\--- io.spinnaker.kork:kork-bom:7.116.0
     \--- compileClasspath

io.grpc:grpc-stub:[1.10.0,) -> 1.27.2
\--- compileClasspath

I haven't been able to find a place in kork that specifies anything for io.grpc, nor where
1.27.2 comes from.  I chose version 1.37.1 because it's the newest version that works, yet
sticks with guava 30.0-jre.  Anything newer and we end up with 30.1-android.  Now we have:

$ ./gradlew :clouddriver-titus:dI --dependency io.grpc:grpc-stub

> Task :clouddriver-titus:dependencyInsight
io.grpc:grpc-stub:1.37.1
   variant "compile" [
      org.gradle.status                  = release (not requested)
      org.gradle.usage                   = java-api
      org.gradle.libraryelements         = jar (compatible with: classes+resources)
      org.gradle.category                = library

      Requested attributes not found in the selected variant:
         org.gradle.dependency.bundling     = external
         org.jetbrains.kotlin.platform.type = jvm
         org.gradle.jvm.version             = 11
   ]
   Selection reasons:
      - By constraint
      - By conflict resolution : between versions 1.37.1 and 1.27.2

io.grpc:grpc-stub:1.37.1
\--- compileClasspath

io.grpc:grpc-stub:1.27.2 -> 1.37.1
\--- io.spinnaker.kork:kork-bom:7.116.0
     \--- compileClasspath

Co-authored-by: root <root@11084802b712>
(cherry picked from commit 71f8f0d)

# Conflicts:
#	gradle.properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants