Skip to content

Conversation

@LukasKarel
Copy link
Contributor

Summary of changes

Make event queue size of the cellulardevice configurable. The state machine crashes sometimes at my application because it cant add a new callback to the queue.

Impact of changes

Migration actions required

Documentation


Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[x] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Reviewers


@0xc0170
Copy link
Contributor

0xc0170 commented Sep 13, 2023

Hi @LukasKarel ,

you need to update unittests , see the error here https://github.com/ARMmbed/mbed-os/actions/runs/6145924743/job/16674270974?pr=15454#step:5:581

@LukasKarel LukasKarel force-pushed the feature/cellularevents branch from 46156ae to ab46d51 Compare September 13, 2023 13:11
@LukasKarel
Copy link
Contributor Author

Hi @0xc0170

Should be fixed

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 13, 2023

Ci started

@0xc0170 0xc0170 added the release-type: patch Indentifies a PR as containing just a patch label Sep 13, 2023
@mbed-ci
Copy link

mbed-ci commented Sep 13, 2023

Jenkins CI Test : ✔️ SUCCESS

Build Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & Artifacts

CLICK for Detailed Summary

jobs Status
jenkins-ci/mbed-os-ci_build-cloud-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-cloud-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_unittests ✔️
jenkins-ci/mbed-os-ci_build-greentea-ARM ✔️
jenkins-ci/mbed-os-ci_build-greentea-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_build-example-ARM ✔️
jenkins-ci/mbed-os-ci_build-example-GCC_ARM ✔️
jenkins-ci/mbed-os-ci_greentea-test ✔️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-type: patch Indentifies a PR as containing just a patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants