Skip to content

Commit

Permalink
Merge pull request #489 from stuggi/fix_novanovnc_endpoint
Browse files Browse the repository at this point in the history
Fix nova-novncproxy route endpoint type
  • Loading branch information
openshift-merge-robot authored Sep 25, 2023
2 parents 168cbd1 + 0c8330e commit cb0efeb
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/openstack/nova.go
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ func ReconcileNova(ctx context.Context, instance *corev1beta1.OpenStackControlPl
nova,
svcs,
map[service.Endpoint]service.RoutedOverrideSpec{
service.EndpointInternal: *cellTemplate.NoVNCProxyServiceTemplate.Override.Service,
service.EndpointPublic: *cellTemplate.NoVNCProxyServiceTemplate.Override.Service,
},
instance.Spec.Nova.CellOverride[cellName].NoVNCProxy.Route,
corev1beta1.OpenStackControlPlaneExposeNovaReadyCondition,
Expand All @@ -165,7 +165,7 @@ func ReconcileNova(ctx context.Context, instance *corev1beta1.OpenStackControlPl
return ctrlResult, nil
}

cellTemplate.NoVNCProxyServiceTemplate.Override.Service = ptr.To(routedOverrideSpec[service.EndpointInternal])
cellTemplate.NoVNCProxyServiceTemplate.Override.Service = ptr.To(routedOverrideSpec[service.EndpointPublic])

instance.Spec.Nova.Template.CellTemplates[cellName] = cellTemplate

Expand Down

0 comments on commit cb0efeb

Please sign in to comment.