From 06ba21861550a91642abb84fb0577cebe310194f Mon Sep 17 00:00:00 2001 From: Alex Figueiredo Date: Mon, 24 Mar 2025 17:37:47 -0300 Subject: [PATCH] Removes unused docker images By upversioning openstack-helm [1], openstack-helm-infra [2] and integrating the new ingress-nginx chart [3], new docker images were introduced to stx-openstack. Even though some of these images are not used by the app, their image values are not set to null and they are still downloaded during the application apply. This change removes the following images that are currently not required for stx-openstack: - bitnami/kubectl - ingress-nginx/kube-webhook-certgen - ingress-nginx/opentelemetry [1]https://review.opendev.org/c/starlingx/openstack-armada-app/+/941982 [2]https://review.opendev.org/c/starlingx/openstack-armada-app/+/941981 [3]https://review.opendev.org/c/starlingx/openstack-armada-app/+/936194 Test Plan: [PASS] build stx-openstack packages and tarball [PASS] apply stx-openstack tarball to a virtual AIO-SX system [PASS] check sysinv log and docker registries to ensure that images removed are no longer downloaded. Closes-Bug: #2104057 Change-Id: I8c0ff659c0d9ab4bba2117c54a1ae1576d517479 Signed-off-by: Alex Figueiredo --- .../ingress-nginx-static-overrides.yaml | 6 ++---- .../manifests/libvirt/libvirt-static-overrides.yaml | 1 + 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/ingress-nginx-openstack/ingress-nginx-static-overrides.yaml b/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/ingress-nginx-openstack/ingress-nginx-static-overrides.yaml index 7671818a..f867ffce 100644 --- a/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/ingress-nginx-openstack/ingress-nginx-static-overrides.yaml +++ b/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/ingress-nginx-openstack/ingress-nginx-static-overrides.yaml @@ -11,8 +11,7 @@ nameOverride: ingress-nginx-openstack controller: opentelemetry: image: - repository: registry.k8s.io/ingress-nginx/opentelemetry - tag: "v20230721-3e2062ee5" + image: null config: bind-address: '::' worker-processes: "4" @@ -68,8 +67,7 @@ controller: admissionWebhooks: patch: image: - repository: registry.k8s.io/ingress-nginx/kube-webhook-certgen - tag: v1.4.1 + image: null enabled: false updateStrategy: type: RollingUpdate diff --git a/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/libvirt/libvirt-static-overrides.yaml b/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/libvirt/libvirt-static-overrides.yaml index f737af1d..0b139bbe 100644 --- a/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/libvirt/libvirt-static-overrides.yaml +++ b/stx-openstack-helm-fluxcd/stx-openstack-helm-fluxcd/manifests/libvirt/libvirt-static-overrides.yaml @@ -43,4 +43,5 @@ images: libvirt: docker.io/starlingx/stx-libvirt:master-debian-stable-latest pre_apply_cleanup: docker.io/starlingx/stx-vault-manager:master-debian-stable-latest libvirt_exporter: null + kubectl: null ...