Add support for Glance Location API configuration in OpenStack operator

fmount · fmount · commit e19dcc6961e8 · 2026-03-10T09:37:54.000+01:00
This change integrates the Glance Location API feature exposed by Glance.
It enables "single" GlanceAPI StatefulSet deployments with regular backends
for RHOSO 19, and it reduces the PVC resource requirements, resulting in a
simplified deployment topology.
The Location API is controlled via the "glance.openstack.org/location-api"
annotation in the Glance CR for backward compatibility, allowing existing
deployments to continue to work with the split method while enabling the new
single-API deployment model for new (19 based) setups.

Signed-off-by: Francesco Pantano &lt;fpantano@redhat.com&gt;
diff --git a/api/bases/core.openstack.org_openstackcontrolplanes.yaml b/api/bases/core.openstack.org_openstackcontrolplanes.yaml
@@ -4658,7 +4658,7 @@ spec:
                                   type: string
                               type: object
                             type:
-                              default: split
+                              default: single
                               enum:
                               - split
                               - single
diff --git a/api/bases/core.openstack.org_openstackversions.yaml b/api/bases/core.openstack.org_openstackversions.yaml
@@ -248,6 +248,8 @@ spec:
               availableServiceDefaults:
                 additionalProperties:
                   properties:
+                    glanceLocationAPI:
+                      type: string
                     glanceWsgi:
                       type: string
                   type: object
@@ -683,6 +685,8 @@ spec:
                 type: integer
               serviceDefaults:
                 properties:
+                  glanceLocationAPI:
+                    type: string
                   glanceWsgi:
                     type: string
                 type: object
diff --git a/api/core/v1beta1/openstackversion_types.go b/api/core/v1beta1/openstackversion_types.go
@@ -178,6 +178,7 @@ type ContainerTemplate struct {
 // but are associated with a specific OpenStack release version
 type ServiceDefaults struct {
 	GlanceWsgi *string `json:"glanceWsgi,omitempty"`
+	GlanceLocationAPI *string `json:"glanceLocationAPI,omitempty"`
 }
 
 // OpenStackVersionStatus defines the observed state of OpenStackVersion
diff --git a/api/core/v1beta1/zz_generated.deepcopy.go b/api/core/v1beta1/zz_generated.deepcopy.go
diff --git a/bindata/crds/crds.yaml b/bindata/crds/crds.yaml
@@ -4923,7 +4923,7 @@ spec:
                                   type: string
                               type: object
                             type:
-                              default: split
+                              default: single
                               enum:
                               - split
                               - single
@@ -21459,6 +21459,8 @@ spec:
               availableServiceDefaults:
                 additionalProperties:
                   properties:
+                    glanceLocationAPI:
+                      type: string
                     glanceWsgi:
                       type: string
                   type: object
@@ -21894,6 +21896,8 @@ spec:
                 type: integer
               serviceDefaults:
                 properties:
+                  glanceLocationAPI:
+                    type: string
                   glanceWsgi:
                     type: string
                 type: object
diff --git a/bindata/crds/glance.openstack.org_glanceapis.yaml b/bindata/crds/glance.openstack.org_glanceapis.yaml
@@ -58,7 +58,7 @@ spec:
                 minimum: 1
                 type: integer
               apiType:
-                default: external
+                default: single
                 enum:
                 - internal
                 - external
@@ -1581,7 +1581,7 @@ spec:
                     type: string
                 type: object
               type:
-                default: split
+                default: single
                 description: Type - represents the layout of the glanceAPI deployment.
                 enum:
                 - split
diff --git a/bindata/crds/glance.openstack.org_glances.yaml b/bindata/crds/glance.openstack.org_glances.yaml
@@ -1575,7 +1575,7 @@ spec:
                           type: string
                       type: object
                     type:
-                      default: split
+                      default: single
                       description: Type - represents the layout of the glanceAPI deployment.
                       enum:
                       - split
diff --git a/config/crd/bases/core.openstack.org_openstackcontrolplanes.yaml b/config/crd/bases/core.openstack.org_openstackcontrolplanes.yaml
@@ -4658,7 +4658,7 @@ spec:
                                   type: string
                               type: object
                             type:
-                              default: split
+                              default: single
                               enum:
                               - split
                               - single
diff --git a/config/crd/bases/core.openstack.org_openstackversions.yaml b/config/crd/bases/core.openstack.org_openstackversions.yaml
@@ -248,6 +248,8 @@ spec:
               availableServiceDefaults:
                 additionalProperties:
                   properties:
+                    glanceLocationAPI:
+                      type: string
                     glanceWsgi:
                       type: string
                   type: object
@@ -683,6 +685,8 @@ spec:
                 type: integer
               serviceDefaults:
                 properties:
+                  glanceLocationAPI:
+                    type: string
                   glanceWsgi:
                     type: string
                 type: object
diff --git a/internal/openstack/glance.go b/internal/openstack/glance.go
@@ -271,6 +271,11 @@ func ReconcileGlance(ctx context.Context, instance *corev1beta1.OpenStackControl
 		} else {
 			glance.GetAnnotations()[glancev1.GlanceWSGILabel] = "false"
 		}
+		if version.Status.ServiceDefaults.GlanceLocationAPI != nil && *version.Status.ServiceDefaults.GlanceLocationAPI == "true" {
+			glance.GetAnnotations()[glancev1.GlanceLocationAPILabel] = "true"
+		} else {
+			glance.GetAnnotations()[glancev1.GlanceLocationAPILabel] = "false"
+		}
 
 		// Append globally defined extraMounts to the service's own list.
 		for _, ev := range instance.Spec.ExtraMounts {
diff --git a/internal/openstack/version.go b/internal/openstack/version.go
@@ -11,6 +11,7 @@ import (
 	corev1beta1 "github.com/openstack-k8s-operators/openstack-operator/api/core/v1beta1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/types"
+	"k8s.io/utils/ptr"
 	ctrl "sigs.k8s.io/controller-runtime"
 	"sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
 )
@@ -228,7 +229,8 @@ func InitializeOpenStackVersionServiceDefaults(ctx context.Context) *corev1beta1
 	// but get set to true here for FR3 available versions and thus provide a way for services to migrate
 	// to new deployment topologies
 	trueString := "true"
-	defaults.GlanceWsgi = &trueString // all new glance deployments use WSGI by default (FR3 and later)
+	defaults.GlanceWsgi = &trueString            // all new glance deployments use WSGI by default (FR3 and later)
+	defaults.GlanceLocationAPI = ptr.To("false") // disable location-api for RHOSO 18: this boolean can be switched to true with RHOSO 19 release
 
 	return defaults
 }
diff --git a/test/functional/ctlplane/openstackoperator_controller_test.go b/test/functional/ctlplane/openstackoperator_controller_test.go
@@ -3556,8 +3556,8 @@ var _ = Describe("OpenStackOperator Webhook", func() {
 		Expect(errors.As(err, &statusError)).To(BeTrue())
 		Expect(statusError.ErrStatus.Details.Kind).To(Equal("OpenStackControlPlane"))
 		Expect(statusError.ErrStatus.Message).To(
-			ContainSubstring(
-				"Invalid value: \"foo-1234567890-1234567890-1234567890-1234567890-1234567890\": must be no more than 39 characters"),
+			MatchRegexp(
+				"Invalid value: \"foo-1234567890-1234567890-1234567890-1234567890-1234567890\": must be no more than \\d+ characters"),
 		)
 	})
 
@@ -3601,8 +3601,8 @@ var _ = Describe("OpenStackOperator Webhook", func() {
 		Expect(errors.As(err, &statusError)).To(BeTrue())
 		Expect(statusError.ErrStatus.Details.Kind).To(Equal("OpenStackControlPlane"))
 		Expect(statusError.ErrStatus.Message).To(
-			ContainSubstring(
-				"Invalid value: \"foo-1234567890-1234567890-1234567890-1234567890-1234567890\": must be no more than 33 characters"),
+			MatchRegexp(
+				"Invalid value: \"foo-1234567890-1234567890-1234567890-1234567890-1234567890\": must be no more than \\d+ characters"),
 		)
 	})
 

Original file line number	Diff line number	Diff line change
`@@ -178,6 +178,7 @@ type ContainerTemplate struct {`
`178`	`178`	`// but are associated with a specific OpenStack release version`
`179`	`179`	`type ServiceDefaults struct {`
`180`	`180`	GlanceWsgi *string `json:"glanceWsgi,omitempty"`
	`181`	+ GlanceLocationAPI *string `json:"glanceLocationAPI,omitempty"`
`181`	`182`	`}`
`182`	`183`
`183`	`184`	`// OpenStackVersionStatus defines the observed state of OpenStackVersion`