aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCédric Jeanneret <cjeanner@redhat.com>2018-09-26 17:06:50 +0200
committerCédric Jeanneret <cjeanner@redhat.com>2018-11-07 16:32:04 +0100
commitb4a7bcae624e554e7ae982699dbf3ee5acb25457 (patch)
tree9c8b90f04d64ede817b8fe90f9419035414f47db
parent188c46cc4a89c12ea8c1d5d851b295f2b70ea283 (diff)
downloadkolla-b4a7bcae624e554e7ae982699dbf3ee5acb25457.tar.gz
kolla-b4a7bcae624e554e7ae982699dbf3ee5acb25457.zip
Do not load iscsi_tcp from within a container.
The module must be loaded from the host directly in order to avoid any SELinux issues. There are examples in kolla-ansible and tripleo-heat-templates: - https://review.openstack.org/#/c/605624/ - https://review.openstack.org/#/c/605450/ Change-Id: Ica68a109c3ed577e6a339ef315b576061cd557e5 Related-Bug: 1794550
-rw-r--r--docker/ironic/ironic-conductor/Dockerfile.j25
-rw-r--r--docker/ironic/ironic-conductor/extend_start.sh3
-rw-r--r--docker/ironic/ironic-conductor/iscsi_tcp_sudoers1
3 files changed, 0 insertions, 9 deletions
diff --git a/docker/ironic/ironic-conductor/Dockerfile.j2 b/docker/ironic/ironic-conductor/Dockerfile.j2
index c094835a2..d81204e7b 100644
--- a/docker/ironic/ironic-conductor/Dockerfile.j2
+++ b/docker/ironic/ironic-conductor/Dockerfile.j2
@@ -103,11 +103,6 @@ RUN {{ macros.install_pip(ironic_conductor_pip_packages | customizable("pip_pack
{{ macros.install_packages(ironic_conductor_packages | customizable("packages")) }}
-COPY extend_start.sh /usr/local/bin/kolla_ironic_extend_start
-COPY iscsi_tcp_sudoers /etc/sudoers.d/kolla_iscsi_tcp_sudoers
-RUN chmod 755 /usr/local/bin/kolla_ironic_extend_start \
- && chmod 440 /etc/sudoers.d/kolla_iscsi_tcp_sudoers
-
{% block ironic_conductor_footer %}{% endblock %}
{% block footer %}{% endblock %}
diff --git a/docker/ironic/ironic-conductor/extend_start.sh b/docker/ironic/ironic-conductor/extend_start.sh
deleted file mode 100644
index e058fad54..000000000
--- a/docker/ironic/ironic-conductor/extend_start.sh
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/bash
-
-sudo modprobe iscsi_tcp
diff --git a/docker/ironic/ironic-conductor/iscsi_tcp_sudoers b/docker/ironic/ironic-conductor/iscsi_tcp_sudoers
deleted file mode 100644
index 5d38ac691..000000000
--- a/docker/ironic/ironic-conductor/iscsi_tcp_sudoers
+++ /dev/null
@@ -1 +0,0 @@
-ironic ALL=(root) NOPASSWD: /usr/sbin/modprobe iscsi_tcp, /sbin/modprobe iscsi_tcp