Skip to content

Conversation

@RobertGalatNordic
Copy link
Contributor

remove sidewalk project from west.yml,
to move it as out of tree project in add-on.

@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 Mar 12, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 12, 2025

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

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@1bdf7de nrfconnect/sdk-nrfxlib@b89f398 (main) nrfconnect/[email protected]
sidewalk ❌ nrfconnect/sdk-sidewalk@735afa5 N/A (Removed) N/A
zephyr nrfconnect/sdk-zephyr@a2aa12b nrfconnect/sdk-zephyr@8339016 (main) nrfconnect/[email protected]

DNM label due to: 1 removed project

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 12, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: e87c164eb12b35e12f5276d678345e1dbfd34f58
nrfxlib: PR head: b89f3981d80e1c337a199e1c450f3a4817632128
zephyr: PR head: 83390163009bfb440f9d811bd99a58970d286f9a

more details

sdk-nrf:

PR head: e87c164eb12b35e12f5276d678345e1dbfd34f58
merge base: 50ae83eba24c37fde286a5a601a2875f9e639cc2
target head (main): 50ae83eba24c37fde286a5a601a2875f9e639cc2
Diff

nrfxlib:

PR head: b89f3981d80e1c337a199e1c450f3a4817632128
merge base: 1bdf7dee04ba6cdc24537420b6c7c9f15363ee00
Diff

zephyr:

PR head: 83390163009bfb440f9d811bd99a58970d286f9a
merge base: a2aa12bb3dcc1e9fac6ce451692f326bf6a6467c
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 (9)
.github
│  ├── test-spec.yml
│  ├── workflows
│  │  │ src-mirror.yml
nrfxlib
│  ├── .github
│  │  │ test-spec.yml
scripts
│  ├── ci
│  │  │ tags.yaml
│  │ tag_west_repos.sh
west.yml
zephyr
│  ├── .github
│  │  │ test-spec.yml
│  ├── drivers
│  │  ├── wifi
│  │  │  ├── nrf_wifi
│  │  │  │  ├── Kconfig.nrfwifi
│  │  │  │  ├── src
│  │  │  │  │  │ fmac_main.c

Outputs:

Toolchain

Version: 4ffa2202d5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4ffa2202d5_e579f9fbe6

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 338
    • sdk-zephyr test count: 1286
  • ❌ Integration tests
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-nrf-iot_mosh
    • ✅ test-fw-nrfconnect-nrf-iot_positioning
    • ❌ test-low-level
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-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • 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
    • test-fw-nrfconnect-zigbee
    • 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

RobertGalatNordic added a commit to RobertGalatNordic/sdk-sidewalk that referenced this pull request Mar 12, 2025
final remove workarounds for sidewalk in nrf manifest
nrfconnect/sdk-nrf#20901

Signed-off-by: Robert Gałat <[email protected]>
@github-actions
Copy link

You can find the documentation preview for this PR here.

RobertGalatNordic added a commit to RobertGalatNordic/sdk-sidewalk that referenced this pull request Mar 13, 2025
add west.yml to sidewalk project
update github actions
improve build and test in GHA,
by dynamically selecting number of runners

nrfconnect/sdk-nrf#20901

update template for PRs

Signed-off-by: Robert Gałat <[email protected]>
@totyz totyz force-pushed the remove_sidewalk branch 2 times, most recently from 2441a92 to c2741a5 Compare March 19, 2025 14:38
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Mar 19, 2025
@totyz totyz removed the DNM label Mar 20, 2025
@RobertGalatNordic RobertGalatNordic marked this pull request as ready for review March 20, 2025 08:31
@RobertGalatNordic RobertGalatNordic requested review from a team as code owners March 20, 2025 08:31
@mariusz-nordicsemi
Copy link
Contributor

Should below entries be removed as well?

remove sidewalk project from west.yml,
to move it as out of tree project in add-on.

remove Sidewalk from test-spec

Signed-off-by: Robert Gałat <[email protected]>
@totyz totyz force-pushed the remove_sidewalk branch from 3953ca0 to e87c164 Compare March 20, 2025 12:16
@rlubos rlubos removed the DNM label Mar 20, 2025
@sonarqubecloud
Copy link

@rlubos
Copy link
Contributor

rlubos commented Mar 20, 2025

@nrfconnect/ncs-ci Please review.

@totyz totyz requested a review from jangalda-nsc March 20, 2025 12:23
@rlubos
Copy link
Contributor

rlubos commented Mar 20, 2025

Failed low-level job passed on rerun

@rlubos rlubos merged commit 10dd3dd into nrfconnect:main Mar 20, 2025
17 of 19 checks passed
totyz pushed a commit to nrfconnect/sdk-sidewalk that referenced this pull request Mar 21, 2025
add west.yml to sidewalk project
update github actions
improve build and test in GHA,
by dynamically selecting number of runners

nrfconnect/sdk-nrf#20901

update template for PRs

Signed-off-by: Robert Gałat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants