Skip to content
This repository has been archived by the owner on Sep 16, 2023. It is now read-only.

feat: add enforce linkage checker #51

Closed
wants to merge 3 commits into from

Conversation

athakor
Copy link
Contributor

@athakor athakor commented Jan 2, 2020

Towards #6396

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 2, 2020
@athakor athakor closed this Jan 2, 2020
@athakor athakor reopened this Jan 2, 2020
@codecov
Copy link

codecov bot commented Jan 2, 2020

Codecov Report

Merging #51 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #51   +/-   ##
=========================================
  Coverage     58.34%   58.34%           
  Complexity      117      117           
=========================================
  Files            18       18           
  Lines          1145     1145           
  Branches          3        3           
=========================================
  Hits            668      668           
  Misses          475      475           
  Partials          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b62b256...11409ec. Read the comment docs.

@athakor athakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 2, 2020
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 2, 2020
@elharo
Copy link
Contributor

elharo commented Jan 2, 2020

We need to fix this before we can turn on this check:

Dependency convergence error for com.google.errorprone:error_prone_annotations:2.3.3 paths to dependency are:
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-io.grpc:grpc-api:1.26.0
+-com.google.errorprone:error_prone_annotations:2.3.3
and
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-com.google.guava:guava:28.2-android
+-com.google.errorprone:error_prone_annotations:2.3.4

[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 com.google.errorprone:error_prone_annotations:2.3.3 paths to dependency are:
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-io.grpc:grpc-api:1.26.0
+-com.google.errorprone:error_prone_annotations:2.3.3
and
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-com.google.guava:guava:28.2-android
+-com.google.errorprone:error_prone_annotations:2.3.4
]

@athakor
Copy link
Contributor Author

athakor commented Jan 3, 2020

We need to fix this before we can turn on this check:

Dependency convergence error for com.google.errorprone:error_prone_annotations:2.3.3 paths to dependency are:
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-io.grpc:grpc-api:1.26.0
+-com.google.errorprone:error_prone_annotations:2.3.3
and
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-com.google.guava:guava:28.2-android
+-com.google.errorprone:error_prone_annotations:2.3.4

[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 com.google.errorprone:error_prone_annotations:2.3.3 paths to dependency are:
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-io.grpc:grpc-api:1.26.0
+-com.google.errorprone:error_prone_annotations:2.3.3
and
+-com.google.api.grpc:grpc-google-cloud-asset-v1:0.81.1-SNAPSHOT
+-com.google.guava:guava:28.2-android
+-com.google.errorprone:error_prone_annotations:2.3.4
]

I tried to resolve it but we have to wait until this pr gets merge grpc/grpc-java#6574

@athakor athakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 4, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 4, 2020
@athakor athakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@athakor athakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@athakor athakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@pmakani pmakani added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
Copy link
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

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

Test failure looks related though since it's only on Java 8 maybe you just need to clear caches or some such:

[ERROR] Failed to execute goal on project proto-google-cloud-asset-v1: Could not resolve dependencies for project com.google.api.grpc:proto-google-cloud-asset-v1:jar:0.81.1-SNAPSHOT: Could not transfer artifact com.google.errorprone:error_prone_annotations:jar:2.3.4 from/to central (https://repo.maven.apache.org/maven2): /root/.m2/repository/com/google/errorprone/error_prone_annotations/2.3.4/error_prone_annotations-2.3.4.jar.part (No such file or directory) -> [Help 1]
[ERROR]

@athakor athakor added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jan 7, 2020
@athakor
Copy link
Contributor Author

athakor commented Jan 7, 2020

@elharo PTAL

Copy link
Contributor

@elharo elharo left a comment

Choose a reason for hiding this comment

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

This is upper bounds deps, not the linkage checker

@athakor
Copy link
Contributor Author

athakor commented Jan 17, 2020

This enforcer config is already present in the shared-config since at least version 0.3.0 as can be seen here

@athakor athakor closed this Jan 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
6 participants