From 2b9343a7a496c6916142ac231c14f996f87f8380 Mon Sep 17 00:00:00 2001 From: Nicholas Kuechler Date: Thu, 24 Jul 2025 11:44:08 -0500 Subject: [PATCH] feat(openstack-helm): Enable resource limits for OpenStack components in helm values --- components/cinder/values.yaml | 3 +++ components/horizon/values.yaml | 2 ++ components/ironic/values.yaml | 2 ++ components/keystone/values.yaml | 2 +- components/neutron/values.yaml | 3 +++ components/nova/values.yaml | 2 ++ components/octavia/values.yaml | 4 ++++ components/placement/values.yaml | 2 ++ components/skyline/values.yaml | 2 ++ 9 files changed, 21 insertions(+), 1 deletion(-) diff --git a/components/cinder/values.yaml b/components/cinder/values.yaml index 73fb534e3..c5d3a5174 100644 --- a/components/cinder/values.yaml +++ b/components/cinder/values.yaml @@ -76,6 +76,9 @@ pod: # this should be set to no more than (pod.replicas.server - 1) # usually set on per-deployment basis. min_available: 0 + resources: + enabled: true + dependencies: dynamic: common: diff --git a/components/horizon/values.yaml b/components/horizon/values.yaml index 71d7437d9..4b54ab560 100644 --- a/components/horizon/values.yaml +++ b/components/horizon/values.yaml @@ -76,6 +76,8 @@ pod: # this should be set to no more than (pod.replicas.horizon - 1) # usually set on per-deployment basis. min_available: 0 + resources: + enabled: true annotations: # we need to modify the annotations on OpenStack Helm diff --git a/components/ironic/values.yaml b/components/ironic/values.yaml index 1b02933b1..ef49e30e4 100644 --- a/components/ironic/values.yaml +++ b/components/ironic/values.yaml @@ -219,6 +219,8 @@ pod: # this should be set to no more than (pod.replicas.api - 1) # usually set on per-deployment basis. min_available: 0 + resources: + enabled: true annotations: # we need to modify the annotations on OpenStack Helm diff --git a/components/keystone/values.yaml b/components/keystone/values.yaml index 5d0cde77c..92341147e 100644 --- a/components/keystone/values.yaml +++ b/components/keystone/values.yaml @@ -33,7 +33,7 @@ dependencies: pod: resources: - enabled: false + enabled: true api: requests: memory: "64Mi" diff --git a/components/neutron/values.yaml b/components/neutron/values.yaml index c08504f12..66fde281a 100644 --- a/components/neutron/values.yaml +++ b/components/neutron/values.yaml @@ -138,6 +138,9 @@ pod: - name: undersync-token secret: secretName: undersync-token + resources: + enabled: true + # (nicholas.kuechler) updating the jobs list to remove the 'neutron-rabbit-init' job. dependencies: dynamic: diff --git a/components/nova/values.yaml b/components/nova/values.yaml index 48f61050d..0a4e0f707 100644 --- a/components/nova/values.yaml +++ b/components/nova/values.yaml @@ -154,6 +154,8 @@ pod: # this should be set to no more than (pod.replicas.osapi - 1) # usually set on per-deployment basis. min_available: 0 + resources: + enabled: true manifests: job_db_init: false diff --git a/components/octavia/values.yaml b/components/octavia/values.yaml index f7561c144..62467b7c5 100644 --- a/components/octavia/values.yaml +++ b/components/octavia/values.yaml @@ -82,6 +82,10 @@ dependencies: db_sync: jobs: +pod: + resources: + enabled: true + manifests: job_db_init: false job_rabbit_init: false diff --git a/components/placement/values.yaml b/components/placement/values.yaml index e913d368b..70c01a8ee 100644 --- a/components/placement/values.yaml +++ b/components/placement/values.yaml @@ -30,6 +30,8 @@ pod: # this should be set to no more than (pod.replicas.api - 1) # usually set on per-deployment basis. min_available: 0 + resources: + enabled: true endpoints: placement: diff --git a/components/skyline/values.yaml b/components/skyline/values.yaml index a4ac041a7..b70260742 100644 --- a/components/skyline/values.yaml +++ b/components/skyline/values.yaml @@ -43,6 +43,8 @@ pod: # this should be set to no more than (pod.replicas.skyline - 1) # usually set on per-deployment basis. min_available: 0 + resources: + enabled: true annotations: # we need to modify the annotations on OpenStack Helm