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

Test timeout exploration #3048

Merged
merged 15 commits into from
Jan 18, 2025
Merged

Test timeout exploration #3048

merged 15 commits into from
Jan 18, 2025

Conversation

drmason789
Copy link
Contributor

This PR is host to code changes made to explore recent timeouts of the test suite.

@drmason789
Copy link
Contributor Author

It seems github can't pull from github right now. Will need to check up on this later.

@cagnulein
Copy link
Owner

Thanks @drmason789

@drmason789
Copy link
Contributor Author

It is taking a long time between these:

qt.bluetooth: Using BlueZ kernel ATT interface
qt.bluetooth.bluez: Cannot open HCI socket

@cagnulein
Copy link
Owner

cagnulein commented Jan 14, 2025 via email

Reenabled Fitplus bike and CSC bike
@drmason789
Copy link
Contributor Author

These tests run okay on my Linux development machine. Perhaps there's a system setting in the Github environment that is a timeout for trying to open the HCI socket that's changed.

This may take a while to solve. Perhaps you could comment them out in the Github Actions workflow until we figure this out.

@cagnulein
Copy link
Owner

Ok I will disable them, thanks for your time!

@drmason789
Copy link
Contributor Author

@cagnulein An update of the linux environment to ubuntu 24.04 appears to have fixed it.

@cagnulein
Copy link
Owner

@cagnulein An update of the linux environment to ubuntu 24.04 appears to have fixed it.

Wow that's great! So we would had just to wait? I can simply restore back the tests?

@drmason789
Copy link
Contributor Author

Yes, restore the tests, but also update the Linux environment. See the code change in this PR.

@cagnulein cagnulein merged commit 2570f28 into cagnulein:master Jan 18, 2025
11 checks passed
@cagnulein
Copy link
Owner

Thanks as usual!

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