Skip to content

Commit

Permalink
chore: Update tssdk-ci.yml, go-ci.yml, and python-ci.yml to stop usin…
Browse files Browse the repository at this point in the history
…g deprecated runner (#1456)

Update to stop using deprecated runner.


https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/
  • Loading branch information
drstrangelooker committed Apr 17, 2024
1 parent ce27df4 commit c8ccff6
Show file tree
Hide file tree
Showing 11 changed files with 5,437 additions and 814 deletions.
29 changes: 14 additions & 15 deletions .github/workflows/go-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,15 +38,15 @@ jobs:

steps:
- name: Cancel Previous Runs
uses: styfle/cancel-workflow-action@0.11.0
uses: styfle/cancel-workflow-action@0.12.1
with:
access_token: ${{ secrets.GITHUB_TOKEN }}

- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Install Go ${{ matrix.go-version }}
uses: actions/setup-go@v3
uses: actions/setup-go@v5
with:
go-version: ${{ matrix.go-version }}

Expand All @@ -61,9 +61,9 @@ jobs:
- name: Upload Go unit test results
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ env.JUNIT_ARTIFACT_DIR }}
name: ${{ env.JUNIT_ARTIFACT_DIR }}-${{ matrix.os }}-${{ matrix.go-version }}
path: ${{ env.JUNIT_OUTPUT_DIR }}

setup:
Expand Down Expand Up @@ -110,18 +110,18 @@ jobs:
# looker: '7_20'
steps:
- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Authenticate to Google Cloud
uses: google-github-actions/auth@v1.0.0
uses: google-github-actions/auth@v2
with:
credentials_json: ${{ secrets.GCP_AR_READER_SA_KEY }}
project_id: ${{ secrets.GCP_PROJECT_ID }}
create_credentials_file: true
export_environment_variables: true

- name: Set up Cloud SDK
uses: google-github-actions/setup-gcloud@v1.1.0
uses: google-github-actions/setup-gcloud@v2

- name: Authenticate Artifact Repository
run: gcloud auth configure-docker us-west1-docker.pkg.dev --quiet
Expand Down Expand Up @@ -160,7 +160,7 @@ jobs:
docker logs -f looker-sdk-codegen-ci --until=30s &
- name: Install Go ${{ matrix.go-version }}
uses: actions/setup-go@v3
uses: actions/setup-go@v5
with:
go-version: 1.17.6
check-latest: true
Expand All @@ -180,9 +180,9 @@ jobs:
- name: Upload Go unit test results
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: ${{ env.JUNIT_ARTIFACT_DIR }}
name: ${{ env.JUNIT_ARTIFACT_DIR }}-${{ matrix.os }}-${{ matrix.looker }}
path: ${{ env.JUNIT_OUTPUT_DIR }}

publish-test-results:
Expand All @@ -192,18 +192,17 @@ jobs:

steps:
- name: Download Artifacts
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
path: artifacts

- name: Publish All Test Results
uses: EnricoMi/publish-unit-test-result-action@v1.12
uses: EnricoMi/publish-unit-test-result-action@v2
with:
# Cosmetic issue with `check_name` being associated to the wrong
# workflow: https://github.com/EnricoMi/publish-unit-test-result-action/issues/12
check_name: Go Tests
github_token: ${{ secrets.GITHUB_TOKEN }}
report_individual_runs: true
hide_comments: orphaned commits
check_run_annotations_branch: '*'
files: 'artifacts/${{ env.JUNIT_ARTIFACT_DIR }}/*.xml'
files: 'artifacts/${{ env.JUNIT_ARTIFACT_DIR }}*/*.xml'
39 changes: 19 additions & 20 deletions .github/workflows/python-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,11 @@ jobs:

steps:
- name: Cancel Previous Runs
uses: styfle/cancel-workflow-action@0.11.0
uses: styfle/cancel-workflow-action@0.12.1
with:
access_token: ${{ secrets.GITHUB_TOKEN }}
- uses: actions/checkout@v3
- uses: actions/setup-python@v4.5.0
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: '3.10'
- run: pip install -e .
Expand Down Expand Up @@ -75,10 +75,10 @@ jobs:

steps:
- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v4.5.0
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}

Expand All @@ -92,9 +92,9 @@ jobs:

- name: Upload pytest test results
if: ${{ always() }}
uses: actions/upload-artifact@v3.1.2
uses: actions/upload-artifact@v4
with:
name: python-test-results
name: python-test-results-${{ matrix.os }}-${{ matrix.python-version }}
path: python/results/

integration:
Expand Down Expand Up @@ -142,23 +142,23 @@ jobs:
# looker: '7_20'
steps:
- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Set up Python 3.10
uses: actions/setup-python@v4.5.0
uses: actions/setup-python@v5
with:
python-version: '3.10'

- name: Authenticate to Google Cloud
uses: google-github-actions/auth@v1.0.0
uses: google-github-actions/auth@v2
with:
credentials_json: ${{ secrets.GCP_AR_READER_SA_KEY }}
project_id: ${{ secrets.GCP_PROJECT_ID }}
create_credentials_file: true
export_environment_variables: true

- name: Set up Cloud SDK
uses: google-github-actions/setup-gcloud@v1.1.0
uses: google-github-actions/setup-gcloud@v2

