Skip to content

test: set flex_start_max_run_duration_minutes in test runs#391

Merged
olivermeyer merged 1 commit intomainfrom
test/set-flex_start_max_run_duration_minutes
Jan 19, 2026
Merged

test: set flex_start_max_run_duration_minutes in test runs#391
olivermeyer merged 1 commit intomainfrom
test/set-flex_start_max_run_duration_minutes

Conversation

@olivermeyer
Copy link
Copy Markdown
Collaborator

@olivermeyer olivermeyer commented Jan 19, 2026

When runs are submitted with flex_start_max_run_duration_minutes=None, applications ignore the provisioning mode and fall back to ON_DEMAND. This PR excludes flex_start_max_run_duration_minutes from the serialized representation of the model.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Files with missing lines Coverage Δ
src/aignostics/platform/_sdk_metadata.py 100.00% <ø> (ø)

... and 8 files with indirect coverage changes

@olivermeyer olivermeyer force-pushed the test/set-flex_start_max_run_duration_minutes branch from 22b30bb to 25fbd0f Compare January 19, 2026 12:06
If flex_start_max_run_duration_minutes is not set, the key should
be excluded from the serialized representation of the model.
@olivermeyer olivermeyer force-pushed the test/set-flex_start_max_run_duration_minutes branch from 25fbd0f to 952ae33 Compare January 19, 2026 12:11
@sonarqubecloud
Copy link
Copy Markdown

@olivermeyer olivermeyer merged commit 420c548 into main Jan 19, 2026
38 of 41 checks passed
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