Skip to content

SRE-3534 ci: adapt CI to use el9.7 for Build and UT#17632

Open
grom72 wants to merge 62 commits intomasterfrom
grom72/SRE-3534-final-tuning
Open

SRE-3534 ci: adapt CI to use el9.7 for Build and UT#17632
grom72 wants to merge 62 commits intomasterfrom
grom72/SRE-3534-final-tuning

Conversation

@grom72
Copy link
Contributor

@grom72 grom72 commented Mar 3, 2026

Requires:

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

grom72 and others added 30 commits January 14, 2026 12:56
Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true

Allow-unstable-test: true

Skip-func-hw-test-medium: false

Skip-func-hw-test-medium-md-on-ssd: false

Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Priority: 2

Skip-unit-tests:true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Cancel-prev-build: false
Priority: 2

Skip-unit-tests:true

Allow-unstable-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: false
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-func-vm: true

Skip-func-hw-test-medium: false
Skip-func-hw-test-medium-md-on-ssd: false
Skip-func-hw-test-medium-vmd: false
Skip-func-hw-test-medium-verbs-provider: false
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: false
Skip-func-hw-test-medium-ucx-provider: false
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true

Allow-unstable-test: true

Skip-func-vm: true

Skip-func-hw-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Skip-func-vm: true
Skip-func-test-vm-valgrind: true
Skip-func-test-el8: true
Skip-func-test-el9: true
Skip-func-test-leap15: true
Skip-fault-injection-test: true
Skip-test-el-9.6-rpms: false
Skip-test-leap-15-rpms: true

Skip-func-hw-test-medium: false
Skip-func-hw-test-medium-md-on-ssd: false
Skip-func-hw-test-medium-vmd: false
Skip-func-hw-test-medium-verbs-provider: false
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: false
Skip-func-hw-test-medium-ucx-provider: false
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: false
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-func-vm: true
Skip-func-test-vm-valgrind: true
Skip-func-test-el8: true
Skip-func-test-el9: true
Skip-func-test-leap15: true
Skip-fault-injection-test: true
Skip-test-el-9.6-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test: false
Skip-func-hw-test-medium: false
Skip-func-hw-test-medium-md-on-ssd: false
Skip-func-hw-test-medium-vmd: false
Skip-func-hw-test-medium-verbs-provider: false
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: false
Skip-func-hw-test-medium-ucx-provider: false
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true

Skip-func-test-vm-valgrind: false
Skip-func-test-el8: false
Skip-func-test-el9: false

Skip-func-hw-test-medium: false
Skip-func-hw-test-medium-md-on-ssd: false
Skip-func-hw-test-medium-vmd: false
Skip-func-hw-test-medium-verbs-provider: false
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: false
Skip-func-hw-test-medium-ucx-provider: false
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false
Skip-func-test: true
Skip-func-test-vm: true
Skip-func-test-hw: true
Skip-nlt: true
Skip-coverity-test: true
Skip-fault-injection-test: true
Skip-el8: true

Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true
Skip-build-leap15-icc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-func-test-vm-valgrind: false
Skip-func-test-el8: true
Skip-func-test-el9: true
Skip-func-test-leap15: true
Skip-fault-injection-test: true
Skip-test-el-8.6-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test: true
Skip-func-hw-test-medium: true
Skip-func-hw-test-medium-md-on-ssd: true
Skip-func-hw-test-medium-vmd: true
Skip-func-hw-test-medium-verbs-provider: true
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: true
Skip-func-hw-test-medium-ucx-provider: true
Skip-func-hw-test-large: true
Skip-func-hw-test-large-md-on-ssd: true
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Skip-build-leap15-gcc: true
Skip-build-leap15-icc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-func-test-vm-valgrind: false
Skip-func-test-leap15: true
Skip-fault-injection-test: true
Skip-test-el-8.6-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test: true
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-func-vm: true
Skip-func-test-vm-valgrind: true
Skip-func-test-el8: true
Skip-func-test-el9: true
Skip-func-test-leap15: true
Skip-fault-injection-test: true
Skip-test-el-9.6-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test-medium: true
Skip-func-hw-test-medium-md-on-ssd: true
Skip-func-hw-test-medium-vmd: true
Skip-func-hw-test-medium-verbs-provider: true
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: true
Skip-func-hw-test-medium-ucx-provider: true
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-func-vm: true
Skip-func-test-vm-valgrind: true
Skip-func-test-el8: true
Skip-func-test-el9: true
Skip-func-test-leap15: true
Skip-fault-injection-test: true
Skip-test-el-9.6-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test-medium: true
Skip-func-hw-test-medium-md-on-ssd: true
Skip-func-hw-test-medium-vmd: true
Skip-func-hw-test-medium-verbs-provider: true
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: true
Skip-func-hw-test-medium-ucx-provider: true
Skip-func-hw-test-large: false
Skip-func-hw-test-large-md-on-ssd: false
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Use image_version parameter to provide OS version

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true
This reverts commit d4871a1.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true
Skip-func-hw-test-medium-verbs-provider: false
Skip-func-hw-test-medium-verbs-provider-md-on-ssd: false

Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true
To test that default EL9 FT works.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2
Test-tag: test_self_test

Skip-python-bandit: true

Skip-build-leap15-gcc: true
Skip-build-leap15-icc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-fault-injection-test: true
Skip-test-el-8.6-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test: true
@daosbuild3
Copy link
Collaborator

Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17632/1/execution/node/1353/log

grom72 added 2 commits March 3, 2026 21:39
This reverts commit f106467.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
CI_MORE_FUNCTIONAL_PR_TESTS is no longer needed to trigger FT on
el8,leap15.
Only CI_FUNCTIONAL_*_TEST parameters are used to controll which tests
should be run.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Skip-func-hw-test: true
janekmi
janekmi previously approved these changes Mar 4, 2026
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Skip-python-bandit: true

Skip-build-leap15-gcc: true
Skip-build-leap15-icc: true

Skip-unit-tests:true
Skip-unit-test: true
Skip-NLT: true
Skip-unit-test-memcheck: true

Skip-func-test-el8: true
Skip-func-test-el9: true
Skip-test-el9-rpms: true
Skip-test-leap-15-rpms: true

Skip-func-hw-test: true
@grom72 grom72 requested a review from janekmi March 4, 2026 12:38
@grom72 grom72 marked this pull request as ready for review March 4, 2026 12:43
@grom72 grom72 requested a review from a team as a code owner March 4, 2026 12:43
osalyk
osalyk previously approved these changes Mar 4, 2026
janekmi
janekmi previously approved these changes Mar 4, 2026
@grom72
Copy link
Contributor Author

grom72 commented Mar 4, 2026

Final validation results:
https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-17632/8

New warnings in Fault Injection stage are triggered by fault injection operation.
6 of them are because of misinterpretation of Logging allocation failure text in fault injection log line.
Similar situation in NLT tests.

New "warning" in NLT -> srv_container.c: has already reported by: https://daosio.atlassian.net/browse/DAOS-18631

src/container/srv_container.c:2628 cont_open() a3e7e074/98b8d535/018a3529: failed to open the container with flags 82, capas 1/0: DER_NO_PERM(-1001): 'Operation not permitted'

@grom72 grom72 requested a review from a team March 4, 2026 17:11
@daltonbohning
Copy link
Contributor

Why have the point release in the fault injection stage but not Functional stage?

  • Fault injection testing on EL 9.7
  • Functional on EL 9

Also, the build stages no longer specify the point release, but the functional stages do? E.g.

  • Build on Leap 15
  • Functional on Leap 15.6

The reason I'm being strict here is because changing the merge requirements to reflect the new stage names is a pain because it requires everyone to rebase and/or force landings each time we change required stage names.

Move the point release to the image_version parameter.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2

Allow-unstable-test: true

Skip-func-test-el8: false
Skip-func-test-leap15: false
Skip-test-el-9-rpms: false
Skip-test-leap-15-rpms: false

Skip-func-hw-test: true
@grom72 grom72 dismissed stale reviews from janekmi and osalyk via dd00593 March 4, 2026 23:22
@grom72 grom72 requested review from janekmi and osalyk March 4, 2026 23:22
@grom72
Copy link
Contributor Author

grom72 commented Mar 4, 2026

Why have the point release in the fault injection stage but not Functional stage?

  • Fault injection testing on EL 9.7
  • Functional on EL 9

Also, the build stages no longer specify the point release, but the functional stages do? E.g.

  • Build on Leap 15
  • Functional on Leap 15.6

The reason I'm being strict here is because changing the merge requirements to reflect the new stage names is a pain because it requires everyone to rebase and/or force landings each time we change required stage names.

@daltonbohning I have removed all point release from stage names in dd00593.
Please confirm that they are proper now.

@daosbuild3
Copy link
Collaborator

@grom72 grom72 removed the request for review from a team March 5, 2026 11:27
@grom72
Copy link
Contributor Author

grom72 commented Mar 5, 2026

@grom72 grom72 requested a review from a team March 5, 2026 13:41
stage('Test RPMs on EL 9.6') {
when {
beforeAgent true
expression { params.CI_TEST_EL8_RPMs && !skipStage() }
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are we removing params.CI_TEST_EL8_RPMs? We are keeping params.CI_TEST_LEAP15_RPMs on line 1082. Shouldn't this be the following?
expression { params.CI_TEST_EL_RPMs && !skipStage() }

defaultValue: true,
description: 'Run the Test RPMs on EL 8 test stage')
description: 'Run the Test RPMs on EL stage')
booleanParam(name: 'CI_TEST_LEAP15_RPMs',
Copy link
Contributor

Choose a reason for hiding this comment

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

Based upon the above change, should this be?

Suggested change
booleanParam(name: 'CI_TEST_LEAP15_RPMs',
booleanParam(name: 'CI_TEST_LEAP_RPMs',

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants