Skip to content

Commit 6760c2a

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 b779128 commit 6760c2a

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
@@ -53,7 +53,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
5353
{% if base_package_type == 'rpm' %}
5454

5555
{% set nova_base_packages = [
56-
'openvswitch'
56+
'openvswitch',
57+
'python3-openvswitch'
5758
] %}
5859
{% if base_arch == 'x86_64' %}
5960
{% set nova_base_packages = nova_base_packages + [
@@ -74,7 +75,8 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
7475
'iptables',
7576
'openssh-client',
7677
'openvswitch-switch',
77-
'python3-libvirt'
78+
'python3-libvirt',
79+
'python3-openvswitch'
7880
] %}
7981
{% if base_arch == 'x86_64' %}
8082
{% 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
@@ -24,6 +24,12 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
2424
{{ macros.install_packages(octavia_base_packages | customizable("packages")) }}
2525

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

2935
{% set octavia_base_pip_packages = [

docker/openstack-base/Dockerfile.j2

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

322322
ADD openstack-base-archive /openstack-base-source
323323
RUN ln -s openstack-base-source/* /requirements \
324+
{# NOTE(mnasiadka): Remove ovs from upper-constraints.txt because python3-openvswitch
325+
is usually newer than UC entry and older version would get installed
326+
in venv (see https://launchpad.net/bugs/1961874).
327+
#}
328+
&& sed -i /^ovs=/d /requirements/upper-constraints.txt \
324329
&& mkdir -p /var/lib/kolla \
325330
&& {{ macros.install_pip(['virtualenv'])}} \
326331
&& 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)