Skip to content

DAOS-19452 test: Update soak yaml files to use old default - #18939

Open
shimizukko wants to merge 3 commits into
masterfrom
makito/DAOS-19452
Open

DAOS-19452 test: Update soak yaml files to use old default#18939
shimizukko wants to merge 3 commits into
masterfrom
makito/DAOS-19452

Conversation

@shimizukko

Copy link
Copy Markdown
Contributor

Use old default property values for pool.
properties: rd_fac:0,space_rb:0

(Container properties are already set.)

Skip-fault-injection-test: true
Skip-func-hw-test-large: false
Test-tag: test_soak_smoke

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

Use old default property values for pool.
properties: rd_fac:0,space_rb:0

(Container properties are already set.)

Skip-fault-injection-test: true
Skip-func-hw-test-large: false
Test-tag: test_soak_smoke
Signed-off-by: Makito Kano <makito.kano@hpe.com>
@github-actions

Copy link
Copy Markdown

Ticket title is 'Update functional tests to use new pool/container defaults'
Status is 'Open'
Labels: 'testp2'
https://daosio.atlassian.net/browse/DAOS-19452

@daltonbohning daltonbohning added the release-3.0.0 Targeted for release 3.0.0 label Aug 27, 2026
@shimizukko
shimizukko marked this pull request as ready for review August 28, 2026 03:29
@shimizukko
shimizukko requested review from a team as code owners August 28, 2026 03:29

@jamesanunez jamesanunez left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

PR looks good. I just have a couple of questions.

pool_query_timeout: 120
properties: rd_fac:0,space_rb:0

pool_reserved:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should 'properties: rd_fac:0,space_rb:0' be added here for pool_reserved?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. Thanks.

storage: auto

# pool_params - attributes of the pools to create; Currently only create one
pool_jobs:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should 'properties: rd_fac:0,space_rb:0' be added here for pool_jobs?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done

Skip-fault-injection-test: true
Skip-func-hw-test-large: false
Test-tag: test_soak_smoke
Signed-off-by: Makito Kano <makito.kano@hpe.com>
@shimizukko
shimizukko requested a review from jamesanunez August 29, 2026 04:08
Skip-fault-injection-test: true
Skip-func-hw-test-large: false
Test-tag: test_soak_smoke
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-3.0.0 Targeted for release 3.0.0

Development

Successfully merging this pull request may close these issues.

3 participants