Skip to content

Update unrestricted & airlock review workspace base version#4087

Merged
tim-p-allen merged 4 commits into
microsoft:mainfrom
nwsde:jr/61-update-workspace-base-version
Sep 19, 2024
Merged

Update unrestricted & airlock review workspace base version#4087
tim-p-allen merged 4 commits into
microsoft:mainfrom
nwsde:jr/61-update-workspace-base-version

Conversation

@jonnyry

@jonnyry jonnyry commented Sep 5, 2024

Copy link
Copy Markdown
Collaborator

This is a follow up to #3970 PR which amended the base workspace for Azure SQL support.

Workspaces derived from base (i.e. unrestricted and airlock-import-review) both pull in the base workspace code from GitHub again using a git ref.

This PR amends the base version to 0.19.0 that the following workspace templates are built off:

  • unrestricted
  • airlock import review

This allows these workspaces to use the Azure SQL component.

@github-actions github-actions Bot added the external PR from an external contributor label Sep 5, 2024
@github-actions

github-actions Bot commented Sep 5, 2024

Copy link
Copy Markdown

Unit Test Results

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

Results for commit 72af49b.

♻️ This comment has been updated with latest results.

@tim-p-allen tim-p-allen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tim-p-allen tim-p-allen enabled auto-merge (squash) September 17, 2024 09:08
@tim-p-allen

Copy link
Copy Markdown

@jonnyry can you update the branch? then we can merge it in

@jonnyry

jonnyry commented Sep 18, 2024

Copy link
Copy Markdown
Collaborator Author

@jonnyry can you update the branch? then we can merge it in

Done

@tim-p-allen

Copy link
Copy Markdown

/test

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

⚠️ 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-p-allen

Copy link
Copy Markdown

/test 72af49b

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

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

@tim-p-allen

Copy link
Copy Markdown

/test-destroy-env

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

@tim-p-allen

Copy link
Copy Markdown

/test 72af49b

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

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

@tim-p-allen

Copy link
Copy Markdown

/test-destroy-env

@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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

@tim-p-allen

Copy link
Copy Markdown

/test

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

⚠️ 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-p-allen

Copy link
Copy Markdown

/test 72af49b

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

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

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

@tim-p-allen tim-p-allen merged commit 7dd1915 into microsoft:main Sep 19, 2024
@jonnyry jonnyry deleted the jr/61-update-workspace-base-version branch September 24, 2024 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external PR from an external contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants