Skip to content

feat: Double wait time every try and decrease tries#37

Open
locnnil wants to merge 2 commits into
canonical:mainfrom
locnnil:wait-exponentially
Open

feat: Double wait time every try and decrease tries#37
locnnil wants to merge 2 commits into
canonical:mainfrom
locnnil:wait-exponentially

Conversation

@locnnil

@locnnil locnnil commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

Summary

The current operation isn't being efficient. It basically fails if it takes more than 10s for the logs to show up. Which is too little time.

My points are:

  • We cannot establish a rule of thumb for the average time it takes because each setup is different.
  • With this change, the maximum wait time would be 31s, with the possible wait times being: 1s, 3s, 7s, 15s, 31s.
  • It may take longer to try to run the test multiple times rather than waiting properly from the first time.
  • The current operation is noisy and will become even more noisy if we just increase the tries.

The drawbacks that this adds is:

  • The test might take longer.

Signed-off-by: Lincoln Wallace <lincoln.wallace@canonical.com>
@locnnil locnnil requested a review from jpm-canonical July 14, 2025 16:37
@locnnil locnnil changed the title feat: Increase wait time exponentially every try and decrease tries feat: Double wait time every try and decrease tries Jul 15, 2025
Signed-off-by: Lincoln Wallace <lincoln.wallace@canonical.com>
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