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

scripts: tools-versions-*.yml: update zephyr-sdk==0.16.8 #15145

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

thst-nordic
Copy link
Contributor

@thst-nordic thst-nordic commented Apr 30, 2024

scripts: tools-versions-*.yml: update zephyr-sdk==0.16.8
Release Notes:
Removed experimental ISO C11 <threads.h>-based gthread implementation

@thst-nordic thst-nordic requested review from tejlmand and a team as code owners April 30, 2024 14:11
@thst-nordic thst-nordic marked this pull request as draft April 30, 2024 14:11
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 30, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 30, 2024

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite

Detailed information of selected test modules

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.

carlescufi

This comment was marked as outdated.

@thst-nordic thst-nordic changed the title scripts: tools-versions-*.yml: update zephyr-sdk==0.16.6.rc1 scripts: tools-versions-*.yml: update zephyr-sdk==0.16.6 May 3, 2024
@thst-nordic
Copy link
Contributor Author

some trouble with building windows toolchain. nrfutil team is looking at it today.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 7, 2024

The changes to the 'requirements-fixed.txt' have been added to this PR.
All further changes to any of the requirements file will automatically be applied as long an 👀 is present.

Note: This comment is automatically posted and updated by the Comment GitHub Action.

@jangalda-nsc jangalda-nsc marked this pull request as ready for review May 8, 2024 07:40
@jangalda-nsc jangalda-nsc self-requested a review May 8, 2024 07:40
@thst-nordic
Copy link
Contributor Author

thst-nordic commented May 10, 2024

There are problems with the 0.16.6 release that have been reverted for 0.16.7.
zephyrproject-rtos/sdk-ng#751
0.16.7 was tagged but not yet 'released' with binaries.

I will update this PR to use 0.16.7 as soon as the artifacts are ready.

@thst-nordic
Copy link
Contributor Author

With the revert of the C11-threads feature there are only 2 commits in 0.16.7, and neither is urgent.
Lets wait for next zephyr-ng release.
zephyrproject-rtos/gcc@b016c34
zephyrproject-rtos/gcc@06c248d

@thst-nordic thst-nordic marked this pull request as draft May 10, 2024 11:04
@thst-nordic thst-nordic changed the title scripts: tools-versions-*.yml: update zephyr-sdk==0.16.6 scripts: tools-versions-*.yml: update zephyr-sdk==0.16.8 Aug 8, 2024
@thst-nordic
Copy link
Contributor Author

I updated this to 0.16.8, and am rebuilding.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 12, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 3b84ae825fae408b58033b58242b501524b810a1

more details

sdk-nrf:

PR head: 3b84ae825fae408b58033b58242b501524b810a1
merge base: 29d06abd637a7712762fe32ed09f1fa15825678b
target head (main): 7f81ccf2da3556f2902a40da57bc2971c5dd7508> 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)
scripts
│  ├── tools-versions-darwin.yml
│  ├── tools-versions-linux.yml
│  │ tools-versions-win10.yml

Outputs:

Toolchain

Version: 5425d178a8
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:5425d178a8_81ed5a52d6

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 31
  • ✅ Integration tests
Disable integration tests
    • desktop52_verification
    • doc-internal
    • latest/ble/nrf/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-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_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • 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
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi

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

@thst-nordic thst-nordic marked this pull request as ready for review August 12, 2024 14:07
@thst-nordic thst-nordic added CI-all-test Run All integration tests CI-force-downstream CI-run-zephyr-twister CI-run-twister and removed CI-all-test Run All integration tests labels Aug 12, 2024
https://github.com/zephyrproject-rtos/sdk-ng/releases/tag/v0.16.8
Release Notes say:
  Removed experimental ISO C11 <threads.h>-based gthread implementation

Signed-off-by: Thomas Stilwell <[email protected]>
@thst-nordic
Copy link
Contributor Author

testing is complete. all twister and downstream passed (with exemption for known issue on wifi)

CI Information

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

@thst-nordic thst-nordic merged commit b81ab8e into nrfconnect:main Aug 23, 2024
19 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants