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

config: rt-tests: reduce the duration of the test #2696

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

musamaanjum
Copy link
Contributor

The timeout of the job is set to 10 minutes. We should run the test for less than that. Reduce the total duration of the test to 540 seconds. Leave 60 seconds for logging in, setting the environment and triggering the test runner script.

We wanted to have total time of this job to be 10 minutes and run the test for as much duration as possible. So reducing to 540 seconds or 9 minutes is okay.

The timeout of the job is set to 10 minutes. We should run the test for
less than that. Reduce the total duration of the test to 540 seconds.
Leave 60 seconds for logging in, setting the environment and triggering
the test runner script.

We wanted to have total time of this job to be 10 minutes and run the
test for as much duration as possible. So reducing to 540 seconds or 9
minutes is okay.

Signed-off-by: Muhammad Usama Anjum <[email protected]>
@musamaanjum
Copy link
Contributor Author

@pawiecz Thanks for explaining here

@musamaanjum
Copy link
Contributor Author

@nuclearcat @pawiecz please can we merge this PR? Only the timeout is being reduced.

Copy link
Contributor

@pawiecz pawiecz left a comment

Choose a reason for hiding this comment

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

LGTM

@nuclearcat nuclearcat added this pull request to the merge queue Oct 2, 2024
Merged via the queue into kernelci:main with commit 34e668c Oct 2, 2024
4 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.

3 participants