Skip to content

Conversation

krish2718
Copy link
Contributor

  • Revert revert of quarantine, left a couple of Matter over Wi-Fi tests in Quarantine.
  • Pull fixes for Wi-Fi RC1 testing issues

@krish2718 krish2718 added this to the 2.8.0 milestone Nov 4, 2024
@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 Nov 4, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 4, 2024

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@0f0e5ad nrfconnect/sdk-zephyr@2d92e09 (v3.7.99-ncs1-branch) nrfconnect/[email protected]

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 4, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 6693309ed1b9818233747082b230785bf844c82d
zephyr: PR head: 2d92e0998e24d6f2dd204f419f7e7868e71e630d

more details

sdk-nrf:

PR head: 6693309ed1b9818233747082b230785bf844c82d
merge base: 73448b7595faa6b16a5d91d17ad36c0ef06f4fa3
target head (v2.8-branch): 73448b7595faa6b16a5d91d17ad36c0ef06f4fa3
Diff

zephyr:

PR head: 2d92e0998e24d6f2dd204f419f7e7868e71e630d
merge base: 0f0e5ad7da0d1e423a1991485e9dfc6fef3757fe
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 (13)
applications
│  ├── matter_bridge
│  │  │ sample.yaml
samples
│  ├── wifi
│  │  ├── radio_test
│  │  │  │ sample.yaml
scripts
│  │ quarantine.yaml
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── thingy53
│  │  │  │  │ thingy53_nrf5340_cpunet.dts
│  ├── drivers
│  │  ├── wifi
│  │  │  ├── nrfwifi
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── inc
│  │  │  │  │  │ wpa_supp_if.h
│  │  │  │  ├── src
│  │  │  │  │  ├── fmac_main.c
│  │  │  │  │  ├── qspi
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │ qspi_if.c
│  │  │  │  │  │ wpa_supp_if.c
│  ├── modules
│  │  ├── hostap
│  │  │  ├── CMakeLists.txt
│  │  │  │ Kconfig
│  │ west.yml

Outputs:

Toolchain

Version: b44b7a08c9
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b44b7a08c9_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 582
    • sdk-zephyr test count: 7229
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-sdk-find-my
    • ✅ test-fw-nrfconnect-nrf-iot_mosh
    • ✅ test-fw-nrfconnect-nrf-iot_positioning
    • ✅ test-sdk-wifi
    • ✅ test-low-level
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-nfc
    • 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_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-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk

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.

west.yml Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

@krish2718 Please update SHA: 2d92e0998e24d6f2dd204f419f7e7868e71e630d

Assorted Wi-Fi fixes for RC1 issues.

Signed-off-by: Chaitanya Tata <[email protected]>
nRF7002EB Wi-Fi support needs two shields to be passed. Please note
nRF70 Wi-Fi snippet is skipped as this applications overwrites the DTS.

Signed-off-by: Chaitanya Tata <[email protected]>
Using the same arg overwrites the previous shield, they should be passed
together and in proper order.

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

Signed-off-by: Chaitanya Tata <[email protected]>
54H + Wi-Fi is PSA only, but Matter is Legacy crypto and that is causing
build issues (MbedTLS configs).

For now Quarantine these samples till we figure out the issue.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718 krish2718 force-pushed the revert_quarantine_nrf70_28 branch from 1428b67 to 6693309 Compare November 4, 2024 15:57
@NordicBuilder NordicBuilder removed the DNM label Nov 4, 2024
@rlubos rlubos merged commit af698f8 into nrfconnect:v2.8-branch Nov 4, 2024
13 of 14 checks passed
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. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants