Skip to content

Commit 0432a49

Browse files
authored
ci: fix paths for nightly sdk-platform-java builds (#12338)
* The nightly sdk-platform-java tests should run nightly regardless of whether the last commit touched that folder. Thus we remove the filter that was added during migration * The nightly test is now scoped to just the sdk-platform-java test, but we need to install the necessary dependencies first Tested with workflow_dispatch: https://github.com/googleapis/google-cloud-java/actions/runs/23818326376 Fixes #12206
1 parent 1e0c859 commit 0432a49

2 files changed

Lines changed: 24 additions & 21 deletions

File tree

.github/workflows/sdk-platform-java-nightly.yaml

Lines changed: 14 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -6,24 +6,8 @@ on:
66
defaults:
77
run:
88
shell: bash
9-
env:
10-
BUILD_SUBDIR: sdk-platform-java
119
jobs:
12-
filter:
13-
runs-on: ubuntu-latest
14-
outputs:
15-
library: ${{ steps.filter.outputs.library }}
16-
steps:
17-
- uses: actions/checkout@v4
18-
- uses: dorny/paths-filter@v3
19-
id: filter
20-
with:
21-
filters: |
22-
library:
23-
- 'sdk-platform-java/**'
2410
nightly:
25-
needs: filter
26-
if: ${{ needs.filter.outputs.library == 'true' }}
2711
strategy:
2812
fail-fast: false
2913
matrix:
@@ -39,8 +23,15 @@ jobs:
3923
distribution: temurin
4024
cache: maven
4125
- run: mvn -version
26+
- name: Install all modules first
27+
shell: bash
28+
run: .kokoro/build.sh
29+
env:
30+
BUILD_SUBDIR: sdk-platform-java
31+
JOB_TYPE: install
4232
- name: Install
4333
run: mvn install --errors --batch-mode --no-transfer-progress -Dcheckstyle.skip -Dfmt.skip
34+
working-directory: sdk-platform-java
4435
- name: Create issue if previous step fails
4536
if: ${{ failure() }}
4637
env:
@@ -50,8 +41,6 @@ jobs:
5041
--title "Nightly build for Java ${{ matrix.java }} on ${{ matrix.os }} failed." \
5142
--body "The build has failed : https://github.com/googleapis/google-cloud-java/actions/runs/${GITHUB_RUN_ID}"
5243
nightly-java8: # Compile with JDK 11. Run tests with JDK 8.
53-
needs: filter
54-
if: ${{ needs.filter.outputs.library == 'true' }}
5544
strategy:
5645
fail-fast: false
5746
matrix:
@@ -66,9 +55,12 @@ jobs:
6655
distribution: temurin
6756
cache: maven
6857
- run: mvn -version
69-
- name: Install with Java 11
70-
run: mvn install --errors --batch-mode --no-transfer-progress -Dcheckstyle.skip -DskipTests -Dfmt.skip
71-
58+
- name: Install all modules first
59+
shell: bash
60+
run: .kokoro/build.sh
61+
env:
62+
BUILD_SUBDIR: sdk-platform-java
63+
JOB_TYPE: install
7264
- uses: actions/setup-java@v5
7365
with:
7466
java-version: 8
@@ -77,6 +69,7 @@ jobs:
7769
- name: Test with Java 8
7870
# Direct goal invocation ("surefire:test") prevents recompiling tests
7971
run: mvn surefire:test --errors --batch-mode --no-transfer-progress -Dfmt.skip
72+
working-directory: sdk-platform-java
8073
- name: Create issue if previous step fails
8174
if: ${{ failure() }}
8275
env:

.kokoro/common.sh

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -422,6 +422,16 @@ function install_modules() {
422422
'sdk-platform-java/java-shared-dependencies/first-party-dependencies'
423423
'sdk-platform-java/java-shared-dependencies/third-party-dependencies'
424424
'sdk-platform-java/gapic-generator-java-bom'
425+
'sdk-platform-java/java-iam/grpc-google-iam-v1'
426+
'sdk-platform-java/java-iam/grpc-google-iam-v2'
427+
'sdk-platform-java/java-iam/grpc-google-iam-v2beta'
428+
'sdk-platform-java/java-iam/grpc-google-iam-v3'
429+
'sdk-platform-java/java-iam/grpc-google-iam-v3beta'
430+
'sdk-platform-java/java-iam/proto-google-iam-v1'
431+
'sdk-platform-java/java-iam/proto-google-iam-v2'
432+
'sdk-platform-java/java-iam/proto-google-iam-v2beta'
433+
'sdk-platform-java/java-iam/proto-google-iam-v3'
434+
'sdk-platform-java/java-iam/proto-google-iam-v3beta'
425435
'sdk-platform-java/java-core/google-cloud-core-bom'
426436
'sdk-platform-java/java-core/google-cloud-core'
427437
'sdk-platform-java/java-core/google-cloud-core-grpc'

0 commit comments

Comments
 (0)