- name: Authenticate Artifact Repository
run: gcloud auth configure-docker us-west1-docker.pkg.dev --quiet
Expand Down Expand Up @@ -210,9 +210,9 @@ jobs:

- name: Upload pytest test results
if: ${{ always() }}
uses: actions/upload-artifact@v3.1.2
uses: actions/upload-artifact@v4
with:
name: python-test-results
name: python-test-results-${{ matrix.os }}-${{ matrix.looker }}
path: python/results/

twine:
Expand All @@ -224,8 +224,8 @@ jobs:
working-directory: python/

steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v4.5.0
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: '3.10'
- name: Twine upload check
Expand All @@ -241,7 +241,7 @@ jobs:

steps:
- name: Download Artifacts
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
path: artifacts

Expand All @@ -251,7 +251,7 @@ jobs:
- name: Check Results Files Exist
id: check-results-exist
run: |
if [[ $(ls -l artifacts/python-test-results/*.xml 2> /dev/null | wc -l) -ge "1" ]]; then
if [[ $(ls -l artifacts/python-test-results*/*.xml 2> /dev/null | wc -l) -ge "1" ]]; then
exit 0
else
curl --request POST \
Expand All @@ -272,13 +272,12 @@ jobs:
- name: Publish Unit Test Results
if: steps.check-results-exist.outcome == 'success'
uses: EnricoMi/publish-unit-test-result-action@v1.12
uses: EnricoMi/publish-unit-test-result-action@v2
with:
# Cosmetic issue with `check_name` being associated to the wrong
# workflow: https://github.com/EnricoMi/publish-unit-test-result-action/issues/12
check_name: Python Tests
github_token: ${{ secrets.GITHUB_TOKEN }}
report_individual_runs: true
hide_comments: orphaned commits
check_run_annotations_branch: '*'
files: 'artifacts/python-test-results/*.xml'
files: 'artifacts/python-test-results*/*.xml'
4 changes: 2 additions & 2 deletions .github/workflows/required-checks-hack-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- uses: tony84727/changed-file-filter@v0.2.2
- uses: tony84727/changed-file-filter@v0.2.3
id: filter
with:
filters: |
Expand Down
23 changes: 11 additions & 12 deletions .github/workflows/tssdk-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,15 +47,15 @@ jobs:

steps:
- name: Cancel Previous Runs
uses: styfle/cancel-workflow-action@0.11.0
uses: styfle/cancel-workflow-action@0.12.1
with:
access_token: ${{ secrets.GITHUB_TOKEN }}

- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Install Node ${{ matrix.node-version }}
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}

Expand Down Expand Up @@ -104,9 +104,9 @@ jobs:

- name: Upload ts unit test results
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: tssdk-test-results
name: tssdk-test-results-${{ matrix.os }}-${{ matrix.node-version }}
path: results/tssdk

setup:
Expand All @@ -129,7 +129,7 @@ jobs:

steps:
- name: Repo Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Authenticate to Google Cloud
uses: google-github-actions/auth@v1.0.0
Expand All @@ -155,7 +155,7 @@ jobs:
docker logs -f looker-sdk-codegen-ci --until=30s &
- name: Install Node ${{ matrix.node-version }}
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}

Expand All @@ -173,9 +173,9 @@ jobs:

- name: Upload integration test results
if: ${{ always() }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: tssdk-test-results
name: tssdk-test-results-${{ matrix.os }}-${{ matrix.node-version }}-${{ matrix.looker }}
path: results/tssdk

publish-test-results:
Expand All @@ -185,7 +185,7 @@ jobs:

steps:
- name: Download Artifacts
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
path: artifacts

Expand All @@ -197,6 +197,5 @@ jobs:
check_name: Typescript Tests
github_token: ${{ secrets.GITHUB_TOKEN }}
report_individual_runs: true
hide_comments: orphaned commits
check_run_annotations_branch: '*'
files: 'artifacts/tssdk-test-results/*.xml'
files: 'artifacts/tssdk-test-results*/*.xml'
2 changes: 1 addition & 1 deletion csharp/rtl/Constants.cs
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public struct Constants

public const string DefaultApiVersion = "4.0";
public const string AgentPrefix = "CS-SDK";
public const string LookerVersion = "24.4";
public const string LookerVersion = "24.6";

public const string Bearer = "Bearer";
public const string LookerAppiId = "x-looker-appid";
Expand Down
2 changes: 1 addition & 1 deletion kotlin/src/main/com/looker/sdk/Constants.kt
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ package com.looker.sdk

const val ENVIRONMENT_PREFIX = "LOOKERSDK"
const val SDK_TAG = "KT-SDK"
const val LOOKER_VERSION = "24.4"
const val LOOKER_VERSION = "24.6"
const val API_VERSION = "4.0"
const val AGENT_TAG = "$SDK_TAG $LOOKER_VERSION"
const val LOOKER_APPID = "x-looker-appid"
2 changes: 1 addition & 1 deletion packages/sdk/src/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@
*/

export const sdkVersion = '24.4';
export const sdkVersion = '24.6';
export const environmentPrefix = 'LOOKERSDK';
2 changes: 1 addition & 1 deletion python/looker_sdk/sdk/constants.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,5 +20,5 @@
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
# THE SOFTWARE.

sdk_version = "24.4"
sdk_version = "24.6"
environment_prefix = "LOOKERSDK"
Loading

0 comments on commit c8ccff6

Please sign in to comment.