Skip to content

Add FLASHIAP for MTB_STM_S2LP #10968

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

Merged
merged 1 commit into from
Jul 5, 2019
Merged

Add FLASHIAP for MTB_STM_S2LP #10968

merged 1 commit into from
Jul 5, 2019

Conversation

teetak01
Copy link
Contributor

@teetak01 teetak01 commented Jul 5, 2019

Description

Fix device management client compilation on MTB_STM_S2LP

./mbed-cloud-client/mbed-client-pal/Source/Port/Reference-Impl/OS_Specific/mbedOS/ROT/pal_plat_rot.cpp:67:5: error: 'DeviceKey' was not declared in this scope
DeviceKey &devkey = DeviceKey::get_instance();

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@mikaleppanen @juhhei01 @katariinaliehu

Release Notes

Fix device management client compilation on MTB_STM_S2LP

Fixes:

./mbed-cloud-client/mbed-client-pal/Source/Port/Reference-Impl/OS_Specific/mbedOS/ROT/pal_plat_rot.cpp:67:5: error: 'DeviceKey' was not declared in this scope
    DeviceKey &devkey = DeviceKey::get_instance();
@artokin
Copy link
Contributor

artokin commented Jul 5, 2019

CI started.

@mbed-ci
Copy link

mbed-ci commented Jul 5, 2019

Test run: FAILED

Summary: 1 of 11 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_cloud-client-test

@artokin
Copy link
Contributor

artokin commented Jul 5, 2019

Test failed with K66F: "Could not create api key HTTP status 403"

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 5, 2019

@artokin Looks like we get this problem now on master. let me create testing PR where we can take this further. Wait, this one is the good candidate to fix this!

@teetak01 Any known issue today why client test are failing for Mbed OS PR? or is this network problem

@teetak01
Copy link
Contributor Author

teetak01 commented Jul 5, 2019

@OPpuolitaival @0xc0170 the problem is in your Pelion test account used to run those tests. Most likely your CI has generated too many API-keys, and its unable to create new ones anymore.

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 5, 2019

cc @ARMmbed/mbed-os-test Need your help ^^

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 5, 2019

cloud client test restarted (should be now fixed)

@0xc0170 0xc0170 merged commit 1eeed8c into ARMmbed:master Jul 5, 2019
@teetak01 teetak01 deleted the fix-mtb-stm-s2lp-for-client branch July 5, 2019 11:58
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.

6 participants