[libvirt PATCH 03/12] ci: integration: Adjust the check for CentOS Stream version

Erik Skultety posted 12 patches 1 year, 1 month ago
There is a newer version of this series
[libvirt PATCH 03/12] ci: integration: Adjust the check for CentOS Stream version
Posted by Erik Skultety 1 year, 1 month ago
All supported versions of Fedora and CentOS Stream 9 default to modular
setup, it's probably better if we cosmetically adjust the CentOS Stream
version check to make it explicit that monolithic daemon services ought
to be started only on Stream 8.

Signed-off-by: Erik Skultety <eskultet@redhat.com>
---
 ci/jobs.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ci/jobs.sh b/ci/jobs.sh
index 2bc6f86e45..adb798d127 100644
--- a/ci/jobs.sh
+++ b/ci/jobs.sh
@@ -89,7 +89,7 @@ run_integration() {
     sudo systemctl daemon-reexec # need to reexec systemd after changing config
 
     source /etc/os-release  # in order to query the vendor-provided variables
-    if test "$ID" = "centos" && test "$VERSION_ID" -lt 9
+    if test "$ID" = "centos" && test "$VERSION_ID" -eq 8
     then
         DAEMONS="libvirtd virtlockd virtlogd"
     else
-- 
2.41.0
Re: [libvirt PATCH 03/12] ci: integration: Adjust the check for CentOS Stream version
Posted by Daniel P. Berrangé 1 year, 1 month ago
On Mon, Sep 18, 2023 at 12:22:39PM +0200, Erik Skultety wrote:
> All supported versions of Fedora and CentOS Stream 9 default to modular
> setup, it's probably better if we cosmetically adjust the CentOS Stream
> version check to make it explicit that monolithic daemon services ought
> to be started only on Stream 8.
> 
> Signed-off-by: Erik Skultety <eskultet@redhat.com>
> ---
>  ci/jobs.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|