Skip to content

Conversation

@SeppoTakalo
Copy link
Contributor

@SeppoTakalo SeppoTakalo commented Oct 31, 2024

Pull fixes from upstream for coap_client.

Fixes issues reported in: NCSDK-29890

@SeppoTakalo SeppoTakalo requested a review from a team as a code owner October 31, 2024 08:33
@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 Oct 31, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 31, 2024

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@25f14b4 (main) nrfconnect/sdk-zephyr#2205 nrfconnect/sdk-zephyr#2205/files

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 31, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 2d40b8e665922acacc3dfecd7a81b058b6075112
zephyr: PR head: 5c922fc2513e792d3dfe12593c3dd0f58e57c33b

more details

sdk-nrf:

PR head: 2d40b8e665922acacc3dfecd7a81b058b6075112
merge base: 6621a392d3bf020b753956eda24961338dcef099
target head (main): 80f588c04c8439be1314c34c6df2eb27a27aaf62
Diff

zephyr:

PR head: 5c922fc2513e792d3dfe12593c3dd0f58e57c33b
merge base: 25f14b4f8ecba7423f39ead3c117c0d86a096c5b
target head (main): 25f14b4f8ecba7423f39ead3c117c0d86a096c5b
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 (11)
west.yml
zephyr
│  ├── include
│  │  ├── zephyr
│  │  │  ├── net
│  │  │  │  ├── coap.h
│  │  │  │  ├── coap_client.h
│  │  │  │  │ coap_service.h
│  ├── subsys
│  │  ├── net
│  │  │  ├── lib
│  │  │  │  ├── coap
│  │  │  │  │  ├── coap.c
│  │  │  │  │  │ coap_client.c
│  ├── tests
│  │  ├── net
│  │  │  ├── lib
│  │  │  │  ├── coap
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  │  ├── coap_client
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── src
│  │  │  │  │  │  ├── main.c
│  │  │  │  │  │  ├── stubs.c
│  │  │  │  │  │  │ stubs.h

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: 649
    • sdk-zephyr test count: 7203
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • ✅ test-fw-nrfconnect-nrf-iot_samples
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-fw-nrfconnect-thread
    • ✅ test-sdk-find-my
    • ✅ test-fw-nrfconnect-nrf-iot_mosh
    • ✅ test-fw-nrfconnect-nrf-iot_positioning
    • ✅ test-low-level
    • ✅ test-sdk-dfu
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-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-zigbee
    • test-sdk-audio
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi

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.

Pull fixes from upstream for coap_client.

Signed-off-by: Seppo Takalo <[email protected]>
@SeppoTakalo SeppoTakalo added this to the 2.8.0 milestone Nov 1, 2024
@SeppoTakalo SeppoTakalo closed this Nov 4, 2024
@SeppoTakalo SeppoTakalo deleted the coap_client_fixes branch November 4, 2024 14:09
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. DNM manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants