Skip to content

Commit a6cfb23

Browse files
chore: update to deprecate reusable actions
1 parent ae218a5 commit a6cfb23

5 files changed

Lines changed: 10 additions & 25 deletions

File tree

‎.github/workflows/codegen-ci.yml‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,8 @@ env:
3131
TS_JUNIT_OUTPUT_DIR: results/sdk-codegen
3232

3333
jobs:
34-
setup:
35-
uses: looker-open-source/reusable-actions/.github/workflows/primary-version.yml@main
3634

3735
unit:
38-
needs: setup
3936
env:
4037
JEST_JUNIT_OUTPUT_DIR: results/sdk-codegen
4138
JEST_JUNIT_OUTPUT_NAME: ubuntu-latest.sdk-codegen15x.xml
@@ -71,9 +68,9 @@ jobs:
7168
# TODO: can we cache some layers of the image for faster download?
7269
# we probably don't want to cache the final image for IP security...
7370
run: |
74-
docker pull --quiet us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(needs.setup.outputs.matrix_json)[0] }}
71+
docker pull --quiet us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(vars.PRIMARY_LOOKER_VERSION)[0] }}
7572
# set $LOOKER_OPTS to --no-ssl if we want to turn off ssl
76-
docker run --name looker-sdk-codegen-ci -d -p 10000:9999 -p 20000:19999 us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(needs.setup.outputs.matrix_json)[0] }}
73+
docker run --name looker-sdk-codegen-ci -d -p 10000:9999 -p 20000:19999 us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(vars.PRIMARY_LOOKER_VERSION)[0] }}
7774
docker logs -f looker-sdk-codegen-ci --until=30s &
7875
7976
- uses: actions/setup-node@v4

‎.github/workflows/go-ci.yml‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,8 @@ jobs:
6969
name: ${{ env.JUNIT_ARTIFACT_DIR }}-${{ matrix.os }}-${{ matrix.go-version }}
7070
path: ${{ env.JUNIT_OUTPUT_DIR }}
7171

72-
setup:
73-
uses: looker-open-source/reusable-actions/.github/workflows/supported-versions.yml@main
74-
7572
integration:
76-
needs: [setup, unit]
73+
needs: [unit]
7774
name: Integration - ${{ matrix.os }} / Looker.${{ matrix.looker }}
7875
env:
7976
LOOKERSDK_CLIENT_ID: ${{ secrets.LOOKERSDK_CLIENT_ID }}
@@ -89,7 +86,7 @@ jobs:
8986
os:
9087
- ubuntu
9188
looker:
92-
${{ fromJson(needs.setup.outputs.matrix_json) }}
89+
${{ fromJson(vars.SUPPORTED_LOOKER_VERSIONS) }}
9390

9491
# TODO uncomment `include:` when either macos or windows works to satisfaction.
9592
#include:

‎.github/workflows/lerna-publish.yml‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,7 @@ on:
1111
required: true
1212

1313
jobs:
14-
setup:
15-
uses: looker-open-source/reusable-actions/.github/workflows/primary-version.yml@main
16-
1714
publish:
18-
needs: setup
1915
if: >
2016
(github.event.inputs) ||
2117
(startsWith(github.event.release.tag_name, 'sdk-codegen-all') &&
@@ -50,8 +46,8 @@ jobs:
5046

5147
- name: Pull and run Looker docker image
5248
run: |
53-
docker pull --quiet us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(needs.setup.outputs.matrix_json)[0] }}
54-
docker run --name looker-sdk-codegen-ci -d -p 10000:9999 -p 20000:19999 us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(needs.setup.outputs.matrix_json)[0] }}
49+
docker pull --quiet us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(vars.PRIMARY_LOOKER_VERSION)[0] }}
50+
docker run --name looker-sdk-codegen-ci -d -p 10000:9999 -p 20000:19999 us-west1-docker.pkg.dev/cloud-looker-sdk-codegen-cicd/looker/${{ fromJson(vars.PRIMARY_LOOKER_VERSION)[0] }}
5551
docker logs -f looker-sdk-codegen-ci --until=30s &
5652
5753
- uses: actions/setup-node@v4

‎.github/workflows/python-ci.yml‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,6 @@ permissions:
2727
checks: write
2828

2929
jobs:
30-
setup:
31-
uses: looker-open-source/reusable-actions/.github/workflows/supported-versions.yml@main
32-
3330
typecheck:
3431
name: Mypy
3532
runs-on: ubuntu-latest
@@ -107,7 +104,7 @@ jobs:
107104
path: python/results/
108105

109106
integration:
110-
needs: [unit, setup]
107+
needs: [unit]
111108
name: Integration - ${{ matrix.os }} / Looker.${{ matrix.looker }}
112109
env:
113110
TOX_JUNIT_OUTPUT_NAME: ${{ matrix.os }}.Looker-${{ matrix.looker }}.py3.x
@@ -127,7 +124,7 @@ jobs:
127124
os:
128125
- large-ubuntu
129126
looker:
130-
${{ fromJson(needs.setup.outputs.matrix_json) }}
127+
${{ fromJson(vars.SUPPORTED_LOOKER_VERSIONS) }}
131128

132129
# TODO uncomment `include:` when either macos or windows works to satisfaction.
133130
#include:

‎.github/workflows/tssdk-ci.yml‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -114,10 +114,8 @@ jobs:
114114
name: tssdk-test-results-${{ matrix.os }}-${{ matrix.node-version }}
115115
path: results/tssdk
116116

117-
setup:
118-
uses: looker-open-source/reusable-actions/.github/workflows/primary-version.yml@main
119117
integration:
120-
needs: [unit, setup]
118+
needs: [unit]
121119
name: Integration - ${{ matrix.os }} / Node.${{ matrix.node-version }} / Looker.${{ matrix.looker }}
122120
env:
123121
JEST_JUNIT_OUTPUT_DIR: results/tssdk
@@ -130,7 +128,7 @@ jobs:
130128
node-version: [20.x, 22.x]
131129
os:
132130
- ubuntu
133-
looker: ${{ fromJson(needs.setup.outputs.matrix_json) }}
131+
looker: ${{ fromJson(vars.PRIMARY_LOOKER_VERSION) }}
134132

135133
steps:
136134
- name: Repo Checkout

0 commit comments

Comments
 (0)