-
Notifications
You must be signed in to change notification settings - Fork 110
Expand file tree
/
Copy pathglance.go
More file actions
355 lines (320 loc) · 14.7 KB
/
Copy pathglance.go
File metadata and controls
355 lines (320 loc) · 14.7 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
package openstack
import (
"context"
"fmt"
glancev1 "github.com/openstack-k8s-operators/glance-operator/api/v1beta1"
rabbitmqv1 "github.com/openstack-k8s-operators/infra-operator/apis/rabbitmq/v1beta1"
"github.com/openstack-k8s-operators/lib-common/modules/common/condition"
"github.com/openstack-k8s-operators/lib-common/modules/common/helper"
"github.com/openstack-k8s-operators/lib-common/modules/common/service"
"github.com/openstack-k8s-operators/lib-common/modules/common/util"
"github.com/openstack-k8s-operators/lib-common/modules/common/webhook"
corev1beta1 "github.com/openstack-k8s-operators/openstack-operator/api/core/v1beta1"
k8s_errors "k8s.io/apimachinery/pkg/api/errors"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/types"
ctrl "sigs.k8s.io/controller-runtime"
"sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
)
const (
// svcSelector is used as selector to get the list of "Services" associated
// to a specific glanceAPI instance. It must be different from glanceAPI
// label set by the service operator as in case of split glanceAPI type the
// the label on public svc gets set to -external and internal instance svc
// to -internal instead of the glance top level glanceType split
svcSelector = "tlGlanceAPI"
)
// ReconcileGlance -
func ReconcileGlance(ctx context.Context, instance *corev1beta1.OpenStackControlPlane, version *corev1beta1.OpenStackVersion, helper *helper.Helper) (ctrl.Result, error) {
Log := GetLogger(ctx)
// Trigger webhook to cache service name if UniquePodNames is enabled and not yet cached
// This handles operator upgrade scenario where existing CRs don't have ServiceName set
if instance.Spec.Glance.UniquePodNames && instance.Spec.Glance.ServiceName == "" {
return webhook.EnsureWebhookTrigger(ctx, instance, corev1beta1.ReconcileTriggerAnnotation, "Glance service name caching", Log, 0)
}
glanceName, altGlanceName := instance.GetServiceNameCached(corev1beta1.GlanceName, instance.Spec.Glance.UniquePodNames, instance.Spec.Glance.ServiceName)
// Ensure the alternate glance CR doesn't exist, as the randomPodNames flag may have been toggled
glance := &glancev1.Glance{
ObjectMeta: metav1.ObjectMeta{
Name: altGlanceName,
Namespace: instance.Namespace,
},
}
if res, err := EnsureDeleted(ctx, helper, glance); err != nil {
return res, err
}
glance = &glancev1.Glance{
ObjectMeta: metav1.ObjectMeta{
Name: glanceName,
Namespace: instance.Namespace,
},
}
if !instance.Spec.Glance.Enabled {
if res, err := EnsureDeleted(ctx, helper, glance); err != nil {
return res, err
}
instance.Status.Conditions.Remove(corev1beta1.OpenStackControlPlaneGlanceReadyCondition)
instance.Status.Conditions.Remove(corev1beta1.OpenStackControlPlaneExposeGlanceReadyCondition)
instance.Status.ContainerImages.GlanceAPIImage = nil
// Clean up AC CRs when service is disabled
if err := CleanupApplicationCredentialForService(ctx, helper, instance, glance.Name); err != nil {
return ctrl.Result{}, err
}
return ctrl.Result{}, nil
}
if instance.Spec.Glance.Template == nil {
instance.Spec.Glance.Template = &glancev1.GlanceSpecCore{}
}
// Migration and inheritance: Set NotificationsBus.Cluster with correct priority order
if instance.Spec.Glance.Template.NotificationsBus == nil || instance.Spec.Glance.Template.NotificationsBus.Cluster == "" {
// Priority 1: Migrate from service-level deprecated field (pointer type)
if instance.Spec.Glance.Template.NotificationBusInstance != nil && *instance.Spec.Glance.Template.NotificationBusInstance != "" {
if instance.Spec.Glance.Template.NotificationsBus == nil {
instance.Spec.Glance.Template.NotificationsBus = &rabbitmqv1.RabbitMqConfig{}
}
instance.Spec.Glance.Template.NotificationsBus.Cluster = *instance.Spec.Glance.Template.NotificationBusInstance
// Priority 2: Inherit from top-level NotificationsBus
} else if instance.Spec.NotificationsBus != nil && instance.Spec.NotificationsBus.Cluster != "" {
instance.Spec.Glance.Template.NotificationsBus = instance.Spec.NotificationsBus
}
// No Priority 3 default - NotificationsBus is optional
}
// Clear deprecated field after migration
if instance.Spec.Glance.Template.NotificationsBus != nil && instance.Spec.Glance.Template.NotificationsBus.Cluster != "" {
instance.Spec.Glance.Template.NotificationBusInstance = nil
}
if instance.Spec.Glance.Template.NodeSelector == nil && len(instance.Spec.NodeSelector) > 0 {
instance.Spec.Glance.Template.NodeSelector = &instance.Spec.NodeSelector
}
// When there's no Topology referenced in the Service Template, inject the
// top-level one
// NOTE: This does not check the Service subCRs: by default the generated
// subCRs inherit the top-level TopologyRef unless an override is present
if instance.Spec.Glance.Template.TopologyRef == nil {
instance.Spec.Glance.Template.TopologyRef = instance.Spec.TopologyRef
}
// When component services got created check if there is the need to create a route
if err := helper.GetClient().Get(ctx, types.NamespacedName{Name: glanceName, Namespace: instance.Namespace}, glance); err != nil {
if !k8s_errors.IsNotFound(err) {
return ctrl.Result{}, err
}
}
// Application Credential Management (Day-2 operation)
// Check if AC should be enabled and manage it accordingly
glanceReady := glance.Status.Conditions != nil && glance.Status.ObservedGeneration == glance.Generation && glance.IsReady()
// Apply same fallback logic as in CreateOrPatch to avoid passing empty values to AC
glanceSecret := instance.Spec.Glance.Template.Secret
if glanceSecret == "" {
glanceSecret = instance.Spec.Secret
}
// Check if any GlanceAPI has AC configured
hasACConfigured := false
for _, glanceAPI := range instance.Spec.Glance.Template.GlanceAPIs {
if glanceAPI.Auth.ApplicationCredentialSecret != "" {
hasACConfigured = true
break
}
}
// Always reconcile AC - EnsureApplicationCredentialForService checks cluster state and handles the full AC lifecycle.
if instance.Spec.Glance.ApplicationCredential != nil || hasACConfigured {
acSecretName, acResult, err := EnsureApplicationCredentialForService(
ctx,
helper,
instance,
glance.Name,
glanceReady,
glanceSecret,
instance.Spec.Glance.Template.PasswordSelectors.Service,
instance.Spec.Glance.Template.ServiceUser,
instance.Spec.Glance.ApplicationCredential,
)
if err != nil {
return ctrl.Result{}, err
}
// If AC is not ready, return immediately without updating the service CR
if (acResult != ctrl.Result{}) {
return acResult, nil
}
// Set ApplicationCredentialSecret for all GlanceAPIs based on what the helper returned:
// - If AC disabled: returns ""
// - If AC enabled and ready: returns the AC secret name
for name, glanceAPI := range instance.Spec.Glance.Template.GlanceAPIs {
glanceAPI.Auth.ApplicationCredentialSecret = acSecretName
instance.Spec.Glance.Template.GlanceAPIs[name] = glanceAPI
}
}
// add selector to service overrides
for name, glanceAPI := range instance.Spec.Glance.Template.GlanceAPIs {
eps := []service.Endpoint{service.EndpointPublic, service.EndpointInternal}
// An Edge glanceAPI has an internal endpoint only
if glanceAPI.Type == glancev1.APIEdge {
eps = []service.Endpoint{service.EndpointInternal}
}
for _, endpointType := range eps {
if glanceAPI.Override.Service == nil {
glanceAPI.Override.Service = map[service.Endpoint]service.RoutedOverrideSpec{}
}
glanceAPI.Override.Service[endpointType] = AddServiceOpenStackOperatorLabel(
glanceAPI.Override.Service[endpointType], glance.Name)
svcOverride := glanceAPI.Override.Service[endpointType]
svcOverride.AddLabel(getGlanceAPILabelMap(glance.Name, name, glanceAPI.Type))
glanceAPI.Override.Service[endpointType] = svcOverride
}
// preserve any previously set TLS certs,set CA cert
if instance.Spec.TLS.PodLevel.Enabled {
glanceAPI.TLS.API = glance.Spec.GlanceAPIs[name].TLS.API
}
glanceAPI.TLS.CaBundleSecretName = instance.Status.TLS.CaBundleSecretName
instance.Spec.Glance.Template.GlanceAPIs[name] = glanceAPI
}
var changed = false
for name, glanceAPI := range instance.Spec.Glance.Template.GlanceAPIs {
// Retrieve the services by Label and filter on glanceAPI: for
// each instance we should get **only** the associated `SVCs`
// and not the whole list. As per the Glance design doc we know
// that a given instance name is made in the form: "<service>
// <apiName> <apiType>", so we build the filter accordingly
// to resolve the label as <service>-<apiName>
svcs, err := service.GetServicesListWithLabel(
ctx,
helper,
instance.Namespace,
util.MergeMaps(
GetServiceOpenStackOperatorLabel(glance.Name),
getGlanceAPILabelMap(glance.Name, name, glanceAPI.Type),
),
)
if err != nil {
return ctrl.Result{}, err
}
// make sure to get to EndpointConfig when all service got created
// Webhook initializes APIOverride and always has at least the timeout override
if len(svcs.Items) == len(glanceAPI.Override.Service) {
endpointDetails, ctrlResult, err := EnsureEndpointConfig(
ctx,
instance,
helper,
glance,
svcs,
glanceAPI.Override.Service,
instance.Spec.Glance.APIOverride[name],
corev1beta1.OpenStackControlPlaneExposeGlanceReadyCondition,
false, // TODO (mschuppert) could be removed when all integrated service support TLS
glanceAPI.TLS,
)
if err != nil {
return ctrlResult, err
}
// set service overrides
glanceAPI.Override.Service = endpointDetails.GetEndpointServiceOverrides()
// update TLS cert secret, but skip Public endpoint for Edge
// instances
if glanceAPI.Type != glancev1.APIEdge {
glanceAPI.TLS.API.Public.SecretName = endpointDetails.GetEndptCertSecret(service.EndpointPublic)
}
glanceAPI.TLS.API.Internal.SecretName = endpointDetails.GetEndptCertSecret(service.EndpointInternal)
// let's keep track of changes for any instance, but return
// only when the iteration on the whole APIList is over
if (ctrlResult != ctrl.Result{}) {
changed = true
}
}
instance.Spec.Glance.Template.GlanceAPIs[name] = glanceAPI
}
if changed {
return ctrl.Result{}, nil
}
Log.Info("Reconciling Glance", "Glance.Namespace", instance.Namespace, "Glance.Name", glanceName)
op, err := controllerutil.CreateOrPatch(ctx, helper.GetClient(), glance, func() error {
instance.Spec.Glance.Template.DeepCopyInto(&glance.Spec.GlanceSpecCore)
glance.Spec.ContainerImage = *version.Status.ContainerImages.GlanceAPIImage
if glance.Spec.Secret == "" {
glance.Spec.Secret = instance.Spec.Secret
}
if glance.Spec.DatabaseInstance == "" {
glance.Spec.DatabaseInstance = "openstack"
}
if glance.Spec.Storage.StorageClass == "" {
glance.Spec.Storage.StorageClass = instance.Spec.StorageClass
}
if glance.GetAnnotations() == nil {
glance.SetAnnotations(make(map[string]string))
}
if version.Status.ServiceDefaults.GlanceWsgi != nil && *version.Status.ServiceDefaults.GlanceWsgi == "true" {
glance.GetAnnotations()[glancev1.GlanceWSGILabel] = "true"
} 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 {
glance.Spec.ExtraMounts = append(glance.Spec.ExtraMounts, glancev1.GlanceExtraVolMounts{
Name: ev.Name,
Region: ev.Region,
VolMounts: ev.VolMounts,
})
}
err := controllerutil.SetControllerReference(helper.GetBeforeObject(), glance, helper.GetScheme())
if err != nil {
return err
}
return nil
})
if err != nil {
instance.Status.Conditions.Set(condition.FalseCondition(
corev1beta1.OpenStackControlPlaneGlanceReadyCondition,
condition.ErrorReason,
condition.SeverityWarning,
corev1beta1.OpenStackControlPlaneGlanceReadyErrorMessage,
err.Error()))
return ctrl.Result{}, err
}
if op != controllerutil.OperationResultNone {
Log.Info(fmt.Sprintf("glance %s - %s", glance.Name, op))
}
if glance.Status.ObservedGeneration == glance.Generation && glance.IsReady() {
Log.Info("Glance ready condition is true")
instance.Status.ContainerImages.GlanceAPIImage = version.Status.ContainerImages.GlanceAPIImage
instance.Status.Conditions.MarkTrue(corev1beta1.OpenStackControlPlaneGlanceReadyCondition, corev1beta1.OpenStackControlPlaneGlanceReadyMessage)
} else {
// We want to mirror the condition of the highest priority from the Glance resource into the instance
// under the condition of type OpenStackControlPlaneGlanceReadyCondition, but only if the sub-resource
// currently has any conditions (which won't be true for the initial creation of the sub-resource, since
// it has not gone through a reconcile loop yet to have any conditions). If this condition ends up being
// the highest priority condition in the OpenStackControlPlane, it will appear in the OpenStackControlPlane's
// "Ready" condition at the end of the reconciliation loop, clearly surfacing the condition to the user in
// the "oc get oscontrolplane -n <namespace>" output.
if len(glance.Status.Conditions) > 0 {
MirrorSubResourceCondition(glance.Status.Conditions, corev1beta1.OpenStackControlPlaneGlanceReadyCondition, instance, glance.Kind)
} else {
// Default to the associated "running" condition message for the sub-resource if it currently lacks any conditions for mirroring
instance.Status.Conditions.Set(condition.FalseCondition(
corev1beta1.OpenStackControlPlaneGlanceReadyCondition,
condition.RequestedReason,
condition.SeverityInfo,
corev1beta1.OpenStackControlPlaneGlanceReadyRunningMessage))
}
}
return ctrl.Result{}, nil
}
func getGlanceAPILabelMap(name string, apiName string, apiType string) map[string]string {
apiFilter := fmt.Sprintf("%s-%s-%s", name, apiName, apiType)
return map[string]string{
svcSelector: apiFilter,
}
}
// GlanceImageMatch - return true if the glance images match on the ControlPlane and Version, or if Glance is not enabled
func GlanceImageMatch(ctx context.Context, controlPlane *corev1beta1.OpenStackControlPlane, version *corev1beta1.OpenStackVersion) bool {
Log := GetLogger(ctx)
if controlPlane.Spec.Glance.Enabled {
if !stringPointersEqual(controlPlane.Status.ContainerImages.GlanceAPIImage, version.Status.ContainerImages.GlanceAPIImage) {
Log.Info("Glance API image mismatch", "controlPlane.Status.ContainerImages.GlanceAPIImage", controlPlane.Status.ContainerImages.GlanceAPIImage, "version.Status.ContainerImages.GlanceAPIImage", version.Status.ContainerImages.GlanceAPIImage)
return false
}
}
return true
}