Skip to content

Commit

Permalink
Nuke _member_ role requirement
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
yankcrime committed Jul 19, 2024
1 parent 588df83 commit 148d940
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions pkg/providers/openstack/provider.go
Original file line number Diff line number Diff line change
Expand Up @@ -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),
})
}

Expand Down Expand Up @@ -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",
}
Expand Down

0 comments on commit 148d940

Please sign in to comment.