From df813ab53325608f02c0ba97c93548dc38c5dd07 Mon Sep 17 00:00:00 2001
From: Alex Welsh <alex@stackhpc.com>
Date: Tue, 11 Aug 2026 12:55:42 +0100
Subject: [PATCH] Ensure openvswitch-base children get ovs repo

Previously, openvswitch-base would enable the openvswitch repo, and
child containers (e.g. ovn-base, ovn-controller, etc) would be built
using ovn packages from that repo. Recently [1] builds were changed to
clean up repos at the end of the build, so now dependent containers must
re-enable the repo manually.

Also added an openvswitch repo override to tests to avoid regresssion.

[1] Ie5b9970a4575b6258dc238059d3f33ddfe155789
Closes-Bug: #2163314
Change-Id: I4ea1cc812167142743e3ebcbd532281bfe8a9799
Signed-off-by: Alex Welsh <alex@stackhpc.com>
---
 docker/ovn/ovn-base/Dockerfile.j2                    | 3 +++
 docker/ovn/ovn-controller/Dockerfile.j2              | 3 +++
 docker/ovn/ovn-nb-db-server/Dockerfile.j2            | 3 +++
 docker/ovn/ovn-northd/Dockerfile.j2                  | 3 +++
 docker/ovn/ovn-sb-db-server/Dockerfile.j2            | 3 +++
 releasenotes/notes/bug-2163314-d99dd8d0f7a76a95.yaml | 6 ++++++
 tests/templates/repos.yaml.j2                        | 5 +++++
 7 files changed, 26 insertions(+)
 create mode 100644 releasenotes/notes/bug-2163314-d99dd8d0f7a76a95.yaml

diff --git a/docker/ovn/ovn-base/Dockerfile.j2 b/docker/ovn/ovn-base/Dockerfile.j2
index 8f04685e56..8d0c17a904 100644
--- a/docker/ovn/ovn-base/Dockerfile.j2
+++ b/docker/ovn/ovn-base/Dockerfile.j2
@@ -10,6 +10,9 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
 {{ macros.kolla_enable_base_repos() }}
 
 {% if base_package_type == 'rpm' %}
+
+{{ macros.enable_extra_repos(['openvswitch']) }}
+
     {% set ovn_base_packages = [
         'ovn${KOLLA_RPM_OVN_VERSION}',
     ] %}
diff --git a/docker/ovn/ovn-controller/Dockerfile.j2 b/docker/ovn/ovn-controller/Dockerfile.j2
index dfd26cbc42..9032c274b1 100644
--- a/docker/ovn/ovn-controller/Dockerfile.j2
+++ b/docker/ovn/ovn-controller/Dockerfile.j2
@@ -10,6 +10,9 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
 {{ macros.kolla_enable_base_repos() }}
 
 {% if base_package_type == 'rpm' %}
+
+{{ macros.enable_extra_repos(['openvswitch']) }}
+
     {% set ovn_controller_packages = [
         'ovn${KOLLA_RPM_OVN_VERSION}-host'
     ] %}
diff --git a/docker/ovn/ovn-nb-db-server/Dockerfile.j2 b/docker/ovn/ovn-nb-db-server/Dockerfile.j2
index 92433bdd29..436f0af16c 100644
--- a/docker/ovn/ovn-nb-db-server/Dockerfile.j2
+++ b/docker/ovn/ovn-nb-db-server/Dockerfile.j2
@@ -10,6 +10,9 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
 {{ macros.kolla_enable_base_repos() }}
 
 {% if base_package_type == 'rpm' %}
+
+{{ macros.enable_extra_repos(['openvswitch']) }}
+
     {% set ovn_nb_db_server_packages = [
         'ovn${KOLLA_RPM_OVN_VERSION}-central',
     ] %}
diff --git a/docker/ovn/ovn-northd/Dockerfile.j2 b/docker/ovn/ovn-northd/Dockerfile.j2
index b79e77a436..66e2dfb8a7 100644
--- a/docker/ovn/ovn-northd/Dockerfile.j2
+++ b/docker/ovn/ovn-northd/Dockerfile.j2
@@ -10,6 +10,9 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
 {{ macros.kolla_enable_base_repos() }}
 
 {% if base_package_type == 'rpm' %}
+
+{{ macros.enable_extra_repos(['openvswitch']) }}
+
     {% set ovn_northd_packages = [
         'ovn${KOLLA_RPM_OVN_VERSION}-central',
     ] %}
diff --git a/docker/ovn/ovn-sb-db-server/Dockerfile.j2 b/docker/ovn/ovn-sb-db-server/Dockerfile.j2
index 5f1ff92e0c..f11fbd029c 100644
--- a/docker/ovn/ovn-sb-db-server/Dockerfile.j2
+++ b/docker/ovn/ovn-sb-db-server/Dockerfile.j2
@@ -10,6 +10,9 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
 {{ macros.kolla_enable_base_repos() }}
 
 {% if base_package_type == 'rpm' %}
+
+{{ macros.enable_extra_repos(['openvswitch']) }}
+
     {% set ovn_sb_db_server_packages = [
         'ovn${KOLLA_RPM_OVN_VERSION}-central',
     ] %}
diff --git a/releasenotes/notes/bug-2163314-d99dd8d0f7a76a95.yaml b/releasenotes/notes/bug-2163314-d99dd8d0f7a76a95.yaml
new file mode 100644
index 0000000000..0683c8cedf
--- /dev/null
+++ b/releasenotes/notes/bug-2163314-d99dd8d0f7a76a95.yaml
@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Fixes child image builds from ``openvswitch-base`` with a custom
+    OpenvSwitch repo.
+    `LP#2163314 <https://launchpad.net/bugs/2163314>`__
diff --git a/tests/templates/repos.yaml.j2 b/tests/templates/repos.yaml.j2
index 9129f74eba..573173d952 100644
--- a/tests/templates/repos.yaml.j2
+++ b/tests/templates/repos.yaml.j2
@@ -32,6 +32,11 @@ rocky:
     build_only: true
     gpgkey: "file:///etc/pki/rpm-gpg/RPM-GPG-KEY-Rocky-10"
     name: "crb"
+  openvswitch:
+    baseurl: "http://centos-stream.mirrorservice.org/SIGs/$releasever-stream/nfv/$basearch/openvswitch-2/"
+    build_only: true
+    gpgkey: "https://www.centos.org/keys/RPM-GPG-KEY-CentOS-SIG-NFV"
+    name: "centos-nfv-openvswitch"
 {% elif base_distro == 'ubuntu' %}
 ubuntu:
   ubuntu: