Skip to content
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.

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

DSE extension for TestDefinition #316

Closed
wants to merge 4 commits into from
Closed

DSE extension for TestDefinition #316

wants to merge 4 commits into from

Conversation

amaslenn
Copy link
Contributor

Summary

PR introduces configuration for DSE parameters. All TestDefinitions get such support automatically.

Test TOML example:

name = "name"
description = "desc"
test_template_name = "template"

[dse.parameters]
"fdl_config" = ["/p1"]                                      # static list with a single element
"fdl.num_groups" = [1, 2]                                   # static list with two elements
# or "fdl.num_groups" = {"start": 1, "end": 2}              # range with default step
# or "fdl.num_groups" = {"start": 1, "end": 2, "step": 0.5} # range with explicit step

[cmd_args]

All dse.paramaters must be present in cmd_args, nested fields are also supported.

Test Plan

CI.

Additional Notes

dse field in TOML will likely get some more parameters for configuring DSE, something like this:

[dse]
agent = "agent name"
other = "value"
[dse.parameters]
# ...

Types for ranges and related fields are not verified yet, to be implemented later.

@amaslenn amaslenn closed this Dec 20, 2024
@amaslenn amaslenn deleted the am/dse-test branch December 20, 2024 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant