microsoft / AzureTRE

An accelerator to help organizations build Trusted Research Environments on Azure.
https://microsoft.github.io/AzureTRE
MIT License
169 stars 133 forks source link

Resource processor - Re-instate az login and az acr login commands prior to porter explain command to fix microsoft/AzureTRE#3950 #3951

Closed jonnyry closed 1 month ago

jonnyry commented 1 month ago

Resolves #3950

What is being addressed

Re-instate az login and az acr login commands prior to porter explain command to fix microsoft/AzureTRE#3950

github-actions[bot] commented 1 month ago

Unit Test Results

0 tests   0 :white_check_mark:  0s :stopwatch: 0 suites  0 :zzz: 0 files    0 :x:

Results for commit d518d526.

:recycle: This comment has been updated with latest results.

tim-allen-ck commented 1 month ago

/test-extended bdd9b5e

github-actions[bot] commented 1 month ago

:robot: pr-bot :robot:

:runner: Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/9223682237 (with refid 39c3e950)

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

github-actions[bot] commented 1 month ago

:robot: pr-bot :robot:

:runner: Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/9223682237 (with refid 39c3e950)

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

github-actions[bot] commented 1 month ago

:robot: pr-bot :robot:

:runner: Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/9223682237 (with refid 39c3e950)

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

jonnyry commented 1 month ago

Hi @tim-allen-ck I've had a look at the test failures and I don't think its related to my change, but let me know if you think otherwise and I'll take another look.

tim-allen-ck commented 1 month ago

Hi @tim-allen-ck I've had a look at the test failures and I don't think its related to my change, but let me know if you think otherwise and I'll take another look.

Yeah nothing to do with your code changes, looks like an environment issue, I'm looking into it.

tim-allen-ck commented 1 month ago

/test-destroy-env

github-actions[bot] commented 1 month ago

Destroying PR test environment (RG: rg-tre39c3e950)... (run: https://github.com/microsoft/AzureTRE/actions/runs/9283038615)

github-actions[bot] commented 1 month ago

PR test environment destroy complete (RG: rg-tre39c3e950)

tim-allen-ck commented 1 month ago

/test

github-actions[bot] commented 1 month ago

:robot: pr-bot :robot:

:warning: When using /test on external PRs, the SHA of the checked commit must be specified

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

tim-allen-ck commented 1 month ago

/test bdd9b5e

github-actions[bot] commented 1 month ago

:robot: pr-bot :robot:

:runner: Running tests: https://github.com/microsoft/AzureTRE/actions/runs/9284485797 (with refid 39c3e950)

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

tim-allen-ck commented 1 month ago

@jonnyry can you update the commit with the latest main branch?

jonnyry commented 1 month ago

@tim-allen-ck updated to latest

tim-allen-ck commented 1 month ago

/test d518d52

github-actions[bot] commented 1 month ago

:robot: pr-bot :robot:

:runner: Running tests: https://github.com/microsoft/AzureTRE/actions/runs/9299672931 (with refid 39c3e950)

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