Michel Loiseleur

Results 607 comments of Michel Loiseleur

@leonardocaylent Do you think you can add a test on this ? In order to keep issues fixed for good, we accept PR only when there is a test with...

Go modules update are done in dedicated PRs. @leonardocaylent Would you please remove update of go.mod & go.sum in this PR ?

I suggest to remove all the changes in `aws.go`. They don't seem to be necessary. Otherwise, it looks very good to me. Thanks a lot !

@leonardocaylent Creating a new map of `Endpoint` may work and comes also with some impact on users with big clusters. I've mixed feelings about this, because it's a fix for...

Just a warning for future users reading this: * Be sure to read **carefully** the (WIP) Traefik V3 migration guide: https://doc.traefik.io/traefik/master/migration/v2-to-v3/

> I think it's a valid option to always set "Cloud API Owned" to true in case the TenantID is present. Downsides would be that we don't know how infoblox...

Hello @Links2004 , That's right. This chart supports stable version of Traefik. Considering the changes on tracing, I'm unsure if and how it can be implemented in this Chart to...

Ah yes, that's right. Thanks @Links2004

We are also seeing this error in (CAPI) metrics : `controller_runtime_reconcile_errors_total{controller="machine", namespace="capi-system}"` It's still present in v2.0.2 of CAPA & v1.3.0 of CAPI

Here are differences, we're also using AWSManagedControlPlane (EKS), arm arch & gp3 rootVolume: ```diff --- e2e.yaml 2023-01-06 09:11:39.092292421 +0100 +++ traefik.yaml 2023-01-06 09:13:12.056007510 +0100 @@ -8,7 +8,14 @@ template: spec:...