From 29430a654ae97a8810d9a94009ada905f59c2bd4 Mon Sep 17 00:00:00 2001 From: Anthony Harivel Date: Thu, 18 Jun 2026 15:02:17 +0200 Subject: [PATCH 1/2] fix(control-plane): add noVNCProxyServiceTemplate to nova cell1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The base OpenStackControlPlane defines cell1 without a noVNCProxyServiceTemplate, which means VNC console access to VMs scheduled on cell1 compute nodes is not available. The authoritative RHOSO documentation specifies that cell1 should include noVNCProxyServiceTemplate with enabled: true and a ctlplane network attachment. Without this, operators must manually patch every deployment to enable console access — a common source of post-deploy debugging. This affects all validated architectures that inherit from lib/control-plane/base. Co-Authored-By: Claude Opus 4.6 Signed-off-by: Anthony Harivel --- lib/control-plane/base/openstackcontrolplane.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lib/control-plane/base/openstackcontrolplane.yaml b/lib/control-plane/base/openstackcontrolplane.yaml index 6ce9bee45..84bf63ed0 100644 --- a/lib/control-plane/base/openstackcontrolplane.yaml +++ b/lib/control-plane/base/openstackcontrolplane.yaml @@ -204,6 +204,10 @@ spec: cluster: rabbitmq-cell1 conductorServiceTemplate: replicas: 1 + noVNCProxyServiceTemplate: + enabled: true + networkAttachments: + - ctlplane hasAPIAccess: true octavia: enabled: false From ed23b12c3b169a950ebf6c22f72c3bab1c60aaa0 Mon Sep 17 00:00:00 2001 From: Anthony Harivel Date: Thu, 18 Jun 2026 15:02:34 +0200 Subject: [PATCH 2/2] feat(nfv/ovs-dpdk-sriov): add metricStorage replacement from service-values MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The NFV OVS-DPDK-SRIOV validated architecture enables telemetry and ceilometer via service-values replacements, but metricStorage (Prometheus monitoring stack) has no replacement — it stays at the base default of enabled: false regardless of what the deployer sets in service-values. This adds a kustomize replacement for telemetry.template.metricStorage.enabled so NFV deployments can toggle Prometheus metric collection through service-values.yaml, consistent with how telemetry.enabled and ceilometer.enabled are already handled. Also adds the metricStorage.enabled field to example service-values for both ovs-dpdk-sriov and ovs-dpdk-sriov-ipv6 to satisfy the kustomize replacement source requirement and provide a working default. Co-Authored-By: Claude Opus 4.6 --- .../va/nfv/ovs-dpdk-sriov-ipv6/service-values.yaml | 2 ++ examples/va/nfv/ovs-dpdk-sriov/service-values.yaml | 2 ++ va/nfv/ovs-dpdk-sriov/kustomization.yaml | 12 ++++++++++++ 3 files changed, 16 insertions(+) diff --git a/examples/va/nfv/ovs-dpdk-sriov-ipv6/service-values.yaml b/examples/va/nfv/ovs-dpdk-sriov-ipv6/service-values.yaml index e6746a373..08f9487cc 100644 --- a/examples/va/nfv/ovs-dpdk-sriov-ipv6/service-values.yaml +++ b/examples/va/nfv/ovs-dpdk-sriov-ipv6/service-values.yaml @@ -63,6 +63,8 @@ data: template: ceilometer: enabled: true + metricStorage: + enabled: true extraMounts: - name: v1 region: r1 diff --git a/examples/va/nfv/ovs-dpdk-sriov/service-values.yaml b/examples/va/nfv/ovs-dpdk-sriov/service-values.yaml index e6746a373..08f9487cc 100644 --- a/examples/va/nfv/ovs-dpdk-sriov/service-values.yaml +++ b/examples/va/nfv/ovs-dpdk-sriov/service-values.yaml @@ -63,6 +63,8 @@ data: template: ceilometer: enabled: true + metricStorage: + enabled: true extraMounts: - name: v1 region: r1 diff --git a/va/nfv/ovs-dpdk-sriov/kustomization.yaml b/va/nfv/ovs-dpdk-sriov/kustomization.yaml index 3bb52f53b..276dcec9c 100644 --- a/va/nfv/ovs-dpdk-sriov/kustomization.yaml +++ b/va/nfv/ovs-dpdk-sriov/kustomization.yaml @@ -128,3 +128,15 @@ replacements: - spec.nova.template.schedulerServiceTemplate.customServiceConfig options: create: true + # Telemetry metric storage toggle for NFV deployments + - source: + kind: ConfigMap + name: service-values + fieldPath: data.telemetry.template.metricStorage.enabled + targets: + - select: + kind: OpenStackControlPlane + fieldPaths: + - spec.telemetry.template.metricStorage.enabled + options: + create: true