-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
openthread_border_router: bump to latest and smaller improvements #3732
Conversation
WalkthroughWalkthroughThe recent updates to the OpenThread Border Router enhance its functionality across multiple aspects. Key improvements include the addition of REST API support for deleting operational datasets, optimized route metrics for better routing preferences, and configurable logging behavior via command line arguments. Versioning has been updated in several components, along with changes in the Docker build process and updates to firmware naming conventions. These enhancements collectively improve system performance, usability, and maintainability. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant REST_API
participant Dataset_Manager
participant Thread_Leader
User->>REST_API: Request to delete dataset
REST_API->>Thread_Leader: Authorize deletion
Thread_Leader-->>REST_API: Authorization status
REST_API->>Dataset_Manager: Delete dataset if authorized
Dataset_Manager-->>REST_API: Confirmation of deletion
REST_API-->>User: Response with deletion confirmation
Recent review detailsConfiguration used: CodeRabbit UI Files selected for processing (9)
Files skipped from review as they are similar to previous changes (9)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (4)
openthread_border_router/0003-logging-support-disable-syslog-for-otbr-web-as-well.patch (1)
4-4
: Clarify the subject line.The subject line "[PATCH] [logging] support disable syslog for otbr-web as well" could be more precise. Consider rephrasing to "Add support to disable syslog for OTBR web interface."
openthread_border_router/0001-support-deleting-the-dataset.patch (3)
Line range hint
6-18
:
Ensure Consistency in API Descriptions.The description for deleting the active operational dataset mentions "Deletes the the active operational dataset." The duplicate "the" should be corrected for clarity.
- Deletes the the active operational dataset on the current node. Only allowed if the Thread node + Deletes the active operational dataset on the current node. Only allowed if the Thread node
Line range hint
6-18
:
Clarify the API Response Description.The response description for the DELETE operation on
/node/dataset/pending
incorrectly states "Successfully deleted the active operational dataset." It should refer to the pending operational dataset instead.- description: Successfully deleted the active operational dataset. + description: Successfully deleted the pending operational dataset.
Line range hint
12-47
:
Consider Refactoring for Clarity and Maintainability.The
DeleteDataset
function contains repeated logic for setting the dataset TLVs based on the dataset type. Consider refactoring to reduce redundancy.void Resource::DeleteDataset(DatasetType aDatasetType, Response &aResponse) const { otbrError error = OTBR_ERROR_NONE; std::string errorCode = GetHttpStatus(HttpStatusCode::kStatusOk); otOperationalDatasetTlvs datasetTlvs = {}; if (aDatasetType == DatasetType::kActive) { VerifyOrExit(otThreadGetDeviceRole(mInstance) == OT_DEVICE_ROLE_DISABLED, error = OTBR_ERROR_INVALID_STATE); } otError otError = (aDatasetType == DatasetType::kActive) ? otDatasetSetActiveTlvs(mInstance, &datasetTlvs) : otDatasetSetPendingTlvs(mInstance, &datasetTlvs); VerifyOrExit(otError == OT_ERROR_NONE, error = OTBR_ERROR_REST); aResponse.SetResponsCode(errorCode); exit: if (error == OTBR_ERROR_INVALID_STATE) { ErrorHandler(aResponse, HttpStatusCode::kStatusConflict); } else if (error != OTBR_ERROR_NONE) { ErrorHandler(aResponse, HttpStatusCode::kStatusInternalServerError); } }
…08-14) Update to the latest OTBR POSIX version b66cabfaa0 (2024-08-14 08:01:56 -0700). Bump to silabs-universal-flasher 0.0.22. This also adds a patch which gets rid of messages printed by OTBR web on the system console.
Update to the latest OTBR RCP builds for Yellow/Connect ZBT-1 (formerly SkyConnect). This firmware is built from the very same sources (there was no bump between Gecko SDK v4.4.0.0 and v4.4.3.0). However, it uses the new build infrastructure which could mean that there are small differences in build configuration.
8204fdb
to
f901a92
Compare
Update to the latest OTBR POSIX version b66cabfaa0 (2024-08-14 08:01:56 -0700). Bump to silabs-universal-flasher 0.0.22.
This also adds a patch which gets rid of messages printed by OTBR web on the system console.
Also update to the latest OTBR RCP builds for Yellow/Connect ZBT-1 (formerly SkyConnect). This firmware is built from the very same sources (there was no bump between Gecko SDK v4.4.0.0 and v4.4.3.0). However, it uses the new build infrastructure which could mean that there are small differences in build configuration.
Summary by CodeRabbit
New Features
otbr-web
component.Improvements
Version Updates
Changelog