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

[dbus] fix introspect.xml for AddOnMeshPrefix() #2126

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

superwhd
Copy link
Contributor

@superwhd superwhd commented Dec 11, 2023

Fix the error calling AddOnMeshPrefix #2079 (comment). The interface in introspect.xml has been out-of-sync for.

Verification: adding a prefix.

$ sudo gdbus call --system --dest io.openthread.BorderRouter.wpan0 --method=io.openthread.BorderRouter.AddOnMeshPrefix --object-path /io/openthread/BorderRouter/wpan0 "(([0xfd,0x11,0x12,0x13,0x13,0x14,0x15,0x16], 64),0x5343, 0x01, false,false,false,false,false,false,false,false,false)"

()
> netdata show
Prefixes:
fd11:1213:1314:1516::/64 high d800
Routes:
Services:
44970 01 1f000500000e10 s d800
Contexts:
fd11:1213:1314:1516::/64 1 c
Commissioning:
12688 - - -
Done

Copy link
Contributor

@morningboata morningboata left a comment

Choose a reason for hiding this comment

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

Please have a local test to make sure it works as expected.

@superwhd
Copy link
Contributor Author

Please have a local test to make sure it works as expected.

Updated in the PR description.

@superwhd superwhd requested a review from jwhui December 12, 2023 02:50
@jwhui jwhui merged commit 1ad8af7 into openthread:main Dec 12, 2023
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants