FIXME: cobbler: setup.sh: Keep extra RPM repos. 93/13593/1
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Wed, 27 Apr 2016 23:48:43 +0000 (01:48 +0200)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Mon, 2 May 2016 14:51:11 +0000 (14:51 +0000)
cobbler-grub-aarch64 (arm64 specific loader) is currently
distributed and added to the cobbler docker container during
ISO build using EXTRA_RPM_REPOS envvar mechanism.

However, since this package installs files in a mounted location,
its contents are not persistent, requiring a test and eventual
forced reinstall during container start.

This, of course, requires the package to be available at that point,
which is not currently possible, as we remove the extra RPM repository
entry from yum config at the end of container build.

Hacky temporary workaround:
Keep extra RPM yum config after container is set up, only for cobbler.

TODO: This patch should be dropped once cobbler-grub-aarch64 RPM
package gets upstreamed.

Change-Id: I58e39df2671d79125c68a5ec994db962c103ce01
(cherry picked from commit 69fd98ae01038e2eb130d55f1f90719d5bc30c67)

patches/fuel-main/0004-FIXME-cobbler-setup.sh-Keep-extra-RPM-repos.patch [new file with mode: 0644]

diff --git a/patches/fuel-main/0004-FIXME-cobbler-setup.sh-Keep-extra-RPM-repos.patch b/patches/fuel-main/0004-FIXME-cobbler-setup.sh-Keep-extra-RPM-repos.patch
new file mode 100644 (file)
index 0000000..c16d05e
--- /dev/null
@@ -0,0 +1,62 @@
+From: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
+Date: Thu, 28 Apr 2016 01:41:45 +0200
+Subject: [PATCH] FIXME: cobbler: setup.sh: Keep extra RPM repos.
+
+cobbler-grub-aarch64 (arm64 specific loader) is currently
+distributed and added to the cobbler docker container during
+ISO build using EXTRA_RPM_REPOS envvar mechanism.
+
+However, since this package installs files in a mounted location,
+its contents are not persistent, requiring a test and eventual
+forced reinstall during container start.
+
+This, of course, requires the package to be available at that point,
+which is not currently possible, as we remove the extra RPM repository
+entry from yum config at the end of container build.
+
+Hacky temporary workaround:
+Keep extra RPM yum config after container is set up, only for cobbler.
+
+TODO: This patch should be dropped once cobbler-grub-aarch64 RPM
+package gets upstreamed.
+---
+ docker/cobbler/setup.sh | 26 +++++++++++++++-----------
+ 1 file changed, 15 insertions(+), 11 deletions(-)
+
+diff --git a/docker/cobbler/setup.sh b/docker/cobbler/setup.sh
+index 90627a8..1a0b0d2 100644
+--- a/docker/cobbler/setup.sh
++++ b/docker/cobbler/setup.sh
+@@ -68,17 +68,21 @@ puppet apply --debug --verbose --color false --detailed-exitcodes \
+   /etc/puppet/modules/nailgun/examples/cobbler-only.pp || [[ $? == 2 ]]
+-cat << EOF > /etc/yum.repos.d/nailgun.repo
+-[nailgun]
+-name=Nailgun Local Repo
+-baseurl=file:/var/www/nailgun/centos/x86_64
+-gpgcheck=0
+-
+-[mos]
+-name=MOS Local Repo
+-baseurl=file:/var/www/nailgun/mos-centos/x86_64
+-gpgcheck=0
+-EOF
++## FIXME(armband): Keep extra RPM repository in Cobbler container
++# Normally, extra RPM repos are discarded post-build, but we
++# need cobbler-grub-aarch64 during each container start.
++
++#  cat << EOF > /etc/yum.repos.d/nailgun.repo
++#  [nailgun]
++#  name=Nailgun Local Repo
++#  baseurl=file:/var/www/nailgun/centos/x86_64
++#  gpgcheck=0
++
++#  [mos]
++#  name=MOS Local Repo
++#  baseurl=file:/var/www/nailgun/mos-centos/x86_64
++#  gpgcheck=0
++#  EOF
+ yum clean all