Skip to content

test: fix utop tests #11771

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

test: fix utop tests #11771

wants to merge 1 commit into from

Conversation

rgrinberg
Copy link
Member

Signed-off-by: Rudi Grinberg [email protected]

@rgrinberg rgrinberg force-pushed the ps/rr/test__fix_utop_tests branch 2 times, most recently from b9313a5 to abd4a55 Compare May 7, 2025 22:16
@Leonidas-from-XIV Leonidas-from-XIV force-pushed the ps/rr/test__fix_utop_tests branch from abd4a55 to 26574e6 Compare May 9, 2025 08:41
@Leonidas-from-XIV
Copy link
Collaborator

I was worried that the test never terminates on Linux because so far it has been stuck for hours running the Github Action, but it works just fine on my Linux machine.

@Alizter
Copy link
Collaborator

Alizter commented May 12, 2025

@Leonidas-from-XIV Which kernel version do you have? I'm on 6.14.2 and I cannot reproduce. The GH actions ubuntu runners have been using 6.11.0-1013-azure since Dedember.

@rgrinberg rgrinberg force-pushed the ps/rr/test__fix_utop_tests branch from 26574e6 to c298e05 Compare May 17, 2025 13:41
@Alizter
Copy link
Collaborator

Alizter commented May 17, 2025

I think the correct fix is Stdlib.exit instead. I got it working in #11788.

@rgrinberg
Copy link
Member Author

Could you push the fix to this PR then?

@Alizter
Copy link
Collaborator

Alizter commented May 17, 2025

It won't work on 5.3 since the help message has extra quotations and that isn't backwards compatiable. We have to wait until we bump the CI to 5.3.

@rgrinberg rgrinberg force-pushed the ps/rr/test__fix_utop_tests branch from c298e05 to 2c4b736 Compare May 17, 2025 16:29
Signed-off-by: Rudi Grinberg <[email protected]>

<!-- ps-id: ad13c693-c903-495c-9b2c-3c8cf6212119 -->

Signed-off-by: Rudi Grinberg <[email protected]>
@rgrinberg rgrinberg force-pushed the ps/rr/test__fix_utop_tests branch from 2c4b736 to 9087df8 Compare May 17, 2025 16:29
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