Skip to content

Exclude recovery service vaults from e2e tests by disabling backup in…#4937

Draft
ChrisChapman-gh wants to merge 3 commits into
mainfrom
4920-recovery-services-vaults-stopping-cicd-tests-from-running
Draft

Exclude recovery service vaults from e2e tests by disabling backup in…#4937
ChrisChapman-gh wants to merge 3 commits into
mainfrom
4920-recovery-services-vaults-stopping-cicd-tests-from-running

Conversation

@ChrisChapman-gh

Copy link
Copy Markdown
Collaborator

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
    }
}

duplicating @maxmartin-cgi work so we can get the pipelines running (#4921)

@ChrisChapman-gh ChrisChapman-gh self-assigned this Jun 19, 2026
@ChrisChapman-gh

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27819357172 (with refid 6dd05385)

(in response to this comment from @ChrisChapman-gh)

@github-actions

github-actions Bot commented Jun 19, 2026

Copy link
Copy Markdown

Unit Test Results

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

Results for commit 0a32b82.

♻️ This comment has been updated with latest results.

@ChrisChapman-gh

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27827269256 (with refid 6dd05385)

(in response to this comment from @ChrisChapman-gh)

…le_backup parameter for backup configuration
@ChrisChapman-gh

Copy link
Copy Markdown
Collaborator Author

/test-extended

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27833533227 (with refid 6dd05385)

(in response to this comment from @ChrisChapman-gh)

2 similar comments
@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27833533227 (with refid 6dd05385)

(in response to this comment from @ChrisChapman-gh)

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27833533227 (with refid 6dd05385)

(in response to this comment from @ChrisChapman-gh)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recovery services vaults stopping CI/CD tests from running

1 participant