Reduced configuration continued #1

teetak01 · 2019-07-01T07:47:54Z

No description provided.

MarceloSalazar

Is looking good. Just a minor question for now.

main.cpp

teetak01 · 2019-07-01T13:29:21Z

Added E2E-tests:

icetea --suite basic_os_tests.json --suitedir TESTS --tcdir ./TESTS --tc_cfg TESTS/pelion.tc_cfg --type hardware --reset

+----------------------+---------+--------------------------------------------+-------------+-----------+-----------+
| Testcase             | Verdict |                Fail Reason                 | Skip Reason | platforms |  duration |
+----------------------+---------+--------------------------------------------+-------------+-----------+-----------+
| register_and_read_id |   pass  |                                            |             |    K64F   | 22.686142 |
| get                  |   pass  |                                            |             |           |  6.217801 |
| put                  |   pass  |                                            |             |           |  6.639619 |
| post                 |   pass  |                                            |             |           |  6.289506 |
| observation          |   pass  |                                            |             |           |  7.901215 |
| deregister           |   fail  | POST request failed with 400 and msg (400) |             |           | 35.730681 |
|                      |         |             Reason: Bad Reque              |             |           |           |
+----------------------+---------+--------------------------------------------+-------------+-----------+-----------+
+---------------+----------------+
|    Summary    |                |
+---------------+----------------+
| Final Verdict |      FAIL      |
|     count     |       6        |
|    passrate   |    83.33 %     |
|      pass     |       5        |
|      fail     |       1        |
|    Duration   | 0:01:25.464964 |
+---------------+----------------+

One test failing because there is no Unregister feature. Either we could drop this test, or add the resource for that functionality.

yogpan01 · 2019-07-01T15:01:57Z

We need to add Update test case to the automated suite. Also, unregister is a good test case to be added in order to remove the registration from Portal and not leave any hanging devices in the Cloud.

This is default behaviour for Update client.

SST is enabled by default. Keep CEC client disabled also for K66F. No need to have advanced features on this example.

* All platforms will use hardcoded entropy injection. This is needed only for non-TRNG boards, but for code simplicity, do it for all. * Accept also CA_CERT existing error from fcc_developer() API.

Make Blocksize to be 512 KiB and update buffer 1 KiB by default. This saves some RAM and 512 is going to be the client default in future.

* Seperate functions for deregister callback and deregister handling. * Refactored the logic for handling shutdown. * Bring down network as part of shutdown.

* Simplify trace usage * Cleanup: remove unnessary carriage returns from printfs * Remove update authorize This is default behaviour for Update client. * Print IP * Handle fcc_init errors * Cleanup tracing * Fix default certificate template * Cleanup default configurations SST is enabled by default. Keep CEC client disabled also for K66F. No need to have advanced features on this example. * Cleanup printing * Use same resources as MCCE * Remove mbed_lib.json * Remove developer-mode flag * Implement reset+reboot on key-press 'r' * Remove configs folder with all content * Remove the need for PAL config file (requires patch) * Fix printing of counter * Remove button + led configs * Remove old tests * Add Device Management E2E tests * Allow reading API key from MBED_CLOUD_SDK_API_KEY variable * Add entropy injection with hardcoded entropy * All platforms will use hardcoded entropy injection. This is needed only for non-TRNG boards, but for code simplicity, do it for all. * Accept also CA_CERT existing error from fcc_developer() API. * Fix F411RE configuration * Cleanup mbed_cloud_client_user_config file Make Blocksize to be 512 KiB and update buffer 1 KiB by default. This saves some RAM and 512 is going to be the client default in future. * Make INFO the default max for trace * Remove esp-config as its not needed * Add deregister (5000/0/1) and refactor shutdown handling * Seperate functions for deregister callback and deregister handling. * Refactored the logic for handling shutdown. * Bring down network as part of shutdown. * Update to MCC 3.3.0 and Mbed OS 5.13.0

MarceloSalazar reviewed Jul 1, 2019

View reviewed changes

main.cpp Outdated Show resolved Hide resolved

teetak01 added 15 commits July 1, 2019 18:07

Simplify trace usage

b17c28f

Cleanup: remove unnessary carriage returns from printfs

34c26ba

Remove update authorize

091f590

This is default behaviour for Update client.

Print IP

17e02cc

Handle fcc_init errors

97fba57

Cleanup tracing

d67515a

Fix default certificate template

8538c24

Cleanup default configurations

a85ca61

SST is enabled by default. Keep CEC client disabled also for K66F. No need to have advanced features on this example.

Cleanup printing

6fa4b44

Use same resources as MCCE

c97042a

Remove mbed_lib.json

6cf4b7c

Remove developer-mode flag

0099338

Implement reset+reboot on key-press 'r'

9b282ab

Remove configs folder with all content

7c6d2fe

Remove the need for PAL config file (requires patch)

4e35271

yogpan01 force-pushed the reduced-configuration-continued branch from 252ba9c to 86facf0 Compare July 1, 2019 15:08

teetak01 added 10 commits July 2, 2019 08:01

Fix printing of counter

c5cc450

Remove button + led configs

8d083fc

Remove old tests

2a1b9f2

Add Device Management E2E tests

f9d2b1a

Allow reading API key from MBED_CLOUD_SDK_API_KEY variable

07f4d33

Add entropy injection with hardcoded entropy

600578a

* All platforms will use hardcoded entropy injection. This is needed only for non-TRNG boards, but for code simplicity, do it for all. * Accept also CA_CERT existing error from fcc_developer() API.

Fix F411RE configuration

40606e0

Cleanup mbed_cloud_client_user_config file

55a2a7f

Make Blocksize to be 512 KiB and update buffer 1 KiB by default. This saves some RAM and 512 is going to be the client default in future.

Make INFO the default max for trace

ca20243

Remove esp-config as its not needed

3b8e47e

teetak01 force-pushed the reduced-configuration-continued branch from 86facf0 to 3b8e47e Compare July 2, 2019 06:38

teetak01 added 2 commits July 2, 2019 10:32

Add deregister (5000/0/1) and refactor shutdown handling

a871f8e

* Seperate functions for deregister callback and deregister handling. * Refactored the logic for handling shutdown. * Bring down network as part of shutdown.

Update to MCC 3.3.0 and Mbed OS 5.13.0

442c2c4

teetak01 merged commit db712dd into ARMmbed:reduced-configuration Jul 4, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Reduced configuration continued #1

Reduced configuration continued #1

Uh oh!

teetak01 commented Jul 1, 2019

Uh oh!

MarceloSalazar left a comment

Uh oh!

Uh oh!

teetak01 commented Jul 1, 2019 •

edited

Loading

Uh oh!

yogpan01 commented Jul 1, 2019

Uh oh!

Uh oh!

Reduced configuration continued #1

Reduced configuration continued #1

Uh oh!

Conversation

teetak01 commented Jul 1, 2019

Uh oh!

MarceloSalazar left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

teetak01 commented Jul 1, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yogpan01 commented Jul 1, 2019

Uh oh!

Uh oh!

teetak01 commented Jul 1, 2019 •

edited

Loading