Jack Francis

Results 586 comments of Jack Francis

> Not sure if this is relevant, but should the commit message also be changed to be more relevant to what the PR is actually doing now? I'm not sure...

/retitle AKS: enable isVnetManaged, add caching

> > Additionally, in order to account for additional calls against the Azure APIs I've wired in a rate limiting + HTTP 429 retry solution (inspired by cloud-provider-azure). > >...

@mtougeron @CecileRobertMichon this is ready for another (hopefully final? 🤞) review round

I agree. 1. We should probably as a rule "require" test changes/fixes/evolutions to be cherry-picked into the last release branch, so that we minimize friction for any critical test fix...

@jsturtevant I'm not sure I understand the need for this, but I think I have an idea how we might do this. Do we still want to make this change...