Skip to content

ci: linkage monitor CI job to use build.sh #947

Merged
gcf-merge-on-green[bot] merged 1 commit intogoogleapis:masterfrom
suztomo:linkage_monitor_option
Feb 16, 2021
Merged

ci: linkage monitor CI job to use build.sh #947
gcf-merge-on-green[bot] merged 1 commit intogoogleapis:masterfrom
suztomo:linkage_monitor_option

Conversation

@suztomo
Copy link
Member

@suztomo suztomo commented Feb 12, 2021

This PR updates the GitHub Actions configuration for Linkage Monitor. Before this change, the CI configuration has "mvn install" command with 6 options. It turned out that a slight difference in the command line option from "build.sh" fails the build in google-auth-library-java repository (GoogleCloudPlatform/cloud-opensource-java#1941). (@Neenu1995 found the build failure. Thanks!)

Therefore, this change updates the configuration so that Linkage Monitor job just uses "build.sh" to install the artifacts into local Maven repository.

How did I test?

I tested this change in java-spanner repository with this draft PR: googleapis/java-spanner#868

Screen Shot 2021-02-12 at 16 54 18

It succeeded.

@suztomo suztomo requested a review from a team February 12, 2021 21:57
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Feb 12, 2021
-Denforcer.skip=true \
-Dmaven.javadoc.skip=true \
-Dgcloud.download.skip=true
run: .kokoro/build.sh
Copy link
Member Author

Choose a reason for hiding this comment

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

The build.sh has the following mvn command.

retry_with_backoff 3 10 \
  mvn install -B -V \
    -DskipTests=true \
    -Dclirr.skip=true \
    -Denforcer.skip=true \
    -Dmaven.javadoc.skip=true \
    -Dgcloud.download.skip=true \
    -T 1C

Without JOB_TYPE environment variable, the script just finishes the installation of the artifacts to the local Maven repository. Suitable behavior for Linkage Monitor.

@suztomo suztomo changed the title ci: simplifying linkage monitor CI configuraiton ci: linkage monitor CI job to use build.sh Feb 12, 2021
@Neenu1995 Neenu1995 added the automerge Merge the pull request once unit tests and other checks pass. label Feb 16, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit f327d3b into googleapis:master Feb 16, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants