Dedicated storage agent pool #19191
Dedicated storage agent pool #19191
Conversation
mitchdenny
requested review from
chidozieononiwu,
danieljurek and
weshaggard
as code owners
|
/azp run net - storage - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run net - storage - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run net - storage - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
| @@ -5,23 +5,23 @@ | |||
| }, | |||
| "matrix": { | |||
| "Agent": { | |||
| "ubuntu-18.04": { "OSVmImage": "MMSUbuntu18.04", "Pool": "azsdk-pool-mms-ubuntu-1804-general" }, | |||
| "windows-2019": { "OSVmImage": "MMS2019", "Pool": "azsdk-pool-mms-win-2019-general" }, | |||
| "ubuntu-18.04": { "OSVmImage": "MMSUbuntu18.04", "Pool": "azsdk-pool-mms-ubuntu-1804-storage" }, | |||
There was a problem hiding this comment.
cc @benbp
@mitchdenny what strategy are you thinking about for overriding the pools we use here? Should we enable some way to override these values in the template instead of requiring that folks duplicate the entire matrix.json file?
There was a problem hiding this comment.
@weshaggard when I was trying to show the pool override alternative to @mitchdenny , I found a small bug that made it not work :)
With the current import + override behavior (predicated on the above bug fix), this is what it could look like:
{
"matrix": {
"$IMPORT": "./eng/pipelines/templates/stages/platform-matrix.json",
"PoolOverride": {
"StorageLinuxPool": { "Pool": "azsdk-pool-mms-ubuntu-1804-storage" },
"StorageWinPool": { "Pool": "azsdk-pool-mms-win-2019-storage" },
"HostedMacPool": { "Pool": "Azure Pipelines" }
}
},
"exclude": [
{
"Pool": "azsdk-pool-mms-ubuntu-1804-storage",
"OSVmImage": [ "MMS2019", "MacOS-10.15" ]
},
{
"Pool": "azsdk-pool-mms-win-2019-storage",
"OSVmImage": [ "MMSUbuntu18.04", "MacOS-10.15" ]
},
{
"Pool": "Azure Pipelines",
"OSVmImage": [ "MMS2019", "MMSUbuntu18.04" ]
}
]
}
It might be worth thinking about a replace type operator in the config for this kind of scenario.
There was a problem hiding this comment.
Fix for the bug I mentioned: Azure/azure-sdk-tools#1461
There was a problem hiding this comment.
Yeah it might be interesting to have some sort of replace operator that could update the entire agent based on the name of the agent.
|
/azp run net - storage - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
This PR cuts over the storage pipelines to use a dedicated storage pool that has been provisioned in Central Canada which has more memory for their specialized test scenarios.