From 148d9407705ea1cefb0d1fb906127be33abf793f Mon Sep 17 00:00:00 2001 From: Nick Jones Date: Fri, 19 Jul 2024 17:44:23 +0100 Subject: [PATCH] Nuke _member_ role requirement The _member_ role is a vestige from OpenStack days of yore and no longer exists on recent versions by default. Do as the TODO suggests and remove it. --- pkg/providers/openstack/provider.go | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/pkg/providers/openstack/provider.go b/pkg/providers/openstack/provider.go index f81d799..0d9711c 100644 --- a/pkg/providers/openstack/provider.go +++ b/pkg/providers/openstack/provider.go @@ -313,14 +313,14 @@ func (p *Provider) Images(ctx context.Context) (providers.ImageList, error) { for i := range resources { image := &resources[i] - kuebernetesVersion, _ := image.Properties["k8s"].(string) + kubernetesVersion, _ := image.Properties["k8s"].(string) result = append(result, providers.Image{ ID: image.ID, Name: image.Name, Created: image.CreatedAt, Modified: image.UpdatedAt, - KubernetesVersion: semver(kuebernetesVersion), + KubernetesVersion: semver(kubernetesVersion), }) } @@ -393,10 +393,7 @@ func (p *Provider) getRequiredRoles() []string { return p.region.Spec.Openstack.Identity.ClusterRoles } - // TODO: _member_ shouldn't be necessary, delete me when we get a hsndle on it. - // This is quired by Octavia to list providers and load balancers at the very least. defaultRoles := []string{ - "_member_", "member", "load-balancer_member", }