Skip to content

Commit 61f1705

Browse files
mnasiadkayoctozepto
authored andcommitted
Use python3-openvswitch from distro
We should use the same Python OVS bindings package version as running OpenvSwitch. See related bug. Closes-Bug: #1961874 (cherry picked from commit 8e8a499) Change-Id: Ideae0c81d7c3833c8b0e22fce8691e9e42949ec5
1 parent 9b1ff36 commit 61f1705

File tree

4 files changed

+21
-2
lines changed

4 files changed

+21
-2
lines changed

docker/nova/nova-base/Dockerfile.j2

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
5656
{% if base_package_type == 'rpm' %}
5757

5858
{% set nova_base_packages = [
59-
'openvswitch'
59+
'openvswitch',
60+
'python3-openvswitch'
6061
] %}
6162
{% if base_arch == 'x86_64' %}
6263
{% set nova_base_packages = nova_base_packages + [
@@ -79,7 +80,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
7980
'iptables',
8081
'openssh-client',
8182
'openvswitch-switch',
82-
'python3-libvirt'
83+
'python3-libvirt',
84+
'python3-openvswitch'
8385
] %}
8486
{% if base_arch == 'x86_64' %}
8587
{% set nova_base_packages = nova_base_packages + [

docker/octavia/octavia-base/Dockerfile.j2

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,12 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
2323
{{ macros.install_packages(octavia_base_packages | customizable("packages")) }}
2424

2525
{% elif install_type == 'source' %}
26+
{% set octavia_base_packages = [
27+
'python3-openvswitch'
28+
] %}
29+
30+
{{ macros.install_packages(octavia_base_packages | customizable("packages")) }}
31+
2632
ADD octavia-base-archive /octavia-base-source
2733

2834
{% set octavia_base_pip_packages = [

docker/openstack-base/Dockerfile.j2

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -312,6 +312,11 @@ ENV DEBIAN_FRONTEND noninteractive
312312

313313
ADD openstack-base-archive /openstack-base-source
314314
RUN ln -s openstack-base-source/* /requirements \
315+
{# NOTE(mnasiadka): Remove ovs from upper-constraints.txt because python3-openvswitch
316+
is usually newer than UC entry and older version would get installed
317+
in venv (see https://launchpad.net/bugs/1961874).
318+
#}
319+
&& sed -i /^ovs=/d /requirements/upper-constraints.txt \
315320
&& mkdir -p /var/lib/kolla \
316321
&& {{ macros.install_pip(['virtualenv'])}} \
317322
&& virtualenv --system-site-packages /var/lib/kolla/venv
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
---
2+
fixes:
3+
- |
4+
Fixes an issue when older version of Python OpenvSwitch bindings package
5+
was used, than the running OpenvSwitch code.
6+
`LP#1961874 <https://launchpad.net/bugs/1961874>`__

0 commit comments

Comments
 (0)