Skip to content

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