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.

Sign up for GitHub

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

Jump to bottom

manifest: openthread: update to \602167f\ #16524

Merged
merged 1 commit into from
Jul 30, 2024
Merged

manifest: openthread: update to \602167f\ #16524

merged 1 commit into from
Jul 30, 2024

Conversation

NordicBuilder
Copy link
Contributor

Regular OpenThread update to `602167f`

Signed-off-by: Nordic Builder [email protected]

@NordicBuilder NordicBuilder requested a review from rlubos as a code owner July 19, 2024 09:27
@NordicBuilder NordicBuilder requested a review from edmont July 19, 2024 09:27
@NordicBuilder NordicBuilder requested a review from edmont as a code owner July 19, 2024 09:27
@NordicBuilder
Copy link
Contributor Author

List of modified OpenThread API files :
etc/cmake/options.cmake
include/openthread/platform/
include/openthread/platform/dnssd.h
include/openthread/platform/diag.h
include/openthread/platform/radio.h
include/openthread/platform/ble.h

@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Jul 19, 2024
@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Jul 19, 2024 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
openthread nrfconnect/sdk-openthread@a57d927 nrfconnect/sdk-openthread@602167f (main) nrfconnect/[email protected]
zephyr nrfconnect/sdk-zephyr@4c996ec nrfconnect/sdk-zephyr@c784caa (main) nrfconnect/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@maciejbaczmanski
Copy link
Member

fyi, PR has been opened mainly to inspect changes since 2.7.0. It will not build until Zephyr release and merge of this PR: zephyrproject-rtos/openthread#162 . I'll create tasks with necessary alignments for this upmerge

@NordicBuilder
Copy link
Contributor Author

NordicBuilder commented Jul 19, 2024 •
edited
Loading

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-nfc X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_mosh X
test-fw-nrfconnect-nrf-iot_positioning X
test-fw-nrfconnect-nrf-iot_thingy91 X
test-fw-nrfconnect-nrf-iot_zephyr_lwm2m X
test-fw-nrfconnect-rpc X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-thread X
test-low-level X
test-sdk-dfu X
test-sdk-find-my X

test-fw-nrfconnect-thread: added because there was no .github/test-spec.yml in 'openthread'

Detailed information of selected test modules

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor Author

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@maciejbaczmanski maciejbaczmanski removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 29, 2024
@maciejbaczmanski
Copy link
Member

fyi, PR has been opened mainly to inspect changes since 2.7.0. It will not build until Zephyr release and merge of this PR: zephyrproject-rtos/openthread#162 . I'll create tasks with necessary alignments for this upmerge

It has been decided that we should pull that change as soon as possible. Tasks for activities that can be done but are not high priority (like Kconfig alignment) will be created later, but this PR is ready for review

@NordicBuilder @maciejbaczmanski
manifest: openthread: update to 602167f
efc1d1c 
Regular OpenThread update to `602167f`
Additionally Pulls Zephyr with commits needed for upmerge.

Signed-off-by: Nordic Builder <[email protected]>

Co-authored-by: Maciej Baczmanski <[email protected]>
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 30, 2024
@NordicBuilder NordicBuilder removed the DNM label Jul 30, 2024
@rlubos rlubos merged commit 5afa485 into nrfconnect:main Jul 30, 2024
27 of 28 checks passed
@nordic-piks nordic-piks mentioned this pull request Jul 30, 2024
Closed
@NordicBuilder NordicBuilder deleted the pr/update-openthread branch July 31, 2024 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rlubos rlubos rlubos approved these changes

@nordic-piks nordic-piks nordic-piks approved these changes

@edmont edmont Awaiting requested review from edmont

@maciejbaczmanski maciejbaczmanski Awaiting requested review from maciejbaczmanski

@canisLupus1313 canisLupus1313 Awaiting requested review from canisLupus1313

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-openthread manifest-zephyr
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@NordicBuilder @maciejbaczmanski @edmont @rlubos @canisLupus1313 @nordic-piks