Updated deployment setting to use http#13968
Conversation
chidozieononiwu
commented
Feb 12, 2026
chidozieononiwu
commented
- Update pipeline to add http setting for deployments.
- Needed for Enable live test for remote MCP microsoft/mcp#1630
There was a problem hiding this comment.
Pull request overview
Adds a pipeline/template knob to opt into HTTP transport for test resource deployments, intended to support downstream pipeline needs (linked to microsoft/mcp#1630).
Changes:
- Add
UseHttpTransportparameter to thedeploy-test-resources.ymltemplate and pass it through toNew-TestResources.ps1. - Add
UseHttpTransportswitch parameter toNew-TestResources.ps1so it can be forwarded via@PSBoundParametersto pre/post deployment scripts.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| eng/common/TestResources/deploy-test-resources.yml | Introduces UseHttpTransport template parameter and passes it to the deployment script in both auth flows. |
| eng/common/TestResources/New-TestResources.ps1 | Adds UseHttpTransport switch to the script interface for propagation to downstream scripts. |
|
The following pipelines have been queued for testing: |
chidozieononiwu
force-pushed
the
EnableLiveTestOverHttp
branch
from
854c41e to
5435210
Compare
|
The following pipelines have been queued for testing: |
There was a problem hiding this comment.
I think this should be ok. But it is definitely a functional change. I will double check to see if we use this and if we want to change existing that reference ArmTemplateParameters TO AdditionalParameters.
EDIT I have no idea why it was done this way, but ArmTemplatePameters are absolutely being used in a bunch of places via injection. So this functional change is 1000000% going to break pretty much all our livetests. We cannot merge this as-is without functional changes in each repo :|
Due to the merging within new-testresources.ps1 itself, I think it might be ok. We should smoke-test a check that uses the ArmTemplateParameters variable in their platform matrix.
|
The following pipelines have been queued for testing: |
chidozieononiwu
force-pushed
the
EnableLiveTestOverHttp
branch
from
c18f9fa to
ed11f9d
Compare
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#13968 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#13968 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
3 participants
