-
Notifications
You must be signed in to change notification settings - Fork 325
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
Checklist for device integration testing #1434
Comments
additional ideas for this from my forum post: https://forum.freifunk.net/t/archer-c7-v-4-0/16384/7 |
Checklist for now:
Any other additions I might have missed? |
MAC: clarifiy this point that this should be checked like "idem to meshid" (=default routername-suffix, i do not know how this will be on the map for babel) |
|
@blocktrron @achterin maybe you can contribute to the suggested checklist |
|
|
Throughput tests sounds like a nice idea! However I think we would need to specify and document how to do them then as we would otherwise get varying results. |
Throughput tests will vary with several factor: MTU,bandwidth available, encryption, load on gateways, etc... So, in my opinion: stick to basic yes/no function tests |
Throughput is a dealbreaker. If everything works in principle, but you can only achieve 2 mbit/s, the node is not usable. Sure you could change the question to
but you'll have to measure it anyways. Might as well enter result then. |
My opinion below: Yay:
Optional:
Nay:
|
First a comment on @mweinelt: |
Nay=No? About throughput: I think it is important to test not the throughput, but test if the device is not crashing once there is much load (for example the Fonera Router runs fine with all tests, but it crashes, as soon as there are more than some clients and they start causing traffic):
|
I'd like the developers to focus on getting devices to work. The userbase should start testing that integration asap and report back issues. The userbase are many people that can far better find bottlenecks. I'd like to remind you of #125 where a proper limit to client counts couldn't be found as there are too many variables to that. |
i vote for the current proposal by @mweinelt . |
current status: we decided to put the checklist on github as part of a pull request template, see #1433 for progress on this |
fixed by adding it to the gluon github wiki: https://github.com/freifunk-gluon/gluon/wiki/Device-Integration-checklist ( done by @mweinelt ) |
What needs to be verified to do integration testing for a new device. Let's create a checklist.
@Brother-Lal wanted to take care of that.
Make use of our notes from the meeting here: https://md.darmstadt.ccc.de/gluon-device-support-policy#
The text was updated successfully, but these errors were encountered: