Skip to content

Commit

Permalink
DAOS-16500 build: Move to Leap 15.6 (#15561) (#15688)
Browse files Browse the repository at this point in the history
Test with Leap 15.6 instead of Leap 15.5.

To support building Leap 15.5 DAOS RPMs and testing them on Leap 15.6
the Functional on Leap 15.6 stage needs to explicitly specify the Leap
15.6 image for node provisioning.

Combination of #15561, #15684

Signed-off-by: Phil Henderson <[email protected]>
Signed-off-by: Dalton Bohning <[email protected]>
  • Loading branch information
daltonbohning authored Jan 7, 2025
1 parent 776f123 commit a79e848
Show file tree
Hide file tree
Showing 6 changed files with 33 additions and 12 deletions.
20 changes: 11 additions & 9 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
/* groovylint-disable DuplicateMapLiteral, DuplicateNumberLiteral */
/* groovylint-disable DuplicateStringLiteral, NestedBlockDepth, VariableName */
/* Copyright 2019-2024 Intel Corporation
* Copyright 2025 Hewlett Packard Enterprise Development LP
* All rights reserved.
*
* This file is part of the DAOS Project. It is subject to the license terms
Expand Down Expand Up @@ -967,7 +968,7 @@ pipeline {
}
}
} // stage('Functional on EL 9')
stage('Functional on Leap 15.5') {
stage('Functional on Leap 15.6') {
when {
beforeAgent true
expression { !skipStage() }
Expand All @@ -980,15 +981,16 @@ pipeline {
functionalTest(
inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version, 'tests-internal'),
test_function: 'runTestFunctionalV2'))
test_function: 'runTestFunctionalV2',
image_version: 'leap15.6'))
}
post {
always {
functionalTestPostV2()
job_status_update()
}
} // post
} // stage('Functional on Leap 15.5')
} // stage('Functional on Leap 15.6')
stage('Functional on Ubuntu 20.04') {
when {
beforeAgent true
Expand Down Expand Up @@ -1062,7 +1064,7 @@ pipeline {
job_status_update()
}
}
} // stage('Fault inection testing on EL 8.8')
} // stage('Fault injection testing on EL 8.8')
stage('Test RPMs on EL 8.6') {
when {
beforeAgent true
Expand All @@ -1082,8 +1084,8 @@ pipeline {
rpm_test_post(env.STAGE_NAME, env.NODELIST)
}
}
} // stage('Test CentOS 7 RPMs')
stage('Test RPMs on Leap 15.4') {
} // stage('Test RPMs on EL 8.6')
stage('Test RPMs on Leap 15.5') {
when {
beforeAgent true
expression { ! skipStage() }
Expand All @@ -1097,8 +1099,8 @@ pipeline {
* additionally for this use-case, can't override
ftest_arg with this :-(
script {
'Test RPMs on Leap 15.4': getFunctionalTestStage(
name: 'Test RPMs on Leap 15.4',
'Test RPMs on Leap 15.5': getFunctionalTestStage(
name: 'Test RPMs on Leap 15.5',
pragma_suffix: '',
label: params.CI_UNIT_VM1_LABEL,
next_version: next_version,
Expand Down Expand Up @@ -1134,7 +1136,7 @@ pipeline {
rpm_test_post(env.STAGE_NAME, env.NODELIST)
}
}
} // stage('Test Leap 15 RPMs')
} // stage('Test RPMs on Leap 15.5')
} // parallel
} // stage('Test')
stage('Test Storage Prep on EL 8.8') {
Expand Down
4 changes: 4 additions & 0 deletions ci/parse_ci_envs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ if [ -n "${STAGE_NAME:?}" ]; then
: "${TARGET:=centos9}"
: "${REPO_SPEC:=el-9}"
;;
*Leap\ 15.6*|*leap15.6*|*opensuse15.6*|*sles15.6*)
: "${CHROOT_NAME:=opensuse-leap-15.5-x86_64}"
: "${TARGET:=leap15.6}"
;;
*Leap\ 15.5*|*leap15.5*|*opensuse15.5*|*sles15.5*)
: "${CHROOT_NAME:=opensuse-leap-15.5-x86_64}"
: "${TARGET:=leap15.5}"
Expand Down
2 changes: 1 addition & 1 deletion utils/rpms/daos.rpmlintrc
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,6 @@ addFilter("daos-client-tests\.x86_64: E: devel-dependency protobuf-c-devel")

# a functional test builds daos from source, so it needs the various *-devel packages for daos' build dependencies.
addFilter("daos-client-tests\.x86_64: E: devel-dependency capstone-devel")
addFilter("daos-client-tests\.x86_64: E: explicit-lib-dependency lib(capstone|ndctl)-devel")
addFilter("daos-client-tests\.x86_64: E: explicit-lib-dependency lib(capstone|ndctl|protobuf-c)-devel")
addFilter("daos-client-tests\.x86_64: E: devel-dependency libcapstone-devel")
addFilter("daos-client-tests\.x86_64: E: devel-dependency fuse3-devel")
9 changes: 8 additions & 1 deletion utils/rpms/daos.spec
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

Name: daos
Version: 2.6.2
Release: 4%{?relval}%{?dist}
Release: 5%{?relval}%{?dist}
Summary: DAOS Storage Engine

License: BSD-2-Clause-Patent
Expand Down Expand Up @@ -203,7 +203,11 @@ Summary: The DAOS test suite
Requires: %{name}-client%{?_isa} = %{version}-%{release}
Requires: %{name}-admin%{?_isa} = %{version}-%{release}
Requires: %{name}-devel%{?_isa} = %{version}-%{release}
%if (0%{?suse_version} >= 1500)
Requires: libprotobuf-c-devel
%else
Requires: protobuf-c-devel
%endif
Requires: fio
Requires: git
Requires: dbench
Expand Down Expand Up @@ -590,6 +594,9 @@ getent passwd daos_agent >/dev/null || useradd -s /sbin/nologin -r -g daos_agent
# No files in a shim package

%changelog
* Mon Jan 06 2025 Phillip Henderson <[email protected]> 2.6.2-5
- Fix protobuf-c requiremnent for daos-client-tests on Leap.

* Mon Dec 16 2024 Brian J. Murrell <[email protected]> - 2.6.2-4
- Drop BRs for UCX as they were obsoleted as of e01970d

Expand Down
8 changes: 8 additions & 0 deletions utils/rpms/packaging/Makefile_distro_vars.mk
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,14 @@ DISTRO_VERSION ?= $(VERSION_ID)
ORIG_TARGET_VER := 15.5
SED_EXPR := 1p
endif
ifeq ($(CHROOT_NAME),opensuse-leap-15.6-x86_64)
VERSION_ID := 15.6
DISTRO_ID := sl15.6
DISTRO_BASE := LEAP_15
DISTRO_VERSION ?= $(VERSION_ID)
ORIG_TARGET_VER := 15.6
SED_EXPR := 1p
endif
endif
ifeq ($(ID),centos)
ID = el
Expand Down
2 changes: 1 addition & 1 deletion utils/scripts/helpers/repo-helper-leap15.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ set -uex
# for custom packages if present.

: "${REPO_FILE_URL:=}"
: "${BASE_DISTRO:=opensuse/leap:15.5}"
: "${BASE_DISTRO:=opensuse/leap:15.6}"
: "${JENKINS_URL:=}"
: "${REPOS:=}"

Expand Down

0 comments on commit a79e848

Please sign in to comment.