ci: use team build pool #2055
ci: use team build pool #2055
Conversation
hack/aks/Makefile
Outdated
| REGION ?= westus2 | ||
| OS_SKU ?= Ubuntu | ||
| VM_SIZE ?= Standard_B2s | ||
| VM_SIZE ?= Standard_DS3_v2 |
There was a problem hiding this comment.
i don't think the default vm or lb skus should be changed, these are way more expensive. if we need bigger nodes for say a load test, then that test should override this via the env var
There was a problem hiding this comment.
tbh I'm not opinionated about the changes in this pr, I'm just reusing the pipeline for quick clusters at the moment :)
| --load-balancer-sku standard \ | ||
| --network-plugin none \ | ||
| --network-plugin-mode overlay \ | ||
| --pod-cidr 192.168.0.0/16 \ |
There was a problem hiding this comment.
a bunch of the overlay and cilium tests implicitly depend on this value being 192.168.0.0/16, why do we need to change it?
There was a problem hiding this comment.
emphasis on the do-not-merge tag, I'm using this target for a convenient one-click byocni cilium overlay cluster for manual testing
I believe I can still use refs/pull/2055/merge as a target if pr is converted to draft
|
This pull request is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days |
|
Pull request closed due to inactivity. |
4 participants
Reason for Change:
Faster than waiting in the massive pool
edit: using this pr for manual testing, do not merge, no intention to merge
Issue Fixed:
Requirements:
Notes: