Skip to content

Conversation

@krish2718
Copy link
Contributor

@krish2718 krish2718 commented May 15, 2025

  • Fix XIP build error for nRF52
  • Fix test spec for wifi

@krish2718 krish2718 requested review from nordic-piks and rado17 May 15, 2025 13:36
@krish2718 krish2718 requested review from a team and sachinthegreen as code owners May 15, 2025 13:36
@krish2718 krish2718 added the Hotfix Fixes CI or a mistake label May 15, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 15, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 806eca22a515ec0893a29bef9513fcca7c96f1d8

more details

sdk-nrf:

PR head: 806eca22a515ec0893a29bef9513fcca7c96f1d8
merge base: 680aba9ccc9d21d43a4b0560123fc009cda0a3d9
target head (main): 680aba9ccc9d21d43a4b0560123fc009cda0a3d9
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 (4)
.github
│  │ test-spec.yml
scripts
│  ├── ci
│  │  │ tags.yaml
│  │ quarantine.yaml
subsys
│  ├── net
│  │  ├── lib
│  │  │  ├── nrf70_fw_ext
│  │  │  │  │ ext_fw_load.c

Outputs:

Toolchain

Version: 1d2c5d0ab5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:1d2c5d0ab5_e85602c25f

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 7
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-sdk-wifi
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • 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_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • 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
    • 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-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-secdom-samples-public

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

@krish2718 krish2718 force-pushed the fix_52840_xip_twister branch from 7943d42 to 28042e8 Compare May 15, 2025 16:21
@krish2718 krish2718 requested review from a team, PerMac and katgiadla as code owners May 15, 2025 16:21
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 15, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

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

@github-actions
Copy link

You can find the documentation preview for this PR here.

@github-actions
Copy link

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

@krish2718 krish2718 force-pushed the fix_52840_xip_twister branch from 1ce3e36 to 953688b Compare May 16, 2025 11:26
krish2718 added 4 commits May 16, 2025 18:22
The global variable was missing the guard causing twister failure for
nRF52.

Signed-off-by: Chaitanya Tata <[email protected]>
Add the downstream alternatives for hostap crypto and FW loading.

Signed-off-by: Chaitanya Tata <[email protected]>
Due to re-org over multiple releases Wi-Fi files paths have changed, fix
those entries.

Signed-off-by: Chaitanya Tata <[email protected]>
This reverts commit 1da9b3e.

This is now fixed.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718 krish2718 force-pushed the fix_52840_xip_twister branch from 953688b to 806eca2 Compare May 16, 2025 12:52
@krish2718 krish2718 requested a review from nordic-piks May 16, 2025 14:27
@krish2718
Copy link
Contributor Author

@PerMac and @nordic-piks please review.

@PerMac PerMac merged commit 117be0f into nrfconnect:main May 19, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Hotfix Fixes CI or a mistake

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants