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: Change iotlabs DEFAULT_SITE to grenoble #315

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MrKevinWeiss
Copy link
Contributor

It seems like saclay has a bunch of broken nodes on it, maybe grenoble is a bit better maintained...

It seems like saclay has a bunch of broken nodes on it, maybe grenoble is a bit better maintained...
@MrKevinWeiss
Copy link
Contributor Author

If saclay doesn't get fixed soon maybe we switch.

@MrKevinWeiss
Copy link
Contributor Author

@MrKevinWeiss
Copy link
Contributor Author

uggg, don't be fooled by the green, all the tests were skipped due to:

Can't access IoT-LAB front-end grenoble.iot-lab.info via SSH. Use key without password or `ssh-agent` and connect once manually to accept the fingerprint

@MrKevinWeiss
Copy link
Contributor Author

Needs an update from the RIOT action side.

@MrKevinWeiss
Copy link
Contributor Author

OK there we go!

@MrKevinWeiss
Copy link
Contributor Author

Let's wait for this until we have proper info on some failures we have seen.

@miri64
Copy link
Member

miri64 commented Jan 21, 2025

So far all the failing ones include notes that are at the Saclay site. 4.2 and 4.4 tests samr21-xpro to iotlab-m3 ping, 4.11 and 4.12 require an nrf52840dk, 4.5 and 4.6 explicitly pick the Strasbourg site. So I think my hunch got confirmed. At least for those tests, the site needs to be fixed to Saclay, as we did for the openmotes with Strasbourg.

@MrKevinWeiss MrKevinWeiss force-pushed the pr/switchdefaultsite branch 2 times, most recently from eec8e0d to c540e46 Compare January 21, 2025 14:14
@MrKevinWeiss
Copy link
Contributor Author

OK, let's keep this in the back pocket.

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