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.

Sign up for GitHub

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

Jump to bottom

Make test timeout "smart" #21

Closed
k3rn31 opened this issue Jul 14, 2022 · 0 comments · Fixed by #45
Closed

Make test timeout "smart" #21

k3rn31 opened this issue Jul 14, 2022 · 0 comments · Fixed by #45
Assignees
@k3rn31
Labels
c/enhancement Category: An issue or PR related to sn enhancement
Milestone
v0.1.0

Comments

@k3rn31
Copy link
Member

k3rn31 commented Jul 14, 2022 •
edited
Loading

The timeout of the test command is currently fixed. This should be determined dynamically.
One approach could be determining how much time it takes to run the first run for the coverage and use that as a base for the timeout of all the subsequent test runs.
@k3rn31 k3rn31 added the c/feature Category: An issue or PR related to a new feature label Jul 14, 2022
@k3rn31 k3rn31 added this to the v0.1.0 milestone Jul 14, 2022
@k3rn31 k3rn31 added c/enhancement Category: An issue or PR related to sn enhancement c/feature Category: An issue or PR related to a new feature and removed c/feature Category: An issue or PR related to a new feature c/enhancement Category: An issue or PR related to sn enhancement labels Jul 14, 2022
@k3rn31 k3rn31 changed the title Make test timeout either configurable or "smart" Make test timeout "smart" Jul 20, 2022
@k3rn31 k3rn31 self-assigned this Jul 22, 2022
@k3rn31 k3rn31 added c/enhancement Category: An issue or PR related to sn enhancement and removed c/feature Category: An issue or PR related to a new feature labels Jul 22, 2022
k3rn31 added a commit that referenced this issue Jul 22, 2022
@k3rn31
Make test timeout dynamic (#21)
ab8a4dc 
Signed-off-by: Davide Petilli <[email protected]>
@k3rn31 k3rn31 linked a pull request Jul 22, 2022 that will close this issue
Make test timeout dynamic (#21) #45
Merged
k3rn31 added a commit that referenced this issue Jul 22, 2022
@k3rn31
Make test timeout dynamic (#21)
957485d 
Signed-off-by: Davide Petilli <[email protected]>
k3rn31 added a commit that referenced this issue Jul 23, 2022
@k3rn31
Make test timeout dynamic (#21)
1b82ffe 
Signed-off-by: Davide Petilli <[email protected]>
k3rn31 added a commit that referenced this issue Jul 23, 2022
@k3rn31
Make test timeout dynamic (#21)
37a5f68 
Signed-off-by: Davide Petilli <[email protected]>
k3rn31 added a commit that referenced this issue Jul 23, 2022
@k3rn31
Make test timeout dynamic (#21)
32435bb 
Signed-off-by: Davide Petilli <[email protected]>
k3rn31 added a commit that referenced this issue Jul 23, 2022
@k3rn31
Make test timeout dynamic (#21)
f9aa795 
Signed-off-by: Davide Petilli <[email protected]>
k3rn31 added a commit that referenced this issue Jul 23, 2022
@k3rn31
Make test timeout dynamic (#21)
7251898 
Signed-off-by: Davide Petilli <[email protected]>
giose86 added a commit that referenced this issue Jul 23, 2022
@giose86
Merge pull request #45 from k3rn31/21-make-test-timeout-smart
fb1b51c 
Make test timeout dynamic (#21)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@k3rn31 k3rn31

Labels
c/enhancement Category: An issue or PR related to sn enhancement
Projects
None yet
Milestone
v0.1.0
Development

Successfully merging a pull request may close this issue.

Make test timeout dynamic (#21) go-gremlins/gremlins
1 participant
@k3rn31