Skip to content

Commit 99d1f7d

Browse files
committed
Set Ironic route annotations
1 parent d50d76d commit 99d1f7d

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

apis/core/v1beta1/openstackcontrolplane_webhook.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,14 +49,14 @@ import (
4949
networkv1 "github.com/openstack-k8s-operators/infra-operator/apis/network/v1beta1"
5050
rabbitmqv1 "github.com/openstack-k8s-operators/infra-operator/apis/rabbitmq/v1beta1"
5151
redisv1 "github.com/openstack-k8s-operators/infra-operator/apis/redis/v1beta1"
52+
topologyv1 "github.com/openstack-k8s-operators/infra-operator/apis/topology/v1beta1"
5253
ironicv1 "github.com/openstack-k8s-operators/ironic-operator/api/v1beta1"
5354
manilav1 "github.com/openstack-k8s-operators/manila-operator/api/v1beta1"
5455
neutronv1 "github.com/openstack-k8s-operators/neutron-operator/api/v1beta1"
5556
novav1 "github.com/openstack-k8s-operators/nova-operator/api/v1beta1"
5657
octaviav1 "github.com/openstack-k8s-operators/octavia-operator/api/v1beta1"
5758
swiftv1 "github.com/openstack-k8s-operators/swift-operator/api/v1beta1"
5859
telemetryv1 "github.com/openstack-k8s-operators/telemetry-operator/api/v1beta1"
59-
topologyv1 "github.com/openstack-k8s-operators/infra-operator/apis/topology/v1beta1"
6060
)
6161

6262
var ctlplaneWebhookClient client.Client
@@ -788,6 +788,11 @@ func (r *OpenStackControlPlane) DefaultServices() {
788788
r.Spec.Ironic.Template.StorageClass = r.Spec.StorageClass
789789
}
790790
r.Spec.Ironic.Template.Default()
791+
792+
initializeOverrideSpec(&r.Spec.Ironic.APIOverride.Route, true)
793+
initializeOverrideSpec(&r.Spec.Ironic.InspectorOverride.Route, true)
794+
r.Spec.Ironic.Template.SetDefaultRouteAnnotations(r.Spec.Ironic.APIOverride.Route.Annotations)
795+
r.Spec.Ironic.Template.SetDefaultInspectorRouteAnnotations(r.Spec.Ironic.InspectorOverride.Route.Annotations)
791796
}
792797

793798
// Keystone

tests/functional/ctlplane/openstackoperator_controller_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -597,6 +597,10 @@ var _ = Describe("OpenStackOperator controller", func() {
597597
Expect(OSCtlplane.Spec.Keystone.APIOverride.Route).Should(Not(BeNil()))
598598
Expect(OSCtlplane.Spec.Keystone.APIOverride.Route.Annotations).Should(HaveKeyWithValue("haproxy.router.openshift.io/timeout", "60s"))
599599
Expect(OSCtlplane.Spec.Keystone.APIOverride.Route.Annotations).Should(HaveKeyWithValue("api.keystone.openstack.org/timeout", "60s"))
600+
Expect(OSCtlplane.Spec.Ironic.APIOverride.Route).Should(Not(BeNil()))
601+
Expect(OSCtlplane.Spec.Ironic.APIOverride.Route.Annotations).Should(HaveKeyWithValue("haproxy.router.openshift.io/timeout", "60s"))
602+
Expect(OSCtlplane.Spec.Ironic.APIOverride.Route.Annotations).Should(HaveKeyWithValue("api.ironic.openstack.org/timeout", "60s"))
603+
Expect(OSCtlplane.Spec.Ironic.InspectorOverride.Route.Annotations).Should(HaveKeyWithValue("inspector.ironic.openstack.org/timeout", "60s"))
600604
})
601605

602606
It("should create selfsigned issuer and public+internal CA and issuer", func() {

0 commit comments

Comments
 (0)