Azure / azure-capi-cli-extension

Kubernetes Cluster API support in the Azure CLI
MIT License
20 stars 18 forks source link

Remove Felix override workaround #180

Closed CecileRobertMichon closed 1 year ago

CecileRobertMichon commented 1 year ago

Description

Reported by @huntergregory

It seems az capi create is currently failing with the newest capi extension version. It seems this commit removed a file resulting in a 404 error. Remove Calico felix override workaround · kubernetes-sigs/cluster-api-provider-azure@34c05f8 (github.com) cli: ✓ Deployed CNI to workload cluster cli: Applying felix-override manifest. error: unable to read URL "https://raw.githubusercontent.com/kubernetes-sigs/cluster-api-provider-azure/main/templates/addons/calico/felix-override.yaml["](https://raw.githubusercontent.com/kubernetes-sigs/cluster-api-provider-azure/main/templates/addons/calico/felix-override.yaml%22), server reported 404 Not Found, status code=404 cli: Command '['kubectl', 'apply', '-f', 'https://raw.githubusercontent.com/kubernetes-sigs/cluster-api-provider-azure/main/templates/addons/calico/felix-override.yaml['](https://raw.githubusercontent.com/kubernetes-sigs/cluster-api-provider-azure/main/templates/addons/calico/felix-override.yaml%27), '--kubeconfig', 'hgregory-capz-conf-fwdcs7.kubeconfig']' returned non-zero exit status 1.

This PR removes the felix override workaround as it is no longer needed.

History Notes


This checklist is used to make sure that common guidelines for an Azure CLI pull request are followed.