Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: update providerid prefix for aws
This PR updates the ProviderID format for aws resources. There seems to be a bug when using Talos CCM (which consumes this value from Talos) because the format is `aws://x/y` (two slashes) vs. the expected `aws:///x/y` (three slashes) that is set with the AWS CCM code [here](https://github.com/kubernetes/cloud-provider-aws/blob/d0551093673e8c355db17249b8f069767c014748/pkg/providers/v1/instances.go#L47-L53). Setting only two slashes causes important software in the workload cluster to fail, specifically cluster-autoscaler. The regex they use for pulling providerID is [here](https://github.com/kubernetes/autoscaler/blob/702e9685d6c1d002f4a448f2d88007141dfa6d56/cluster-autoscaler/cloudprovider/aws/aws_cloud_provider.go#L195). Signed-off-by: Spencer Smith <spencer.smith@talos-systems.com>
- Loading branch information