Feature/GitHub actions update

[tim-allen-ck]

Resolves #3847

What is being addressed

Some of the actions might stop working soon as they are based on Node16 which is being removed by Github.

How is this addressed

• All actions are updated to their latest version

• Describe the changes made, and if appropriate, why they are addressed this way

• Note any pending work (with links to the issues that will address them)

• Update documentation

• Update CHANGELOG.md if needed

• Increment template version if needed, for guidelines see Authoring templates - versioning

[github-actions]

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit ffa1b3f.

♻️ This comment has been updated with latest results.

[tim-allen-ck]

/test

[github-actions]

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/8567104061 (with refid fe1abbfe)

(in response to this comment from @tim-allen-ck)

[marrobi]

Are there any other actions that need doing?

Issue says: All actions are updated to their latest version

The Azure one was a note as to not forget it, apologies if confusing.

[marrobi]

@tim-allen-ck suggest check the warnings at bottom of https://github.com/microsoft/AzureTRE/actions/runs/8540234100

[tim-allen-ck]

@marrobi ahh okay, no problem, think I was confused. I'll take another look.

[tim-allen-ck]

@marrobi just fyi technote-space/workflow-conclusion-action@v3 has been archived as of Nov 2023 and will not transition to node 20

[marrobi]

@marrobi just fyi technote-space/workflow-conclusion-action@v3 has been archived as of Nov 2023 and will not transition to node 20

I'm not even sure its doing anything?

AzureTRE/.github/workflows/deploy_tre_reusable.yml

Lines 812 to 830 in 086be33

	environment: ${{ inputs.environmentName }}
	steps:
	- uses: technote-space/workflow-conclusion-action@v3
	# For PR builds triggered from comment builds, the GITHUB_REF is set to main
	# so the checks aren't automatically associated with the PR
	# If prHeadSha is specified then explicity mark the checks for that SHA
	- name: Report check status
	if: inputs.prHeadSha != ''
	uses: LouisBrunner/checks-action@v2.0.0
	with:
	token: ${{ secrets.GITHUB_TOKEN }}
	# the name must be identical to the one received by the real job
	sha: ${{ inputs.prHeadSha }}
	name: "Deploy PR / Run E2E Tests (Smoke)"
	status: "completed"
	conclusion: ${{ env.WORKFLOW_CONCLUSION }}
	details_url: "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"

As its the same level as the step below.

[tim-allen-ck]

@marrobi just fyi technote-space/workflow-conclusion-action@v3 has been archived as of Nov 2023 and will not transition to node 20

I'm not even sure its doing anything?

AzureTRE/.github/workflows/deploy_tre_reusable.yml

Lines 812 to 830 in 086be33

	environment: ${{ inputs.environmentName }}
	steps:
	- uses: technote-space/workflow-conclusion-action@v3
	# For PR builds triggered from comment builds, the GITHUB_REF is set to main
	# so the checks aren't automatically associated with the PR
	# If prHeadSha is specified then explicity mark the checks for that SHA
	- name: Report check status
	if: inputs.prHeadSha != ''
	uses: LouisBrunner/checks-action@v2.0.0
	with:
	token: ${{ secrets.GITHUB_TOKEN }}
	# the name must be identical to the one received by the real job
	sha: ${{ inputs.prHeadSha }}
	name: "Deploy PR / Run E2E Tests (Smoke)"
	status: "completed"
	conclusion: ${{ env.WORKFLOW_CONCLUSION }}
	details_url: "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"

As its the same level as the step below.

Looks like its used to notify the teams channel of a failed run

[marrobi]

@marrobi just fyi technote-space/workflow-conclusion-action@v3 has been archived as of Nov 2023 and will not transition to node 20

I'm not even sure its doing anything?

AzureTRE/.github/workflows/deploy_tre_reusable.yml

Lines 812 to 830 in 086be33

	environment: ${{ inputs.environmentName }}
	steps:
	- uses: technote-space/workflow-conclusion-action@v3
	# For PR builds triggered from comment builds, the GITHUB_REF is set to main
	# so the checks aren't automatically associated with the PR
	# If prHeadSha is specified then explicity mark the checks for that SHA
	- name: Report check status
	if: inputs.prHeadSha != ''
	uses: LouisBrunner/checks-action@v2.0.0
	with:
	token: ${{ secrets.GITHUB_TOKEN }}
	# the name must be identical to the one received by the real job
	sha: ${{ inputs.prHeadSha }}
	name: "Deploy PR / Run E2E Tests (Smoke)"
	status: "completed"
	conclusion: ${{ env.WORKFLOW_CONCLUSION }}
	details_url: "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"

As its the same level as the step below.

Looks like its used to notify the teams channel of a failed run

That uses sachinkundu/ms-teams-notification@1.4

Not sure we use it either, and @sachinkundu has left MS, so might not be best to rely on that either. Can we delete the Teams notification piece for now.

[tim-allen-ck]

@marrobi just fyi technote-space/workflow-conclusion-action@v3 has been archived as of Nov 2023 and will not transition to node 20

I'm not even sure its doing anything?

AzureTRE/.github/workflows/deploy_tre_reusable.yml

Lines 812 to 830 in 086be33

	environment: ${{ inputs.environmentName }}
	steps:
	- uses: technote-space/workflow-conclusion-action@v3
	# For PR builds triggered from comment builds, the GITHUB_REF is set to main
	# so the checks aren't automatically associated with the PR
	# If prHeadSha is specified then explicity mark the checks for that SHA
	- name: Report check status
	if: inputs.prHeadSha != ''
	uses: LouisBrunner/checks-action@v2.0.0
	with:
	token: ${{ secrets.GITHUB_TOKEN }}
	# the name must be identical to the one received by the real job
	sha: ${{ inputs.prHeadSha }}
	name: "Deploy PR / Run E2E Tests (Smoke)"
	status: "completed"
	conclusion: ${{ env.WORKFLOW_CONCLUSION }}
	details_url: "https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}"

As its the same level as the step below.

Looks like its used to notify the teams channel of a failed run

That uses sachinkundu/ms-teams-notification@1.4

Not sure we use it either, and @sachinkundu has left MS, so might not be best to rely on that either. Can we delete the Teams notification piece for now.

Yep let remove, I'll add an issue to replace it with something else

[tim-allen-ck]

/test-extended

[github-actions]

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/8572145997 (with refid fe1abbfe)

(in response to this comment from @tim-allen-ck)

[tim-allen-ck]

/test

[github-actions]

🤖 pr-bot 🤖

🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/8596920000 (with refid fe1abbfe)

(in response to this comment from @tim-allen-ck)