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

patch: add tested modem disclaimer #2978

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

Conversation

rmorillo24
Copy link
Contributor

Add a disclaimer to the Supported Wifi Modems sections, to make clear that the tests we are executing are limited to a few things, and not for complete modem + board testing

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@rmorillo24 rmorillo24 requested a review from alexgg May 16, 2024 10:45
@flowzone-app flowzone-app bot enabled auto-merge May 16, 2024 10:50
- Wireless station mode can connect to an AP
- Bluetooth can scan
- A modem can register with a network

Copy link
Contributor

@alexgg alexgg May 16, 2024

Choose a reason for hiding this comment

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

I would remove the examples sanity tests as these can change with time.

Leaving the statement as:

Balena's software validation is only doing a sanity check on the features that are commonly required by balenaOS using the driver provided by the manufacturer and the NetworkManager/ModemManager support. Balena cannot guarantee that a modem or chipset is reliable and ready for production. End users should run the necessary tests to completely validate their use case and work with the vendor to resolve problems.

Copy link
Member

@vipulgupta2048 vipulgupta2048 left a comment

Choose a reason for hiding this comment

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

LGTM

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.

None yet

3 participants