Skip to content

DAOS-10535 tests: Remove daos-tests daos-*-tests-openmpi requirement #8980

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jolivier23 merged 13 commits into from
May 20, 2022
Merged

DAOS-10535 tests: Remove daos-tests daos-*-tests-openmpi requirement #8980

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ef646b2
DAOS-10535 tests: Remove daos-tests daos-*-tests-openmpi requirement
phender May 10, 2022
1777230
Fix typo.
phender May 10, 2022
bbcb78c
Fix other typo.
phender May 10, 2022
8f4f4f0
Adding daos-tests-internal package
phender May 12, 2022
184fe3e
Add missing edits.
phender May 12, 2022
bd78a7f
Adding pipeline-lib/pull/311
phender May 12, 2022
c50a637
Fixed debian changelog.
phender May 12, 2022
71517db
Fix daos.spec typo
phender May 12, 2022
58cc514
Adding .rpmignore changes
phender May 12, 2022
fefd73f
Add installing required packages for CI.
phender May 12, 2022
884aa5e
Fix typo.
phender May 12, 2022
d346e49
Fixing daos.spec
phender May 12, 2022
0c6ed1a
Merge branch 'master' into DAOS-10535_master
phender May 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 16 additions & 13 deletions Jenkinsfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@
// For master, this is just some wildly high number
next_version = "1000"

// Additional daos RPM packages required for functional tests
add_daos_pkgs = ",-client-tests-openmpi"

// Don't define this as a type or it loses it's global scope
target_branch = env.CHANGE_TARGET ? env.CHANGE_TARGET : env.BRANCH_NAME
def sanitized_JOB_NAME = JOB_NAME.toLowerCase().replaceAll('/', '-').replaceAll('%2f', '-')
Expand Down Expand Up @@ -701,7 +704,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -720,7 +723,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -739,7 +742,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -758,7 +761,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -777,7 +780,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -796,7 +799,7 @@ pipeline {
}
steps {
testRpm inst_repos: daosRepos(),
daos_pkg_version: daosPackagesVersion(next_version)
daos_pkg_version: daosPackagesVersion(next_version, add_daos_pkgs)
}
} // stage('Test CentOS 7 RPMs')
stage('Scan CentOS 7 RPMs') {
Expand All @@ -809,7 +812,7 @@ pipeline {
}
steps {
scanRpms inst_repos: daosRepos(),
daos_pkg_version: daosPackagesVersion(next_version)
daos_pkg_version: daosPackagesVersion(next_version, add_daos_pkgs)
}
post {
always {
Expand All @@ -827,7 +830,7 @@ pipeline {
}
steps {
scanRpms inst_repos: daosRepos(),
daos_pkg_version: daosPackagesVersion(next_version)
daos_pkg_version: daosPackagesVersion(next_version, add_daos_pkgs)
}
post {
always {
Expand All @@ -845,7 +848,7 @@ pipeline {
}
steps {
scanRpms inst_repos: daosRepos(),
daos_pkg_version: daosPackagesVersion(next_version)
daos_pkg_version: daosPackagesVersion(next_version, add_daos_pkgs)
}
post {
always {
Expand Down Expand Up @@ -911,7 +914,7 @@ pipeline {
}
steps {
storagePrepTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version)
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs)
}
} // stage('Test Storage Prep')
stage('Test Hardware') {
Expand All @@ -931,7 +934,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -951,7 +954,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand All @@ -971,7 +974,7 @@ pipeline {
}
steps {
functionalTest inst_repos: daosRepos(),
inst_rpms: functionalPackages(1, next_version),
inst_rpms: functionalPackages(1, next_version, add_daos_pkgs),
test_function: 'runTestFunctionalV2'
}
post {
Expand Down
6 changes: 6 additions & 0 deletions debian/changelog
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
daos (2.1.100-8) unstable; urgency=medium
[ Phillip Henderson ]
* Remove doas-client-tests-openmpi dependency from daos-tests

-- Phillip Henderson <[email protected]> Mon, 10 May 2022 10:53:00 -0400

daos (2.3.100-7) unstable; urgency=medium
[ Ashley Pittman ]
* Add dfuse unit-test binary to call from ftest.
Expand Down
2 changes: 1 addition & 1 deletion debian/control
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ Description: The Distributed Asynchronous Object Storage (DAOS) is an open-sourc
Package: daos-tests
Architecture: any
Multi-Arch: same
Depends: daos-client-tests-openmpi (= ${binary:Version})
Depends: daos-client-tests (= ${binary:Version})
Description: This is the package is a metapackage to install all of the test packages
.
This package contains tests
Expand Down
7 changes: 5 additions & 2 deletions utils/rpms/daos.spec
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@

Name: daos
Version: 2.3.100
Release: 7%{?relval}%{?dist}
Release: 8%{?relval}%{?dist}
Summary: DAOS Storage Engine

License: BSD-2-Clause-Patent
Expand Down Expand Up @@ -226,7 +226,7 @@ This is the package needed to run a DAOS client

%package tests
Summary: The entire DAOS test suite
Requires: %{name}-client-tests-openmpi%{?_isa} = %{version}-%{release}
Requires: %{name}-client-tests%{?_isa} = %{version}-%{release}
Copy link
Copy Markdown
Contributor

@brianjmurrell brianjmurrell May 12, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that you can do this all in one commit. This will cause testing on this branch to break until the pipeline-lib change lands. Unless you close landing on this branch after this until the pipeline-lib update lands.

Copy link
Copy Markdown
Contributor Author

@phender phender May 12, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm temporarily including installing the daos-client-tests-openmpi package through the Jenkins file until packaging in all branches generates a daos-test-internal package, at which point pipeline-lib can be updated to install it instead of daos-tests.


%description tests
This is the package is a metapackage to install all of the test packages
Expand Down Expand Up @@ -555,6 +555,9 @@ getent passwd daos_agent >/dev/null || useradd -s /sbin/nologin -r -g daos_agent
# No files in a shim package

%changelog
* Tue May 10 2022 Phillip Henderson <[email protected]> 2.1.102-8
- Remove doas-client-tests-openmpi dependency from daos-tests

* Fri May 6 2022 Ashley Pittman <[email protected]> 2.3.100-7
- Add dfuse unit-test binary to call from ftest.

Expand Down