Skip to content

Commit 53461f7

Browse files
Merge pull request #949 from elfosardo/consolidate-copy-instructions
🌱 Consolidate ironic-config COPY instructions into single layer
2 parents ef25357 + f7dd814 commit 53461f7

File tree

1 file changed

+14
-12
lines changed

1 file changed

+14
-12
lines changed

Dockerfile

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,7 @@ ARG EXTRA_PKGS_LIST
118118
ARG PATCH_LIST
119119

120120
COPY ${PKGS_LIST} ${ARCH_PKGS_LIST} ${EXTRA_PKGS_LIST:-$PKGS_LIST} ${PATCH_LIST:-$PKGS_LIST} /tmp/
121-
COPY ironic-config/inspector.ipxe.j2 ironic-config/httpd-ironic-api.conf.j2 \
122-
ironic-config/ipxe_config.template ironic-config/dnsmasq.conf.j2 \
123-
/templates/
121+
COPY ironic-config/ /tmp/ironic-config/
124122
COPY prepare-image.sh patch-image.sh configure-nonroot.sh scripts/ /bin/
125123

126124
# Install Python packages from pre-built wheels (mounted from both wheel-builder stages)
@@ -134,17 +132,21 @@ RUN --mount=type=cache,target=/var/cache/dnf,sharing=locked \
134132
COPY --from=ironic-builder /tmp/ipxe/out/ /tftpboot/
135133
COPY --from=ironic-builder /tmp/uefi_esp*.img /templates/
136134

137-
COPY ironic-config/ironic.conf.j2 /etc/ironic/
138-
139-
# Custom httpd config, removes all but the bare minimum needed modules
140-
COPY ironic-config/httpd.conf.j2 /etc/httpd/conf/
141-
COPY ironic-config/httpd-modules.conf /etc/httpd/conf.modules.d/
142-
COPY ironic-config/apache2-vmedia.conf.j2 /templates/httpd-vmedia.conf.j2
143-
COPY ironic-config/apache2-ipxe.conf.j2 /templates/httpd-ipxe.conf.j2
144-
145-
# Database, and non-root user configuration
135+
# Database, ironic-config distribution, and non-root user configuration
136+
# Config files are placed here (after prepare-image.sh) because it removes
137+
# /etc/httpd/conf.modules.d/*.conf during package setup.
146138
RUN <<EORUN
147139
set -euxo pipefail
140+
cp /tmp/ironic-config/inspector.ipxe.j2 /tmp/ironic-config/httpd-ironic-api.conf.j2 \
141+
/tmp/ironic-config/ipxe_config.template /tmp/ironic-config/dnsmasq.conf.j2 \
142+
/templates/
143+
mkdir -p /etc/ironic
144+
cp /tmp/ironic-config/ironic.conf.j2 /etc/ironic/
145+
cp /tmp/ironic-config/httpd.conf.j2 /etc/httpd/conf/
146+
cp /tmp/ironic-config/httpd-modules.conf /etc/httpd/conf.modules.d/
147+
cp /tmp/ironic-config/apache2-vmedia.conf.j2 /templates/httpd-vmedia.conf.j2
148+
cp /tmp/ironic-config/apache2-ipxe.conf.j2 /templates/httpd-ipxe.conf.j2
149+
rm -rf /tmp/ironic-config
148150
mkdir -p /var/lib/ironic
149151
sqlite3 /var/lib/ironic/ironic.sqlite "pragma journal_mode=wal"
150152
microdnf remove -y sqlite

0 commit comments

Comments
 (0)