-
Notifications
You must be signed in to change notification settings - Fork 3k
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
FIX: add cbor module in the requirement.txt #12358
Conversation
@jamesbeyond, thank you for your changes. |
CI started |
Thanks for fixing the nightly for MUSCA target |
Test run: FAILEDSummary: 3 of 4 test jobs failed Failed test jobs:
|
I haven't seen this error for some time cc @ARMmbed/mbed-os-test |
CI restarted |
Test run: FAILEDSummary: 3 of 4 test jobs failed Failed test jobs:
|
odd failure, I am checking it out |
Any update for the failures? |
I think that's some CI issue, will try to have a catch-up with Veli-Matti to resolve that |
Can we get it in this week to fix nightly build for this target? |
I think this is fixed in the Infra side by @VeliMattiLahtela |
CI restarted |
Test run: FAILEDSummary: 1 of 4 test jobs failed Failed test jobs:
|
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Hey @0xc0170 , I think CI cache got cleared, finally, this PR should be good to go |
Summary of changes
In PullRequest : #12231
cbor
module is used in https://github.com/ARMmbed/mbed-os/blob/master/tools/psa/tfm/bin_utils/imgtool_lib/boot_record.py#L21However, the module wasn't installed by default. building targets like
ARM_MUSCA_A1_NS
is failingSo adding this module to requirements.txt
Impact of changes
Migration actions required
Documentation
Pull request type
Test results
Reviewers
@Devran01 @VeliMattiLahtela