Add keystone configurable API Timeout

Depends-On: openstack-k8s-operators/keystone-operator#515 Signed-off-by: Veronika Fisarova <[email protected]>
Deydra71 · Jan 21, 2025 · 5f4253a · 5f4253a
1 parent 280c2ab
commit 5f4253a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/apis/core/v1beta1/openstackcontrolplane_webhook.go b/apis/core/v1beta1/openstackcontrolplane_webhook.go
@@ -771,6 +771,8 @@ func (r *OpenStackControlPlane) DefaultServices() {
 			r.Spec.Keystone.Template = &keystonev1.KeystoneAPISpecCore{}
 		}
 		r.Spec.Keystone.Template.Default()
+		initializeOverrideSpec(&r.Spec.Keystone.APIOverride.Route, true)
+		r.Spec.Keystone.Template.SetDefaultRouteAnnotations(r.Spec.Keystone.APIOverride.Route.Annotations)
 	}
 
 	// Manila

diff --git a/tests/functional/ctlplane/openstackoperator_controller_test.go b/tests/functional/ctlplane/openstackoperator_controller_test.go
@@ -594,6 +594,9 @@ var _ = Describe("OpenStackOperator controller", func() {
 			// Expect(OSCtlplane.Spec.Barbican.APIOverride.Route).Should(Not(BeNil()))
 			// Expect(OSCtlplane.Spec.Barbican.APIOverride.Route.Annotations).Should(HaveKeyWithValue("haproxy.router.openshift.io/timeout", "90s"))
 			// Expect(OSCtlplane.Spec.Barbican.APIOverride.Route.Annotations).Should(HaveKeyWithValue("api.barbican.openstack.org/timeout", "90s"))
+			Expect(OSCtlplane.Spec.Keystone.APIOverride.Route).Should(Not(BeNil()))
+			Expect(OSCtlplane.Spec.Keystone.APIOverride.Route.Annotations).Should(HaveKeyWithValue("haproxy.router.openshift.io/timeout", "60s"))
+			Expect(OSCtlplane.Spec.Keystone.APIOverride.Route.Annotations).Should(HaveKeyWithValue("api.keystone.openstack.org/timeout", "60s"))
 		})
 
 		It("should create selfsigned issuer and public+internal CA and issuer", func() {
@@ -875,6 +878,9 @@ var _ = Describe("OpenStackOperator controller", func() {
 			// Expect(OSCtlplane.Spec.Barbican.APIOverride.Route).Should(Not(BeNil()))
 			// Expect(OSCtlplane.Spec.Barbican.APIOverride.Route.Annotations).Should(HaveKeyWithValue("haproxy.router.openshift.io/timeout", "90s"))
 			// Expect(OSCtlplane.Spec.Barbican.APIOverride.Route.Annotations).Should(HaveKeyWithValue("api.barbican.openstack.org/timeout", "90s"))
+			Expect(OSCtlplane.Spec.Keystone.APIOverride.Route).Should(Not(BeNil()))
+			Expect(OSCtlplane.Spec.Keystone.APIOverride.Route.Annotations).Should(HaveKeyWithValue("haproxy.router.openshift.io/timeout", "60s"))
+			Expect(OSCtlplane.Spec.Keystone.APIOverride.Route.Annotations).Should(HaveKeyWithValue("api.keystone.openstack.org/timeout", "60s"))
 		})
 
 		It("should create selfsigned issuer and public, internal, libvirt and ovn CA and issuer", func() {