Skip to content

SRE-3965 tests: Retag cb ftests and refine hw-medium stages - #18810

Open
ryon-jensen wants to merge 1 commit into
masterfrom
ryon-jensen/SRE-3965_test-tags
Open

SRE-3965 tests: Retag cb ftests and refine hw-medium stages#18810
ryon-jensen wants to merge 1 commit into
masterfrom
ryon-jensen/SRE-3965_test-tags

Conversation

@ryon-jensen

Copy link
Copy Markdown
Contributor

This change changes the strategy slightly between hw and cb test tags. All cb tests can run on hw stages as well. Filter out cb tags on HW stages where there is clear cb coverage.

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

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Errors are Unable to load ticket data
https://daosio.atlassian.net/browse/SRE-3965

This change changes the strategy slightly between hw and cb
test tags. All cb tests can run on hw stages as well. Filter
out cb tags on HW stages where there is clear cb coverage.

Signed-off-by: Ryon Jensen <ryon.jensen@hpe.com>
@ryon-jensen
ryon-jensen force-pushed the ryon-jensen/SRE-3965_test-tags branch from 8d60c05 to dc4ff47 Compare August 7, 2026 14:25
Comment thread Jenkinsfile
label: params.FUNCTIONAL_HARDWARE_MEDIUM_LABEL,
next_version: next_version(),
stage_tags: 'hw,medium,-provider',
stage_tags: 'hw,medium,-provider,-cb',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How would someone run a test tagged with both cb and hw in a HW stage?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

They would have to modify a line like this one. If we wanted to be able to control the behavior through commit pragmas, we could make the inclusion of ,-cb optional with something like:

Suggested change
stage_tags: 'hw,medium,-provider,-cb',
stage_tags: 'hw,medium,-provider' +
(cachedCommitPragma(pragma: 'Func-hw-exclude-cb-tests', def_val: 'true').toLowerCase() == 'true' ? ',-cb' : ''),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The reason for this change is we don't have Cluster Box stages in daily/weekly pipelines - including situations where we run the tests with different providers, so to avoid not running those tests they need to support being run in both CB and HW stages.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why don't we add Cluster Box stages to daily/weekly? I think having it be different in PRs vs daily/weekly means we might have failures in one but not the other and trying to fix them might be confusing

@phender phender Aug 7, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We would have to add 4-5 stages (note: Functional Cluster Box Medium MD on SSD already exists):

  • daily-testing

    • Functional Cluster Box Medium
    • Functional Cluster Box Medium Verbs Provider
    • Functional Cluster Box Medium Verbs Provider MD on SSD
    • Functional Cluster Box Medium UCX Provider
    • Functional Cluster Box Medium VMD (potentially if more cb tests added)

    I see an advantage to having a daily run of a cb tagged test on HW. One that can be used to compare if failures are seen on CB or HW and not vice-versa.

Comment thread Jenkinsfile
label: params.FUNCTIONAL_HARDWARE_MEDIUM_LABEL,
next_version: next_version(),
stage_tags: 'hw,medium,-provider',
stage_tags: 'hw,medium,-provider,-cb',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is also needed on the other Functional Hardware Medium * stages:

  • Functional Hardware Medium
  • Functional Hardware Medium VMD
  • Functional Hardware Medium UCX Provider

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.

3 participants