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

fix: TestShell timeouts #128

Merged
merged 2 commits into from
Dec 5, 2024
Merged

fix: TestShell timeouts #128

merged 2 commits into from
Dec 5, 2024

Conversation

CarlosNihelton
Copy link
Collaborator

It's been a while that GoWSL CI has been shown issues with TestShell.

After some investigation I figured that first boot of the images used for testing is taking longer than what the test cases expect. Thus the timeout.
From GoWSL's point-of-view it'd be better to use lighter images, without systemd for example. But since GoWSL's main use case is with Ubuntu, let's keep the images as is and just give the tests more time to run.

It's been a while that GoWSL CI has been shown issues with TestShell.

After some investigation I figured that first boot of the images used for testing is taking longer than what the test cases expect.
Thus the timeout.
From GoWSL's point-of-view it'd be better to use lighter images, without systemd for example.
But since GoWSL's main use case is with Ubuntu, let's keep the images as is and just give the tests more time to run.
@CarlosNihelton CarlosNihelton self-assigned this Dec 5, 2024
@CarlosNihelton CarlosNihelton marked this pull request as ready for review December 5, 2024 20:03
@CarlosNihelton CarlosNihelton requested a review from a team as a code owner December 5, 2024 20:03
shell_test.go Show resolved Hide resolved
Copy link
Collaborator

@ashuntu ashuntu left a comment

Choose a reason for hiding this comment

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

Otherwise, this seems reasonable :)

@CarlosNihelton CarlosNihelton merged commit e95f720 into main Dec 5, 2024
6 checks passed
@CarlosNihelton CarlosNihelton deleted the fix-shell-timeouts branch December 5, 2024 21:08
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