-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Pass the country code via commissioning parameters. #17284
Pass the country code via commissioning parameters. #17284
Conversation
We shouldn't assume the configuration manager exists on the commissioner, or stores anything useful, so switch to passing in the country code via commissioning parameters instead of reading it from the configuration manager.
PR #17284: Size comparison from 2c15978 to cc2fd03 Increases (3 builds for linux)
Decreases (1 build for linux)
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing suggestion: You can test this in the commissiong_test.py cirque test. There is a truly insane amount of plumbing to get through to the python layer, but it's do-able. In src/controller/python/ChipDeviceController-ScriptBinding.cpp, there is a set of commissioning parameters that we use for commissioning. There are functions to set the wifi and thread data sets, this can be extended out to set also the country code (you'd have to add the plumbing). In commissioning_test.py, set the country code before TestKeyExchange, then check that the value is reflected in the basic info cluster.
Filed #17368 |
We shouldn't assume the configuration manager exists on the commissioner, or stores anything useful, so switch to passing in the country code via commissioning parameters instead of reading it from the configuration manager.
We shouldn't assume the configuration manager exists on the
commissioner, or stores anything useful, so switch to passing in the
country code via commissioning parameters instead of reading it from
the configuration manager.
Problem
See above.
Change overview
See above.
Testing
Not sure how to test this so far....