Re: [jenkins-ci PATCH v2 5/8] guests: Install EPEL repo on all CentOS guests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 11, 2019 at 10:44 AM Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote:
>
> On Wed, Dec 11, 2019 at 10:31 AM Andrea Bolognani <abologna@xxxxxxxxxx> wrote:
> >
> > On Tue, 2019-12-10 at 16:38 +0100, Fabiano Fidêncio wrote:
> > > +++ b/guests/playbooks/update/tasks/base.yml
> > > @@ -15,7 +15,6 @@
> > >      state: latest
> > >    when:
> > >      - os_name == 'CentOS'
> > > -    - os_version == '7'
> >
> > Welp, I just realized you need to apply the same change to the
> > _action_dockerfile() function in lcitool too... Up to you whether
> > to respin or fix Dockerfile generation in a follow-up patch.
>
> I will respin the series as it may need an extra patch between those
> this one and the PowerTools one.

Actually, I will not.

This is the fixup for this patch:
```
diff --git a/guests/lcitool b/guests/lcitool
index 803477e..059b789 100755
--- a/guests/lcitool
+++ b/guests/lcitool
@@ -748,7 +748,7 @@ class Application:
                         {package_manager} autoremove -y && \\
                         {package_manager} clean all -y
                 """).format(**varmap))
-            elif os_name == "CentOS" and os_version == "7":
+            elif os_name == "CentOS":
                 sys.stdout.write(textwrap.dedent("""
                     RUN {package_manager} update -y && \\
                         {package_manager} install -y epel-release && \\
```
Best Regards,
-- 
Fabiano Fidêncio


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux