Skip to content

Commit 56d6392

Browse files
committed
Ensure nvme-cli is present in nova-compute images
The nvme-cli package is required to get the nvme command executed by os-brick, which is used by nova-compute. We don't need to explicitely install it for centos binary images, as it is required by os-brick which is required by openstack-nova-common, but all other types of images were missing it. Change-Id: I754939da7636c57d2a8d5b83debb5d8a58e38432 Closes-Bug: #1953509 (cherry picked from commit 30eddf7)
1 parent fcd525d commit 56d6392

File tree

3 files changed

+10
-4
lines changed

3 files changed

+10
-4
lines changed

docker/nova/nova-compute-ironic/Dockerfile.j2

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,12 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
77

88
{% import "macros.j2" as macros with context %}
99

10-
{% set nova_compute_ironic_packages = ['genisoimage'] %}
10+
{% set nova_compute_ironic_packages = ['genisoimage', 'nvme-cli'] %}
1111

1212
{% if install_type == 'binary' %}
1313
{% if base_package_type == 'rpm' %}
1414

1515
{% set nova_compute_ironic_packages = nova_compute_ironic_packages + [
16-
'nvme-cli',
1716
'openstack-nova-compute'
1817
] %}
1918
{{ macros.install_packages(nova_compute_ironic_packages | customizable("packages")) }}

docker/nova/nova-compute/Dockerfile.j2

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
2020
'iscsi-initiator-utils',
2121
'ndctl',
2222
'nfs-utils',
23-
'nvme-cli',
2423
'openstack-nova-compute',
2524
'openvswitch',
2625
'parted',
@@ -54,8 +53,9 @@ LABEL maintainer="{{ maintainer }}" name="{{ image_name }}" build-date="{{ build
5453
'multipath-tools',
5554
'nfs-common',
5655
'nova-compute',
57-
'openvswitch-switch',
56+
'nvme-cli',
5857
'open-iscsi',
58+
'openvswitch-switch',
5959
'parted',
6060
'pmdk-tools',
6161
'python3-cephfs',
@@ -96,6 +96,7 @@ RUN rm -f /etc/nova/nova-compute.conf
9696
'libosinfo',
9797
'ndctl',
9898
'nfs-utils',
99+
'nvme-cli',
99100
'openvswitch',
100101
'parted',
101102
'python3-libguestfs',
@@ -128,6 +129,7 @@ RUN rm -f /etc/nova/nova-compute.conf
128129
'libosinfo-bin',
129130
'multipath-tools',
130131
'nfs-common',
132+
'nvme-cli',
131133
'open-iscsi',
132134
'parted',
133135
'pmdk-tools',
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
fixes:
3+
- |
4+
Ensures the ``nvme-cli`` package is present in ``nova-compute`` images, as
5+
it expected by ``os-brick``.

0 commit comments

Comments
 (0)