Add parameter to payload in e2etests/conftest to disable service vaul…#4921
Add parameter to payload in e2etests/conftest to disable service vaul…#4921maxmartin-cgi wants to merge 6 commits into
Conversation
…t backup in test environments
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates E2E test workspace provisioning to explicitly disable backups when creating (or retrieving) test workspaces.
Changes:
- Adds
enable_backup: Falseto the test workspace creation payload. - Fixes the JSON/dict formatting by adding a trailing comma after
address_space_size.
Unit Test Results673 tests 673 ✅ 7s ⏱️ Results for commit 7e6dcf1. ♻️ This comment has been updated with latest results. |
|
/test-extended d45b558 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/26811484684 (with refid (in response to this comment from @rudolphjacksonm) |
|
/test-extended 904c614 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/26878311975 (with refid (in response to this comment from @rudolphjacksonm) |
|
@microsoft-github-policy-service agree company="CGI" |
|
/test-extended fc654b6 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/26887373327 (with refid (in response to this comment from @rudolphjacksonm) |
|
/test-destroy-env |
|
Destroying PR test environment (RG: rg-treac107ffc)... (run: https://github.com/microsoft/AzureTRE/actions/runs/26956600493) |
|
PR test environment destroy complete (RG: rg-treac107ffc) |
|
/test-extended 1ea17b4 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/26959469098 (with refid (in response to this comment from @rudolphjacksonm) |
|
@copilot bump the api version so container image can be built |
|
/test-extended 7e6dcf1 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27004507579 (with refid (in response to this comment from @rudolphjacksonm) |
|
/test-destroy-env |
|
Destroying PR test environment (RG: rg-treac107ffc)... (run: https://github.com/microsoft/AzureTRE/actions/runs/27005457222) |
|
PR test environment destroy complete (RG: rg-treac107ffc) |
|
/test-extended 1ea17b4 |
|
🤖 pr-bot 🤖
(in response to this comment from @rudolphjacksonm) |
|
/test-extended 7e6dcf1 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27005962244 (with refid (in response to this comment from @rudolphjacksonm) |
|
/test-extended 7e6dcf1 |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27007145453 (with refid (in response to this comment from @rudolphjacksonm) |
1 similar comment
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27007145453 (with refid (in response to this comment from @rudolphjacksonm) |
…t backup in test environments
Resolves #4920
What is being addressed
E2E tests failing to run due to bug in recovery service vaults.
How is this addressed
Added parameter to "properties" payload in e2etests/conftest.py to disable backup (currently enabled by default)
payload = { "templateName": template_name, "properties": { "display_name": f"E2E {description} workspace ({auth_type} AAD)", "description": f"{template_name} test workspace for E2E tests", "auth_type": auth_type, "address_space_size": "small", "enable_backup": False } }