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

openthread: Add support for platform message management #18564

Merged
merged 3 commits into from
Nov 19, 2024

Conversation

adigie
Copy link
Member

@adigie adigie commented Nov 5, 2024

  • doc: openthread: add section on message pool configuration
  • doc: matter: add note on openthread memory configuration
  • manifest: sdk-zephyr:
    • [nrf fromlist] net: openthread: Add platform message management

@adigie adigie requested review from a team as code owners November 5, 2024 10:43
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 5, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 5, 2024

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@90eeeeb nrfconnect/sdk-zephyr@4f3ef0a (main) nrfconnect/[email protected]

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 5, 2024

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 7

Inputs:

Sources:

sdk-nrf: PR head: 9be1b625ff225033a22134fdd2bc1fcd4a7e6b1c
zephyr: PR head: 4f3ef0aa00b54dc19da82629c3402b2af149dfb5

more details

sdk-nrf:

PR head: 9be1b625ff225033a22134fdd2bc1fcd4a7e6b1c
merge base: 193434541373f44df977543b5f568bcd9a41a2cb
target head (main): e96f21fe1614174ca630d1c469a8065713d61326
Diff

zephyr:

PR head: 4f3ef0aa00b54dc19da82629c3402b2af149dfb5
merge base: 90eeeeb6b616d628ba84f3df65c2072bff6b7506
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (7)
doc
│  ├── nrf
│  │  ├── protocols
│  │  │  ├── matter
│  │  │  │  ├── getting_started
│  │  │  │  │  │ memory_optimization.rst
│  │  │  ├── thread
│  │  │  │  │ configuring.rst
west.yml
zephyr
│  ├── modules
│  │  ├── openthread
│  │  │  ├── platform
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── messagepool.c
│  │  │  │  │ openthread-core-zephyr-config.h
│  ├── subsys
│  │  ├── net
│  │  │  ├── l2
│  │  │  │  ├── openthread
│  │  │  │  │  │ Kconfig

Outputs:

Toolchain

Version:
Build docker image:

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
    • ◻️ test-fw-nrfconnect-chip
    • ◻️ test-fw-nrfconnect-nfc
    • ◻️ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • ◻️ test-fw-nrfconnect-nrf-iot_thingy91
    • ◻️ test-fw-nrfconnect-rs
    • ◻️ test-fw-nrfconnect-fem
    • ◻️ test-fw-nrfconnect-thread
    • ◻️ test-sdk-find-my
    • ◻️ test-fw-nrfconnect-nrf-iot_mosh
    • ◻️ test-fw-nrfconnect-nrf-iot_positioning
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@NordicBuilder
Copy link
Contributor

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.

@adigie adigie force-pushed the ot-message-pool-docs branch from cb0149b to 6e47660 Compare November 5, 2024 11:01
@adigie adigie force-pushed the ot-message-pool-docs branch from 6e47660 to 87c1a16 Compare November 12, 2024 08:28
@adigie adigie force-pushed the ot-message-pool-docs branch from 87c1a16 to d95efb7 Compare November 12, 2024 09:08
Add section on openthread message pool configuration.

Signed-off-by: Adrian Gielniewski <[email protected]>
Mention openthread memory configuration in matter docs.

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie adigie force-pushed the ot-message-pool-docs branch from d95efb7 to b6ddaba Compare November 19, 2024 10:38
@NordicBuilder
Copy link
Contributor

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 Publish GitHub Action.

west.yml Outdated Show resolved Hide resolved
* [nrf fromlist] net: openthread: Add platform message management

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie adigie force-pushed the ot-message-pool-docs branch from b6ddaba to 9be1b62 Compare November 19, 2024 13:17
@NordicBuilder NordicBuilder removed the DNM label Nov 19, 2024
@rlubos rlubos merged commit 3e12072 into nrfconnect:main Nov 19, 2024
13 of 14 checks passed
@adigie adigie deleted the ot-message-pool-docs branch November 19, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. manifest manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants