Skip to content

example/test: Reduce total system test run time#217

Merged
roosterfish merged 2 commits into
canonical:v3from
MggMuggins:test-sleeps
Aug 2, 2024
Merged

example/test: Reduce total system test run time#217
roosterfish merged 2 commits into
canonical:v3from
MggMuggins:test-sleeps

Conversation

@MggMuggins
Copy link
Copy Markdown
Contributor

Take advantage of configurable heartbeat intervals to reduce the CI duration by a little less than 2 minutes.

@MggMuggins MggMuggins requested a review from masnax July 31, 2024 19:55
Copy link
Copy Markdown
Contributor

@roosterfish roosterfish left a comment

Choose a reason for hiding this comment

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

Looks good to me, only the CLI command desc requires an update.

Comment thread example/cmd/microd/main.go Outdated
Useful in tests

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
Crank up the heartbeat interval to reduce the total test runtime

Signed-off-by: Wesley Hershberger <wesley.hershberger@canonical.com>
@MggMuggins MggMuggins changed the title Test sleeps example/test: Reduce total system test runtime Aug 1, 2024
@MggMuggins MggMuggins changed the title example/test: Reduce total system test runtime example/test: Reduce total system test run time Aug 1, 2024
@MggMuggins MggMuggins requested a review from roosterfish August 1, 2024 21:22
@roosterfish roosterfish merged commit 04c737d into canonical:v3 Aug 2, 2024
@MggMuggins MggMuggins deleted the test-sleeps branch August 2, 2024 13:59
masnax added a commit that referenced this pull request Aug 28, 2024
This also includes the commits from
#217.

The last commit is taken from
#246 which should be
merged first.
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.

2 participants