diff mbox series

[1/1] ci: Remove CentOS 8

Message ID 20220215074958.6942-1-pvorel@suse.cz (mailing list archive)
State New, archived
Headers show
Series [1/1] ci: Remove CentOS 8 | expand

Commit Message

Petr Vorel Feb. 15, 2022, 7:49 a.m. UTC
It EOL in 12/2021 and CI is failing due removed repo:

CentOS Linux 8 - AppStream                      232  B/s |  38  B     00:00
Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Hi,

the error:
https://github.com/pevik/ima-evm-utils/runs/5196409185?check_suite_focus=true

Mimi, could you please test Travis CI?
I tested only GitHub Actions:
https://github.com/pevik/ima-evm-utils/actions/runs/1845623174

Kind regards,
Petr

 .github/workflows/ci.yml | 5 -----
 .travis.yml              | 4 ----
 2 files changed, 9 deletions(-)

Comments

Mimi Zohar March 22, 2022, 6:25 p.m. UTC | #1
Hi Petr,

On Tue, 2022-02-15 at 08:49 +0100, Petr Vorel wrote:
> It EOL in 12/2021 and CI is failing due removed repo:
> 
> CentOS Linux 8 - AppStream                      232  B/s |  38  B     00:00
> Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
> 
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> Hi,
> 
> the error:
> https://github.com/pevik/ima-evm-utils/runs/5196409185?check_suite_focus=true
> 
> Mimi, could you please test Travis CI?
> I tested only GitHub Actions:
> https://github.com/pevik/ima-evm-utils/actions/runs/1845623174

Removing "centos:latest" works, but is there another option?  Perhaps
defining a different repo - "REPO=quay.io/centos/"?

thanks,

Mimi
Petr Vorel March 22, 2022, 7:30 p.m. UTC | #2
Hi Mimi,

> Hi Petr,

> On Tue, 2022-02-15 at 08:49 +0100, Petr Vorel wrote:
> > It EOL in 12/2021 and CI is failing due removed repo:

> > CentOS Linux 8 - AppStream                      232  B/s |  38  B     00:00
> > Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist

> > Signed-off-by: Petr Vorel <pvorel@suse.cz>
> > ---
> > Hi,

> > the error:
> > https://github.com/pevik/ima-evm-utils/runs/5196409185?check_suite_focus=true

> > Mimi, could you please test Travis CI?
> > I tested only GitHub Actions:
> > https://github.com/pevik/ima-evm-utils/actions/runs/1845623174

> Removing "centos:latest" works, but is there another option?  Perhaps
> defining a different repo - "REPO=quay.io/centos/"?
This fix would be for Travis, which I'm not able to fix, but not for GitHub
Actions :(. I'm not aware how to specify alternative repository for it.

Kind regards,
Petr

> thanks,

> Mimi
Mimi Zohar March 23, 2022, 12:14 p.m. UTC | #3
Hi Petr,

> > Removing "centos:latest" works, but is there another option?  Perhaps
> > defining a different repo - "REPO=quay.io/centos/"?
> This fix would be for Travis, which I'm not able to fix, but not for GitHub
> Actions :(. I'm not aware how to specify alternative repository for it.

FYI, "centos:stream9" results in crypto deprecated warnings and results
in /dev/null errors.  Instead of removing the "centos:latest" entry in
travis, I'll replace it with "centos:stream8" for now.

The "centos:stream9" -devel rpm packages are in the CRB repo.  I
manually set the CRB repo to enabled using sed.  Is there a better way
of enabling CRB?

Do you prefer to limit this patch to just removing the "centos:latest"
entry in ci.yml and reposting it, or should I trim it?

thanks,

Mimi
Petr Vorel March 23, 2022, 12:55 p.m. UTC | #4
> Hi Petr,

> > > Removing "centos:latest" works, but is there another option?  Perhaps
> > > defining a different repo - "REPO=quay.io/centos/"?
> > This fix would be for Travis, which I'm not able to fix, but not for GitHub
> > Actions :(. I'm not aware how to specify alternative repository for it.

> FYI, "centos:stream9" results in crypto deprecated warnings and results
> in /dev/null errors.  Instead of removing the "centos:latest" entry in
> travis, I'll replace it with "centos:stream8" for now.

> The "centos:stream9" -devel rpm packages are in the CRB repo.  I
> manually set the CRB repo to enabled using sed.  Is there a better way
> of enabling CRB?

> Do you prefer to limit this patch to just removing the "centos:latest"
> entry in ci.yml and reposting it, or should I trim it?
I'll check if centos:stream8 works on GitHub Actions.

Kind regards,
Petr

> thanks,

> Mimi
Petr Vorel March 23, 2022, 1:45 p.m. UTC | #5
Hi Mimi,

> Hi Petr,

> > > Removing "centos:latest" works, but is there another option?  Perhaps
> > > defining a different repo - "REPO=quay.io/centos/"?
> > This fix would be for Travis, which I'm not able to fix, but not for GitHub
> > Actions :(. I'm not aware how to specify alternative repository for it.

> FYI, "centos:stream9" results in crypto deprecated warnings and results
> in /dev/null errors.  Instead of removing the "centos:latest" entry in
> travis, I'll replace it with "centos:stream8" for now.
centos:stream8 does not work on GitHub Actions:

/usr/bin/docker pull centos:stream8
  Error response from daemon: manifest for centos:stream8 not found: manifest unknown: manifest unknown
  Warning: Docker pull failed with exit code 1, back off 8.858 seconds before retry.
  /usr/bin/docker pull centos:stream8
  Error response from daemon: manifest for centos:stream8 not found: manifest unknown: manifest unknown
  Warning: Docker pull failed with exit code 1, back off 8.279 seconds before retry.
  /usr/bin/docker pull centos:stream8
  Error response from daemon: manifest for centos:stream8 not found: manifest unknown: manifest unknown
  Error: Docker pull failed with exit code 1

> The "centos:stream9" -devel rpm packages are in the CRB repo.  I
> manually set the CRB repo to enabled using sed.  Is there a better way
> of enabling CRB?

> Do you prefer to limit this patch to just removing the "centos:latest"
> entry in ci.yml and reposting it, or should I trim it?
In this case I'll send another version where I remove it just for GitHub
Actions. It's just a bit unfortunate, that it will have different OS to test
from Travis.

Kind regards,
Petr

> thanks,

> Mimi
diff mbox series

Patch

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 4316ede..5e4498e 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -80,11 +80,6 @@  jobs:
               CC: gcc
               TSS: tpm2-tss
 
-          - container: "centos:latest"
-            env:
-              CC: gcc
-              TSS: tpm2-tss
-
           - container: "debian:testing"
             env:
               CC: clang
diff --git a/.travis.yml b/.travis.yml
index bdf78a1..1429e77 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -55,10 +55,6 @@  matrix:
           env: DISTRO=centos:7 TSS=tpm2-tss
           compiler: gcc
 
-        - os: linux
-          env: DISTRO=centos:latest TSS=tpm2-tss
-          compiler: clang
-
         - os: linux
           env: DISTRO=debian:testing TSS=tpm2-tss
           compiler: clang