Skip to content

DAOS-18737 test: remove deployment/io_sys_admin (#17798)#17840

Merged
daltonbohning merged 1 commit intorelease/2.6from
dbohning/daos-18737-2.6
Mar 31, 2026
Merged

DAOS-18737 test: remove deployment/io_sys_admin (#17798)#17840
daltonbohning merged 1 commit intorelease/2.6from
dbohning/daos-18737-2.6

Conversation

@daltonbohning
Copy link
Copy Markdown
Contributor

Remove deployment/io_sys_admin because it is redundant coverage over existing test cases.

Add MPIIO, HDF5, HDF5-VOL to io/large_file_count since it was the only additional coverage provided by io_sys_admin.

Remove run_dm_activities_with_ior as it is no longer used.

Test-tag: LargeFileCount
Skip-unit-tests: true
Skip-fault-injection-test: true

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).

@daltonbohning daltonbohning self-assigned this Mar 30, 2026
@daltonbohning daltonbohning added the clean-cherry-pick Cherry-pick from another branch that did not require additional edits label Mar 30, 2026
@github-actions
Copy link
Copy Markdown

Ticket title is 'ftest: delete deployment/io_sys_admin'
Status is 'Awaiting backport'
Labels: 'request_for_2.6.5'
Job should run at elevated priority (1)
https://daosio.atlassian.net/browse/DAOS-18737

@github-actions github-actions Bot added the priority Ticket has high priority (automatically managed) label Mar 30, 2026
@daosbuild3
Copy link
Copy Markdown
Collaborator

Test stage Functional Hardware Large completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17840/1/testReport/

Remove deployment/io_sys_admin because it is redundant coverage over
existing test cases.

Add MPIIO, HDF5, HDF5-VOL to io/large_file_count since it was the only
additional coverage provided by io_sys_admin.

Remove run_dm_activities_with_ior as it is no longer used.

Test-tag: LargeFileCount
Skip-unit-tests: true
Skip-fault-injection-test: true

Signed-off-by: Dalton Bohning <dalton.bohning@hpe.com>
@daltonbohning daltonbohning force-pushed the dbohning/daos-18737-2.6 branch from 7dd10e4 to a882c99 Compare March 30, 2026 21:41
@daltonbohning daltonbohning added unclean-cherry-pick Indicates that a cherry-pick had merge conflicts that needed resolving. and removed clean-cherry-pick Cherry-pick from another branch that did not require additional edits labels Mar 30, 2026
@daltonbohning daltonbohning marked this pull request as ready for review March 31, 2026 14:23
@daltonbohning daltonbohning requested review from a team as code owners March 31, 2026 14:23
@daltonbohning daltonbohning added forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. approved-to-merge PR has received release branch merge approval labels Mar 31, 2026
Copy link
Copy Markdown
Contributor

@saurabhtandan saurabhtandan left a comment

Choose a reason for hiding this comment

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

LGTM

@daltonbohning daltonbohning requested a review from a team March 31, 2026 16:16
@daltonbohning daltonbohning merged commit ccea597 into release/2.6 Mar 31, 2026
34 checks passed
@daltonbohning daltonbohning deleted the dbohning/daos-18737-2.6 branch March 31, 2026 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-to-merge PR has received release branch merge approval forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. priority Ticket has high priority (automatically managed) unclean-cherry-pick Indicates that a cherry-pick had merge conflicts that needed resolving.

Development

Successfully merging this pull request may close these issues.

3 participants