Skip to content

Commit 4ecd4f5

Browse files
Merge pull request #330 from Deydra71/aodh-apioverride
Update autoscaling to use service override
2 parents 25f01ea + 7183475 commit 4ecd4f5

File tree

7 files changed

+722
-669
lines changed

7 files changed

+722
-669
lines changed

api/bases/telemetry.openstack.org_autoscalings.yaml

Lines changed: 172 additions & 160 deletions
Large diffs are not rendered by default.

api/bases/telemetry.openstack.org_telemetries.yaml

Lines changed: 182 additions & 171 deletions
Large diffs are not rendered by default.

api/v1beta1/autoscaling_types.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,8 @@ type AodhCore struct {
124124
// APIOverrideSpec to override the generated manifest of several child resources.
125125
type APIOverrideSpec struct {
126126
// Override configuration for the Service created to serve traffic to the cluster.
127-
Service *service.RoutedOverrideSpec `json:"service,omitempty"`
127+
// The key must be the endpoint type (public, internal)
128+
Service map[service.Endpoint]service.RoutedOverrideSpec `json:"service,omitempty"`
128129
}
129130

130131
// AutoscalingSpec defines the desired state of Autoscaling

api/v1beta1/zz_generated.deepcopy.go

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/bases/telemetry.openstack.org_autoscalings.yaml

Lines changed: 172 additions & 160 deletions
Large diffs are not rendered by default.

config/crd/bases/telemetry.openstack.org_telemetries.yaml

Lines changed: 182 additions & 171 deletions
Large diffs are not rendered by default.

controllers/aodh_controller.go

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -329,17 +329,21 @@ func (r *AutoscalingReconciler) reconcileNormalAodh(
329329

330330
apiEndpoints := make(map[string]string)
331331

332+
if instance.Spec.Aodh.Override.Service == nil {
333+
instance.Spec.Aodh.Override.Service = make(map[service.Endpoint]service.RoutedOverrideSpec)
334+
}
335+
332336
for endpointType, data := range aodhEndpoints {
333337
endpointTypeStr := string(endpointType)
334338
endpointName := autoscaling.ServiceName + "-" + endpointTypeStr
335-
svcOverride := instance.Spec.Aodh.Override.Service
336-
if svcOverride == nil {
337-
svcOverride = &service.RoutedOverrideSpec{}
338-
}
339+
340+
svcOverride := instance.Spec.Aodh.Override.Service[endpointType]
339341
if svcOverride.EmbeddedLabelsAnnotations == nil {
340342
svcOverride.EmbeddedLabelsAnnotations = &service.EmbeddedLabelsAnnotations{}
341343
}
342344

345+
instance.Spec.Aodh.Override.Service[endpointType] = svcOverride
346+
343347
exportLabels := util.MergeStringMaps(
344348
serviceLabels,
345349
map[string]string{

0 commit comments

Comments
 (0)