Skip to content

Commit

Permalink
GitHub actions update (#22)
Browse files Browse the repository at this point in the history
* azure login

* reset cron

* update action versions

---------

Co-authored-by: Wojciech <57405495+thewbuk@users.noreply.github.com>
Co-authored-by: wojciechcloudkubed <159798789+wojciechcloudkubed@users.noreply.github.com>
  • Loading branch information
3 people authored Apr 5, 2024
1 parent da1f102 commit 0e7c6f9
Show file tree
Hide file tree
Showing 9 changed files with 49 additions and 49 deletions.
32 changes: 16 additions & 16 deletions .github/workflows/build_docker_images.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,18 +21,18 @@ jobs:
steps:
- name: Upload Event File
# this step is required to publish test results from forks
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: Event File
path: ${{ github.event_path }}

- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false

- name: Filter changes
uses: dorny/paths-filter@v2
uses: dorny/paths-filter@v3
id: filter
with:
filters: |
Expand Down Expand Up @@ -93,7 +93,7 @@ jobs:

- name: Set up Docker Buildx
id: buildx
uses: docker/setup-buildx-action@v2
uses: docker/setup-buildx-action@v3

# Unit Tests are executed by calling the 'test-results' target in the
# Dockerfile's. Test runner exit codes must be swallowed (and kept) so we
Expand All @@ -105,7 +105,7 @@ jobs:
if: |
(steps.filter.outputs.api == 'true'
|| github.event_name == 'workflow_dispatch')
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./api_app/
file: ./api_app/Dockerfile
Expand All @@ -116,7 +116,7 @@ jobs:

- name: "Check pytest failure file existence"
id: check_api_test_result
uses: andstor/file-existence-action@v2
uses: andstor/file-existence-action@v3
with:
files: "test-results/pytest_api_unit_failed"

Expand All @@ -125,7 +125,7 @@ jobs:
(steps.filter.outputs.api == 'true'
|| github.event_name == 'workflow_dispatch')
&& steps.check_api_test_result.outputs.files_exists == 'false'
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./api_app/
file: ./api_app/Dockerfile
Expand All @@ -136,7 +136,7 @@ jobs:
if: |
(steps.filter.outputs.resource_processor == 'true'
|| github.event_name == 'workflow_dispatch')
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./resource_processor
file: ./resource_processor/vmss_porter/Dockerfile
Expand All @@ -147,7 +147,7 @@ jobs:
if: |
(steps.filter.outputs.guacamole_server == 'true'
|| github.event_name == 'workflow_dispatch')
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./templates/workspace_services/guacamole/guacamole-server
file: ./templates/workspace_services/guacamole/guacamole-server/docker/Dockerfile
Expand All @@ -158,7 +158,7 @@ jobs:

- name: "Check maven failure file existence"
id: check_maven_test_result
uses: andstor/file-existence-action@v2
uses: andstor/file-existence-action@v3
with:
files: "test-results/guacamole_package_failed"

Expand All @@ -167,7 +167,7 @@ jobs:
(steps.filter.outputs.guacamole_server == 'true'
|| github.event_name == 'workflow_dispatch')
&& steps.check_maven_test_result.outputs.files_exists == 'false'
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./templates/workspace_services/guacamole/guacamole-server
file: ./templates/workspace_services/guacamole/guacamole-server/docker/Dockerfile
Expand All @@ -178,7 +178,7 @@ jobs:
if: |
(steps.filter.outputs.gitea == 'true'
|| github.event_name == 'workflow_dispatch')
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./templates/shared_services/gitea/docker
file: ./templates/shared_services/gitea/docker/Dockerfile
Expand All @@ -195,7 +195,7 @@ jobs:
if: |
(steps.filter.outputs.airlock_processor == 'true'
|| github.event_name == 'workflow_dispatch')
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./airlock_processor/
file: ./airlock_processor/Dockerfile
Expand All @@ -206,7 +206,7 @@ jobs:

- name: "Check pytest failure file existence"
id: check_airlock_processor_test_result
uses: andstor/file-existence-action@v2
uses: andstor/file-existence-action@v3
with:
files: "test-results/pytest_airlock_processor_unit_failed"

Expand All @@ -215,7 +215,7 @@ jobs:
(steps.filter.outputs.airlock_processor == 'true'
|| github.event_name == 'workflow_dispatch')
&& steps.check_airlock_processor_test_result.outputs.files_exists == 'false'
uses: docker/build-push-action@v4
uses: docker/build-push-action@v5
with:
context: ./airlock_processor/
file: ./airlock_processor/Dockerfile
Expand All @@ -224,7 +224,7 @@ jobs:

- name: Upload Unit Test Results
if: always()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test-results
path: test-results
4 changes: 2 additions & 2 deletions .github/workflows/build_docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,11 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout main
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
fetch-depth: 0
persist-credentials: true
- uses: actions/setup-python@v4
- uses: actions/setup-python@v5
with:
python-version: 3.x
- name: Install Dependencies
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/build_validation_develop.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,14 +19,14 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout code
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
# Full git history is needed to get a proper list of
# changed files within `super-linter`
fetch-depth: 0
persist-credentials: false

- uses: dorny/paths-filter@v2
- uses: dorny/paths-filter@v3
id: filter
with:
filters: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/clean_validation_envs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ name: Clean Validation Environments
on: # yamllint disable-line rule:truthy
schedule:
# Every 2 hours
- cron: "0 19 * * *"
- cron: "0 */2 * * *"
workflow_dispatch:

jobs:
Expand All @@ -14,7 +14,7 @@ jobs:
environment: CICD
timeout-minutes: 30
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
# This is CRITICAL since we're making decisions based on branch existence
fetch-depth: 0
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/cli-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:

steps:
- name: Checkout (GitHub)
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Build and run dev container task
uses: ./.github/actions/devcontainer_run_command
Expand Down Expand Up @@ -50,7 +50,7 @@ jobs:
AZURE_ENVIRONMENT: ${{ secrets.AZURE_ENVIRONMENT }}

- name: Upload Wheel as artifact
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: tre-cli
path: dist/tre-*.whl
34 changes: 17 additions & 17 deletions .github/workflows/deploy_tre_reusable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -185,15 +185,15 @@ jobs:
details_url: "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"

- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
# then the default checkout will apply
ref: ${{ inputs.prRef }}

- name: Set up Docker BuildKit
uses: docker/setup-buildx-action@v2
uses: docker/setup-buildx-action@v3

- name: Azure Login
uses: azure/login@v2
Expand Down Expand Up @@ -274,7 +274,7 @@ jobs:

steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand All @@ -298,7 +298,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand All @@ -322,7 +322,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -408,7 +408,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -454,7 +454,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -500,7 +500,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -560,7 +560,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -609,7 +609,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -643,7 +643,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -692,7 +692,7 @@ jobs:
environment: ${{ inputs.environmentName }}
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -723,7 +723,7 @@ jobs:
timeout-minutes: 10
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -752,7 +752,7 @@ jobs:

- name: Upload Test Results
if: always()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: E2E Test (Smoke) Results
path: "./e2e_tests/pytest_e2e_smoke.xml"
Expand All @@ -766,7 +766,7 @@ jobs:
timeout-minutes: 300
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false
# if the following values are missing (i.e. not triggered via comment workflow)
Expand Down Expand Up @@ -796,7 +796,7 @@ jobs:

- name: Upload Test Results
if: always()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: E2E Test Results
path: "./e2e_tests/pytest_e2e_custom.xml"
Expand Down Expand Up @@ -845,7 +845,7 @@ jobs:
timezone: Europe/Zurich

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

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/flag_external_pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,13 @@ jobs:
steps:
# Ensure we have the script file for the github-script action to use
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false

- id: check_command
name: Check for a command using GitHub script
uses: actions/github-script@v6
uses: actions/github-script@v7
with:
result-encoding: string
script: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/lets_encrypt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,12 @@ jobs:
environment: CICD
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
persist-credentials: false

- name: Install Terraform
uses: hashicorp/setup-terraform@v2
uses: hashicorp/setup-terraform@v3
with:
terraform_version: 1.2.9
terraform_wrapper: false
Expand Down
Loading

0 comments on commit 0e7c6f9

Please sign in to comment.