Cicd test2#4933
Conversation
|
/test |
|
🤖 pr-bot 🤖 🏃 Running tests: https://github.com/microsoft/AzureTRE/actions/runs/27765483193 (with refid (in response to this comment from @ChrisChapman-gh) |
There was a problem hiding this comment.
Pull request overview
This PR appears to introduce intentional “pipeline test” changes across documentation and core Terraform defaults; as written, the changes would be disruptive if merged into Azure TRE’s mainline.
Changes:
- Replaces the README title with placeholder text.
- Changes the default resource processor VMSS SKU.
- Adds an unused Terraform local value.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| README.md | Replaces the top-of-file heading with placeholder content. |
| core/terraform/variables.tf | Updates the default resource_processor_vmss_sku value. |
| core/terraform/resource_processor/vmss_porter/locals.tf | Adds an unused local variable. |
| DC | ||
| :x | ||
|
|
||
|
|
||
|
|
| variable "resource_processor_vmss_sku" { | ||
| type = string | ||
| default = "Standard_B2s" | ||
| default = "Standard_D2as_v5" | ||
| description = "The SKU of the resource processor VMSS." |
| locals { | ||
| cdc = "dud" | ||
| version = replace(replace(replace(data.local_file.version.content, "__version__ = \"", ""), "\"", ""), "\n", "") |
Unit Test Results0 tests 0 ✅ 0s ⏱️ Results for commit d9f0bd0. |
|
/test-extended |
|
🤖 pr-bot 🤖 🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/27770376589 (with refid (in response to this comment from @ChrisChapman-gh) |
|
/test-destroy-env |
|
Destroying PR test environment (RG: rg-trede148849)... (run: https://github.com/microsoft/AzureTRE/actions/runs/27812855076) |
1 similar comment
|
Destroying PR test environment (RG: rg-trede148849)... (run: https://github.com/microsoft/AzureTRE/actions/runs/27812855076) |
|
PR test environment destroy complete (RG: rg-trede148849) |
|
tests look ok |
This is a PR purely to test the pipeline. There is no intention to actually merge it.