Skip to content

Conversation

ludvigsj
Copy link
Contributor

@ludvigsj ludvigsj commented Oct 6, 2025

After the addition of NLC profile support, involving an increase in required RPL RAM usage, and the most recent upmerge adding a few extra bytes, this sample no longer fits in RAM on this platform. Per offline discussions, remove support for this platform.

@ludvigsj ludvigsj requested a review from a team as a code owner October 6, 2025 11:35
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 6, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 6, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 312c8b947750f6ad7d1d22526ff54c4c596aa494

more details

sdk-nrf:

PR head: 312c8b947750f6ad7d1d22526ff54c4c596aa494
merge base: 5390eed3d2c826cece40a03c993d0712628fed60
target head (main): 492a2e909230366db298db9ca4bc4bc8a13c8bf2
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 (3)
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
samples
│  ├── bluetooth
│  │  ├── mesh
│  │  │  ├── sensor_client
│  │  │  │  │ sample.yaml
scripts
│  │ quarantine.yaml

Outputs:

Toolchain

Version: a7529a11f4
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:a7529a11f4_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 12
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-ble_mesh
Disabled integration tests
    • desktop52_verification
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@ludvigsj ludvigsj force-pushed the develop/remove-nrf52832-from-sensor-cli branch from 9000800 to 8a4094c Compare October 6, 2025 11:38
@ludvigsj ludvigsj requested a review from a team as a code owner October 6, 2025 11:38
Copy link

github-actions bot commented Oct 6, 2025

Since quarantine was modified, please make sure you are following the process described in Quarantine Process.

After the addition of NLC profile support, involving an increase in
required RPL RAM usage, and the most recent upmerge adding a few extra
bytes, this sample no longer fits in RAM on this platform. Per offline
discussions, remove support for this platform.

Signed-off-by: Ludvig Jordet <[email protected]>
@ludvigsj ludvigsj force-pushed the develop/remove-nrf52832-from-sensor-cli branch from 8a4094c to 312c8b9 Compare October 6, 2025 11:52
@ludvigsj ludvigsj requested a review from a team as a code owner October 6, 2025 11:52
@NordicBuilder NordicBuilder added doc-required PR must not be merged without tech writer approval. and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 6, 2025
Copy link

github-actions bot commented Oct 6, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-24884/nrf/releases_and_maturity/releases/release-notes-changelog.html

@nordicjm nordicjm merged commit ae566f9 into nrfconnect:main Oct 7, 2025
23 checks passed
@ludvigsj ludvigsj deleted the develop/remove-nrf52832-from-sensor-cli branch October 7, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants