* [PATCH 1/1] ci: Remove CentOS 8
@ 2022-02-15 7:49 Petr Vorel
2022-03-22 18:25 ` Mimi Zohar
0 siblings, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2022-02-15 7:49 UTC (permalink / raw)
To: linux-integrity; +Cc: Petr Vorel, Mimi Zohar
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(-)
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
--
2.35.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 1/1] ci: Remove CentOS 8
2022-02-15 7:49 [PATCH 1/1] ci: Remove CentOS 8 Petr Vorel
@ 2022-03-22 18:25 ` Mimi Zohar
2022-03-22 19:30 ` Petr Vorel
0 siblings, 1 reply; 6+ messages in thread
From: Mimi Zohar @ 2022-03-22 18:25 UTC (permalink / raw)
To: Petr Vorel, linux-integrity; +Cc: Bruno E. O. Meneguele
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ci: Remove CentOS 8
2022-03-22 18:25 ` Mimi Zohar
@ 2022-03-22 19:30 ` Petr Vorel
2022-03-23 12:14 ` Mimi Zohar
0 siblings, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2022-03-22 19:30 UTC (permalink / raw)
To: Mimi Zohar; +Cc: linux-integrity, Bruno E. O. Meneguele
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ci: Remove CentOS 8
2022-03-22 19:30 ` Petr Vorel
@ 2022-03-23 12:14 ` Mimi Zohar
2022-03-23 12:55 ` Petr Vorel
2022-03-23 13:45 ` Petr Vorel
0 siblings, 2 replies; 6+ messages in thread
From: Mimi Zohar @ 2022-03-23 12:14 UTC (permalink / raw)
To: Petr Vorel; +Cc: linux-integrity, Bruno E. O. Meneguele
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ci: Remove CentOS 8
2022-03-23 12:14 ` Mimi Zohar
@ 2022-03-23 12:55 ` Petr Vorel
2022-03-23 13:45 ` Petr Vorel
1 sibling, 0 replies; 6+ messages in thread
From: Petr Vorel @ 2022-03-23 12:55 UTC (permalink / raw)
To: Mimi Zohar; +Cc: linux-integrity, Bruno E. O. Meneguele
> 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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] ci: Remove CentOS 8
2022-03-23 12:14 ` Mimi Zohar
2022-03-23 12:55 ` Petr Vorel
@ 2022-03-23 13:45 ` Petr Vorel
1 sibling, 0 replies; 6+ messages in thread
From: Petr Vorel @ 2022-03-23 13:45 UTC (permalink / raw)
To: Mimi Zohar; +Cc: linux-integrity, Bruno E. O. Meneguele
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
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-03-23 13:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-15 7:49 [PATCH 1/1] ci: Remove CentOS 8 Petr Vorel
2022-03-22 18:25 ` Mimi Zohar
2022-03-22 19:30 ` Petr Vorel
2022-03-23 12:14 ` Mimi Zohar
2022-03-23 12:55 ` Petr Vorel
2022-03-23 13:45 ` Petr Vorel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox