Skip to content
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

Enable building feature storage with a bare metal profile #11275

Merged
merged 3 commits into from Aug 29, 2019

Conversation

@VeijoPesonen
Copy link
Contributor

commented Aug 21, 2019

Description

Enables building storage with bare metal profile.

To compile Mbed Bootloader with bare metal profile certain defines are
not added anymore automatically. Because of this checks for those
defines needed to be introduced.

Pull request type

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

Reviewers

@SeppoTakalo
@teetak01
@michalpasztamobica
@JanneKiiskila

Release Notes

To compile Mbed Bootloader with bare metal profile certain defines are
not added anymore automatically. Because of this checks for those
defines needed to be introduced.
@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 21, 2019

Built tests

Build successes:
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-BLOCKDEVICE-BUFFERED_BLOCK_DEVICE
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-BLOCKDEVICE-FLASHSIM_BLOCK_DEVICE
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-BLOCKDEVICE-GENERAL_BLOCK_DEVICE
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-BLOCKDEVICE-HEAP_BLOCK_DEVICE
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-BLOCKDEVICE-MBR_BLOCK_DEVICE
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-BLOCKDEVICE-UTIL_BLOCK_DEVICE
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-FILESYSTEM-GENERAL_FILESYSTEM
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-DIRECT_ACCESS_DEVICEKEY_TEST
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-FILESYSTEMSTORE_TESTS
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-GENERAL_TESTS_PHASE_1
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-GENERAL_TESTS_PHASE_2
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-SECURESTORE_WHITEBOX
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-STATIC_TESTS
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-KVSTORE-TDBSTORE_WHITEBOX
  * K64F::GCC_ARM::MBED-BUILD


Build skips:
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-FILESYSTEM-FAT_FILESYSTEM
  * K64F::GCC_ARM::FEATURES-STORAGE-TESTS-FILESYSTEM-MULTIPART_FAT_FILESYSTEM

Test results

| target       | platform_name | test suite                                                  | result | elapsed_time (sec) | copy_method |
|--------------|---------------|-------------------------------------------------------------|--------|--------------------|-------------|
| K64F-GCC_ARM | K64F          | features-storage-tests-blockdevice-buffered_block_device    | OK     | 13.82              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-blockdevice-flashsim_block_device    | OK     | 13.09              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-blockdevice-general_block_device     | OK     | 33.25              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-blockdevice-heap_block_device        | OK     | 15.67              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-blockdevice-mbr_block_device         | OK     | 14.68              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-blockdevice-util_block_device        | OK     | 14.17              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-filesystem-general_filesystem        | OK     | 52.33              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-direct_access_devicekey_test | OK     | 19.54              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-filesystemstore_tests        | OK     | 35.26              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-general_tests_phase_1        | OK     | 106.72             | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-general_tests_phase_2        | OK     | 103.02             | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-securestore_whitebox         | OK     | 24.8               | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-static_tests                 | OK     | 48.36              | default     |
| K64F-GCC_ARM | K64F          | features-storage-tests-kvstore-tdbstore_whitebox            | OK     | 18.62              | default     |
mbedgt: test suite results: 14 OK
@ciarmcom ciarmcom requested review from JanneKiiskila, michalpasztamobica, SeppoTakalo, teetak01 and ARMmbed/mbed-os-maintainers Aug 21, 2019
@ciarmcom

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

@TeroJaasko

This comment has been minimized.

Copy link
Contributor

commented Aug 21, 2019

IMHO, if you are breaking existing software by changing the default, it would be nice to add documentation to commit message how to keep existing software (such as MCC) building and running.

@JanneKiiskila

This comment has been minimized.

Copy link
Contributor

commented Aug 21, 2019

Yep, migration guides needed.

@0xc0170

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

Disables NVStore by default. NVStore needs to be enabled explicitly, if needed, from now on.

Besides commit msg as mentioned above, also why are we doing this? Provide a reason so users who come back with "why did you do this?" find the answer in the release notes and in the commit as well.

Yep, migration guides needed.

+1

@0xc0170 0xc0170 added needs: work and removed needs: review labels Aug 21, 2019
@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 22, 2019

@teetak01, @JanneKiiskila, @0xc0170 PR's description updated. Please check if the updated version is sufficient.

@JanneKiiskila

This comment has been minimized.

Copy link
Contributor

commented Aug 22, 2019

Hipsut are not the right ones there, they should be, i.e. x (shift+hipsu key next to your backspace key in upper right corner) rather than the 'x' they are now. The right ones change the formatting in GitHub (and Markdown).

You will get plus points by supplying the piece of text one needs to copy paste to mbed_app.json...

@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 22, 2019

@JanneKiiskila description updated.

@JanneKiiskila

This comment has been minimized.

Copy link
Contributor

commented Aug 22, 2019

No nyt on timanttia!-)

@0xc0170 0xc0170 requested a review from bulislaw Aug 22, 2019
@0xc0170 0xc0170 added needs: review and removed needs: work labels Aug 22, 2019
@0xc0170

This comment has been minimized.

Copy link
Member

commented Aug 26, 2019

@bulislaw Please review

I'll start CI meanwhile

@mbed-ci

This comment has been minimized.

Copy link

commented Aug 27, 2019

Test run: FAILED

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

Failed test jobs:

  • jenkins-ci/mbed-os-ci_cloud-client-test
  • jenkins-ci/mbed-os-ci_greentea-test
@bulislaw

This comment has been minimized.

Copy link
Member

commented Aug 27, 2019

That's a breaking change that should go to major release. That being said I would like us to deprecated nvstore in 5.15 and possibly remove it in 6.

@0xc0170

This comment has been minimized.

Copy link
Member

commented Aug 27, 2019

That's a breaking change that should go to major release. That being said I would like us to deprecated nvstore in 5.15 and possibly remove it in 6.

@VeijoPesonen What can we do here to get this in to the next minor ?

@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 27, 2019

Well, at least some of the test case failures seem to be unrelated to these changes. Need to admit that I haven't gone through all of those. @0xc0170 You wouldn't happen to know if this error is a symptom about some wider issue?

[1566892863.91][GLRM][INF] remote resources flashing with 'BUILD/tests/K64F/GCC_ARM/TESTS/mbed_platform/stats_heap/stats_heap.bin'...
[1566892863.91][urllib3.connectionpool]Starting new HTTPS connection (1): mervi.mbedcloudtesting.com:443
[1566892863.92][urllib3.connectionpool]https://mervi.mbedcloudtesting.com:443 "POST /file HTTP/1.1" 200 246
[1566892879.99][GLRM][INF] opening connection to platform at baudrate='9600'
[1566892879.99][urllib3.connectionpool]Starting new HTTPS connection (1): mervi.mbedcloudtesting.com:443
[1566892880.27][urllib3.connectionpool]https://mervi.mbedcloudtesting.com:443 "PUT /resource/0240000032044e4500287009997b00236781000097969900/connect HTTP/1.1" 200 100
[1566892880.27][GLRM][INF] remote resources reset...
[1566892883.66][GLRM][TXD] mbedmbedmbedmbedmbedmbedmbedmbedmbedmbed
[1566892883.66][CONN][INF] sending up to 10 __sync packets (specified with --sync=10)
[1566892883.66][CONN][INF] sending preamble '0e8452c2-12eb-4463-9c4b-0b9ecd4de172'
[1566892883.77][GLRM][TXD] {{__sync;0e8452c2-12eb-4463-9c4b-0b9ecd4de172}}
[1566892883.82][CONN][RXD] mbedmbedmbedmbedmbedmbedmbedmbed
[1566892883.92][CONN][INF] found SYNC in stream: {{__sync;0e8452c2-12eb-4463-9c4b-0b9ecd4de172}} it is #0 sent, queued...
[1566892883.92][CONN][INF] found KV pair in stream: {{__version;1.3.0}}, queued...
[1566892883.92][CONN][INF] found KV pair in stream: {{__timeout;20}}, queued...
[1566892883.92][HTST][INF] sync KV found, uuid=0e8452c2-12eb-4463-9c4b-0b9ecd4de172, timestamp=1566892883.919954
[1566892883.92][HTST][INF] DUT greentea-client version: 1.3.0
[1566892883.92][HTST][INF] setting timeout to: 20 sec
[1566892884.02][CONN][RXD] >>> Running 4 test cases...
[1566892884.02][CONN][INF] found KV pair in stream: {{__host_test_name;default_auto}}, queued...
[1566892884.02][CONN][INF] found KV pair in stream: {{__testcase_count;4}}, queued...
[1566892884.02][HTST][INF] host test class: '<class 'mbed_os_tools.test.host_tests.default_auto.DefaultAuto'>'
[1566892884.02][HTST][INF] host test setup() call...
[1566892884.02][HTST][INF] CALLBACKs updated
[1566892884.02][HTST][INF] host test detected: default_auto
[1566892884.12][CONN][INF] found KV pair in stream: {{__testcase_name;malloc and free size}}, queued...
[1566892884.12][CONN][INF] found KV pair in stream: {{__testcase_name;allocate size zero}}, queued...
[1566892884.12][CONN][INF] found KV pair in stream: {{__testcase_name;allocation failure}}, queued...
[1566892884.22][CONN][RXD]
[1566892884.22][CONN][RXD] >>> Running case #1: 'malloc and free size'...
[1566892884.22][CONN][INF] found KV pair in stream: {{__testcase_name;realloc size}}, queued...
[1566892884.32][CONN][INF] found KV pair in stream: {{__testcase_start;malloc and free size}}, queued...
[1566892884.32][CONN][INF] found KV pair in stream: {{__testcase_finish;malloc and free size;1;0}}, queued...
[1566892884.42][CONN][RXD] >>> 'malloc and free size': 1 passed, 0 failed
[1566892884.42][CONN][RXD]
[1566892884.42][CONN][RXD] >>> Running case #2: 'allocate size zero'...
[1566892884.52][CONN][INF] found KV pair in stream: {{__testcase_start;allocate size zero}}, queued...
[1566892884.52][CONN][INF] found KV pair in stream: {{__testcase_finish;allocate size zero;1;0}}, queued...
[1566892884.62][CONN][RXD] >>> 'allocate size zero': 1 passed, 0 failed
[1566892884.62][CONN][RXD]
[1566892884.62][CONN][RXD] >>> Running case #3: 'allocation failure'...
[1566892884.72][CONN][RXD]
[1566892884.72][CONN][RXD]
[1566892884.72][CONN][RXD] ++ MbedOS Error Info ++
[1566892884.72][CONN][RXD] Error Status: 0x80010133 Code: 307 Module: 1
[1566892884.72][CONN][INF] found KV pair in stream: {{__testcase_start;allocation failure}}, queued...
[1566892884.82][CONN][RXD] Error Message: Mutex: 0x200024A8, Not allowed in ISR context
[1566892884.82][CONN][RXD] Location: 0x5CB3
[1566892884.82][CONN][RXD] Error Value: 0x200024A8
[1566892884.92][CONN][RXD] Current Thread: main Id: 0x20001268 Entry: 0x5BE3 StackSize: 0x1000 StackMem: 0x20002500 SP: 0x2002FE58
[1566892885.12][CONN][RXD] For more info, visit: https://mbed.com/s/error?error=0x80010133&osver=999999&core=0x410FC241&comp=2&ver=80300&tgt=K64F
[1566892885.12][CONN][RXD] -- MbedOS Error Info --
[1566892885.12][CONN][RXD]
[1566892885.12][CONN][RXD] = System will be rebooted due to a fatal error =
[1566892904.14][HTST][INF] test suite run finished after 20.22 sec...
@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 29, 2019

NUCLEO_F429ZI issues

The issues seem to be possibly related to RAAS usage or may be to a certain mbedgt version. Tests will pass if run on a local machine. Switching to another RAAS instance didn't seem to have any effect.

Fails in RAAS

:~/ws/mbed-os> mbedgt -g NUCLEO_F429ZI:raas_client:https://ruka.mbedcloudtesting.com:443 -n tests-mbed_drivers-reset_reason -v -V
mbedgt: greentea test automation tool ver. 1.6.5
mbedgt: using multiple test specifications from current directory!
        using 'BUILD/tests/NUCLEO_F429ZI/GCC_ARM/test_spec.json'
mbedgt: detecting connected mbed-enabled devices...
mbedgt: entering global resource manager mbed-ls dummy mode!
        adding dummy platform 'NUCLEO_F429ZI'
mbedgt: detected 1 device
        | platform_name | platform_name_unique | serial_port | mount_point | target_id |
        |---------------|----------------------|-------------|-------------|-----------|
        | NUCLEO_F429ZI | NUCLEO_F429ZI[0]     | DUMMY       | DUMMY       | DUMMY     |
mbedgt: processing target 'NUCLEO_F429ZI' toolchain 'GCC_ARM' compatible platforms... (note: switch set to --parallel 1)
        | platform_name | platform_name_unique | serial_port | mount_point | target_id |
        |---------------|----------------------|-------------|-------------|-----------|
        | NUCLEO_F429ZI | NUCLEO_F429ZI[0]     | DUMMY:9600  | DUMMY       | DUMMY     |
mbedgt: test case filter (specified with -n option)
        tests-mbed_drivers-reset_reason
        test filtered in 'tests-mbed_drivers-reset_reason'
mbedgt: running 1 test for platform 'NUCLEO_F429ZI' and toolchain 'GCC_ARM'
        use 1 instance of execution threads for testing
mbedgt: checking for 'host_tests' directory above image directory structure
        found 'host_tests' directory in: 'TESTS/host_tests'
mbedgt: selecting test case observer...
        calling mbedhtrun: mbedhtrun -m NUCLEO_F429ZI -p DUMMY:9600 -f "BUILD/tests/NUCLEO_F429ZI/GCC_ARM/TESTS/mbed_drivers/reset_reason/reset_reason.bin" -e "TESTS/host_tests" --grm raas_client:https://ruka.mbedcloudtesting.com:443 -C 4 --sync 5 -P 60
mbedgt: mbed-host-test-runner: started
[1567078082.73][HTST][INF] host test executor ver. 0.0.6
[1567078082.73][HTST][INF] copy image onto target... SKIPPED!
[1567078082.73][HTST][INF] starting host test process...
[1567078082.74][CONN][INF] starting connection process...
[1567078082.74][CONN][INF] notify event queue about extra 60 sec timeout for serial port pooling
[1567078082.74][CONN][INF] initializing global resource mgr listener...
[1567078082.74][HTST][INF] setting timeout to: 60 sec
[1567078082.91][GLRM][INF] remote resources initialization: remote(host=https://ruka.mbedcloudtesting.com, port=443)
[1567078082.91][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078083.22][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "POST /login HTTP/1.1" 200 273
[1567078083.22][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078083.46][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /socket.io/?token=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VybmFtZSI6InVzZXIiLCJncm91cCI6InVzZXIiLCJpYXQiOjE1NjcwNzgwODMsImV4cCI6MTU2NzY4Mjg4MywianRpIjoiMTA1MWM2ZTAtY2E1MC0xMWU5LWI5ZTMtMDdkMTU2NTRhOTkyIn0.jUPISYaOhyjIdq20p1f2CryFDrRbF00gGK1HNrICmEM&EIO=3&transport=polling&t=1567078083219-0 HTTP/1.1" 200 101
[1567078083.80][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078083.86][dummy]socketio unknown event received: info
[1567078084.04][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /this/resource HTTP/1.1" 200 2735
[1567078084.04][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078084.44][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /resource HTTP/1.1" 200 227511
[1567078085.10][GLRM][INF] remote resources count: 164
[1567078085.37][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078085.70][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /resource/079602210538697A3A7CFFD1 HTTP/1.1" 200 1696
[1567078085.70][GLRM][INF] remote resources flashing with 'BUILD/tests/NUCLEO_F429ZI/GCC_ARM/TESTS/mbed_drivers/reset_reason/reset_reason.bin'...
[1567078085.71][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078086.10][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "POST /file HTTP/1.1" 200 248
[1567078107.75][GLRM][INF] opening connection to platform at baudrate='9600'
[1567078107.75][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078108.26][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "PUT /resource/079602210538697A3A7CFFD1/connect HTTP/1.1" 200 75
[1567078108.26][GLRM][INF] remote resources reset...
[1567078111.74][GLRM][TXD] mbedmbedmbedmbedmbedmbedmbedmbedmbedmbed
[1567078111.74][CONN][INF] sending up to 5 __sync packets (specified with --sync=5)
[1567078111.74][CONN][INF] sending preamble '16962471-521b-4714-87bb-c759b3fc1411'
[1567078111.96][GLRM][TXD] {{__sync;16962471-521b-4714-87bb-c759b3fc1411}}
[1567078111.98][CONN][RXD] mbedmbedmbedmbedmbedmbedmbedmbed
[1567078112.08][CONN][INF] found SYNC in stream: {{__sync;16962471-521b-4714-87bb-c759b3fc1411}} it is #0 sent, queued...
[1567078112.08][CONN][INF] found KV pair in stream: {{__version;1.3.0}}, queued...
[1567078112.08][CONN][INF] found KV pair in stream: {{__timeout;60}}, queued...
[1567078112.08][HTST][INF] sync KV found, uuid=16962471-521b-4714-87bb-c759b3fc1411, timestamp=1567078112.082704
[1567078112.08][HTST][INF] DUT greentea-client version: 1.3.0
[1567078112.08][HTST][INF] setting timeout to: 60 sec
[1567078112.18][CONN][INF] found KV pair in stream: {{__host_test_name;reset_reason}}, queued...
[1567078112.18][CONN][INF] found KV pair in stream: {{ready;wdg_present}}, queued...
[1567078112.18][HTST][INF] host test class: '<class 'reset_reason.ResetReasonTest'>'
[1567078112.18][HTST][INF] host test setup() call...
[1567078112.18][HTST][INF] CALLBACKs updated
[1567078112.19][HTST][INF] host test detected: reset_reason
[1567078112.36][GLRM][TXD] {{reason_raw;get}}
[1567078112.39][CONN][INF] found KV pair in stream: {{reason_raw;14000000}}, queued...
[1567078112.56][GLRM][TXD] {{reason;get}}
[1567078112.59][CONN][INF] found KV pair in stream: {{reason;3}}, queued...
[1567078112.77][GLRM][TXD] {{reason;clear}}
[1567078112.78][CONN][INF] found KV pair in stream: {{reason;cleared}}, queued...
[1567078112.96][GLRM][TXD] {{reset;nvic}}
[1567078113.00][CONN][INF] found KV pair in stream: {{reset;ack}}, queued...
[1567078114.17][GLRM][TXD] {{__sync;0}}
[1567078114.19][CONN][RXD] mbedmbedmbedmbedmbedmbedmbedmbed
[1567078114.29][CONN][INF] found KV pair in stream: {{__sync;0}}, queued...
[1567078114.29][HTST][ERR] orphan event in main phase: {{__sync;0}}, timestamp=1567078114.288429
[1567078114.29][CONN][INF] found KV pair in stream: {{__version;1.3.0}}, queued...
[1567078114.29][HTST][ERR] orphan event in main phase: {{__version;1.3.0}}, timestamp=1567078114.288439
[1567078114.29][CONN][INF] found KV pair in stream: {{__timeout;60}}, queued...
[1567078114.29][CONN][INF] found KV pair in stream: {{__host_test_name;reset_reason}}, queued...
[1567078114.29][HTST][ERR] orphan event in main phase: {{__timeout;60}}, timestamp=1567078114.288446
[1567078114.29][HTST][ERR] orphan event in main phase: {{__host_test_name;reset_reason}}, timestamp=1567078114.288453
[1567078114.39][CONN][INF] found KV pair in stream: {{ready;wdg_present}}, queued...
[1567078114.56][GLRM][TXD] {{reason_raw;get}}
[1567078114.59][CONN][INF] found KV pair in stream: {{reason_raw;14000000}}, queued...
[1567078114.77][GLRM][TXD] {{reason;get}}
[1567078114.79][CONN][INF] found KV pair in stream: {{reason;3}}, queued...
[1567078114.96][GLRM][TXD] {{reason;clear}}
[1567078115.00][CONN][INF] found KV pair in stream: {{reason;cleared}}, queued...
[1567078115.00][CONN][INF] received special event '__host_test_finished' value='True', finishing
[1567078115.00][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078115.41][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "PUT /resource/079602210538697A3A7CFFD1/disconnect HTTP/1.1" 200 61
[1567078115.42][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078118.99][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "PUT /resource/079602210538697A3A7CFFD1/release HTTP/1.1" 200 61
[1567078119.00][HTST][INF] Performing software reset.
[1567078119.01][CONN][INF] starting connection process...
[1567078119.01][CONN][INF] notify event queue about extra 60 sec timeout for serial port pooling
[1567078119.01][CONN][INF] initializing global resource mgr listener...
[1567078119.01][HTST][ERR] orphan event in main phase: {{__conn_process_start;1}}, timestamp=1567078119.007746
[1567078119.02][HTST][ERR] orphan event in main phase: {{__timeout;60}}, timestamp=1567078119.008637
[1567078119.20][GLRM][INF] remote resources initialization: remote(host=https://ruka.mbedcloudtesting.com, port=443)
[1567078119.20][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078119.76][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "POST /login HTTP/1.1" 200 273
[1567078119.76][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078120.03][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /socket.io/?token=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VybmFtZSI6InVzZXIiLCJncm91cCI6InVzZXIiLCJpYXQiOjE1NjcwNzgxMTksImV4cCI6MTU2NzY4MjkxOSwianRpIjoiMjYxNWFkMjAtY2E1MC0xMWU5LWI5ZTMtMDdkMTU2NTRhOTkyIn0.KdXFhbblrK_uF1-s6qMPX70fiaXgaqtl4SIruCubexc&EIO=3&transport=polling&t=1567078119758-0 HTTP/1.1" 200 101
[1567078120.35][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078120.41][dummy]socketio unknown event received: info
[1567078120.61][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /this/resource HTTP/1.1" 200 2735
[1567078120.62][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078121.06][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /resource HTTP/1.1" 200 227512
[1567078121.26][GLRM][INF] remote resources count: 164
[1567078121.53][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078121.86][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "GET /resource/079602210538697A3A7CFFD1 HTTP/1.1" 200 1697
[1567078121.86][GLRM][INF] remote resources flashing with 'BUILD/tests/NUCLEO_F429ZI/GCC_ARM/TESTS/mbed_drivers/reset_reason/reset_reason.bin'...
[1567078121.86][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078122.22][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "POST /file HTTP/1.1" 200 248
[1567078143.78][GLRM][INF] opening connection to platform at baudrate='9600'
[1567078143.78][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078144.31][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "PUT /resource/079602210538697A3A7CFFD1/connect HTTP/1.1" 200 75
[1567078144.31][GLRM][INF] remote resources reset...
[1567078147.78][GLRM][TXD] mbedmbedmbedmbedmbedmbedmbedmbedmbedmbed
[1567078147.78][CONN][INF] sending up to 5 __sync packets (specified with --sync=5)
[1567078147.78][CONN][INF] sending preamble '84f4169d-a0f3-4ba8-ade5-35778bfa8951'
[1567078148.01][GLRM][TXD] {{__sync;84f4169d-a0f3-4ba8-ade5-35778bfa8951}}
[1567078148.01][CONN][RXD] mbedmbedmbedmbedmbedmbedmbedmbed
[1567078148.09][CONN][INF] found SYNC in stream: {{__sync;84f4169d-a0f3-4ba8-ade5-35778bfa8951}} it is #0 sent, queued...
[1567078148.10][CONN][INF] found KV pair in stream: {{__version;1.3.0}}, queued...
[1567078148.10][HTST][ERR] orphan event in main phase: {{__sync;84f4169d-a0f3-4ba8-ade5-35778bfa8951}}, timestamp=1567078148.094652
[1567078148.10][CONN][INF] found KV pair in stream: {{__timeout;60}}, queued...
[1567078148.10][HTST][ERR] orphan event in main phase: {{__version;1.3.0}}, timestamp=1567078148.094640
[1567078148.10][HTST][ERR] orphan event in main phase: {{__timeout;60}}, timestamp=1567078148.094644
[1567078148.20][CONN][INF] found KV pair in stream: {{__host_test_name;reset_reason}}, queued...
[1567078148.20][CONN][INF] found KV pair in stream: {{ready;wdg_present}}, queued...
[1567078148.20][HTST][ERR] orphan event in main phase: {{__host_test_name;reset_reason}}, timestamp=1567078148.197650
[1567078148.37][GLRM][TXD] {{reason_raw;get}}
[1567078148.39][CONN][INF] found KV pair in stream: {{reason_raw;14000000}}, queued...
[1567078148.40][HTST][INF] TEST FAILED: The raw reset reason is not unique. u'14000000' is already present in set([u'14000000']).
[1567078148.40][HTST][INF] __notify_complete(False)
[1567078148.40][HTST][INF] __exit_event_queue received
[1567078148.40][HTST][INF] test suite run finished after 36.31 sec...
[1567078148.56][GLRM][TXD] {{reason;get}}
[1567078148.56][CONN][INF] received special event '__host_test_finished' value='True', finishing
[1567078148.57][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078148.93][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "PUT /resource/079602210538697A3A7CFFD1/disconnect HTTP/1.1" 200 61
[1567078148.93][urllib3.connectionpool]Starting new HTTPS connection (1): ruka.mbedcloudtesting.com:443
[1567078152.89][urllib3.connectionpool]https://ruka.mbedcloudtesting.com:443 "PUT /resource/079602210538697A3A7CFFD1/release HTTP/1.1" 200 61
[1567078152.90][HTST][INF] CONN exited with code: 0
[1567078152.90][HTST][INF] No events in queue
[1567078152.90][HTST][INF] stopped consuming events
[1567078152.90][HTST][INF] host test result() call skipped, received: False
[1567078152.90][HTST][WRN] missing __exit event from DUT
[1567078152.90][HTST][INF] calling blocking teardown()
[1567078152.90][HTST][INF] teardown() finished
[1567078152.90][HTST][INF] {{result;failure}}
mbedgt: retry mbedhtrun 1/1
mbedgt: ['mbedhtrun', '-m', 'NUCLEO_F429ZI', '-p', 'DUMMY:9600', '-f', u'"BUILD/tests/NUCLEO_F429ZI/GCC_ARM/TESTS/mbed_drivers/reset_reason/reset_reason.bin"', '-e', u'"TESTS/host_tests"', '--grm', 'raas_client:https://ruka.mbedcloudtesting.com:443', '-C', '4', '--sync', '5', '-P', '60'] failed after 1 count
mbedgt: checking for GCOV data...
mbedgt: mbed-host-test-runner: stopped and returned 'FAIL'
mbedgt: test case summary event not found
        no test case report present, assuming test suite to be a single test case!
        test suite: tests-mbed_drivers-reset_reason
        test case: tests-mbed_drivers-reset_reason
mbedgt: test on hardware with target id: DUMMY
mbedgt: test suite 'tests-mbed_drivers-reset_reason' ................................................. FAIL in 70.98 sec
        test case: 'tests-mbed_drivers-reset_reason' ................................................. FAIL in 70.98 sec
mbedgt: test case summary: 0 passes, 1 failure
mbedgt: all tests finished!
mbedgt: shuffle seed: 0.8314859056
mbedgt: test suite report:
| target                | platform_name | test suite                      | result | elapsed_time (sec) | copy_method |
|-----------------------|---------------|---------------------------------|--------|--------------------|-------------|
| NUCLEO_F429ZI-GCC_ARM | NUCLEO_F429ZI | tests-mbed_drivers-reset_reason | FAIL   | 70.98              | default     |
mbedgt: test suite results: 1 FAIL
mbedgt: test case report:
| target                | platform_name | test suite                      | test case                       | passed | failed | result | elapsed_time (sec) |
|-----------------------|---------------|---------------------------------|---------------------------------|--------|--------|--------|--------------------|
| NUCLEO_F429ZI-GCC_ARM | NUCLEO_F429ZI | tests-mbed_drivers-reset_reason | tests-mbed_drivers-reset_reason | 0      | 1      | FAIL   | 70.98              |
mbedgt: test case results: 1 FAIL
mbedgt: completed in 71.49 sec
mbedgt: exited with code 1

Passes locally

:~/ws/mbed-os> mbedgt -n tests-mbed_drivers-reset_reason
mbedgt: greentea test automation tool ver. 1.6.5
mbedgt: using multiple test specifications from current directory!
        using 'BUILD/tests/NUCLEO_F429ZI/GCC_ARM/test_spec.json'
mbedgt: detecting connected mbed-enabled devices...
mbedgt: detected 1 device
mbedgt: processing target 'NUCLEO_F429ZI' toolchain 'GCC_ARM' compatible platforms... (note: switch set to --parallel 1)
mbedgt: test case filter (specified with -n option)
        tests-mbed_drivers-reset_reason
        test filtered in 'tests-mbed_drivers-reset_reason'
mbedgt: running 1 test for platform 'NUCLEO_F429ZI' and toolchain 'GCC_ARM'
mbedgt: mbed-host-test-runner: started
mbedgt: checking for GCOV data...
mbedgt: test case summary event not found
        no test case report present, assuming test suite to be a single test case!
        test suite: tests-mbed_drivers-reset_reason
        test case: tests-mbed_drivers-reset_reason
mbedgt: test on hardware with target id: 079602210B586803341CFEA8
mbedgt: test suite 'tests-mbed_drivers-reset_reason' ................................................. OK in 16.13 sec
        test case: 'tests-mbed_drivers-reset_reason' ................................................. OK in 16.13 sec
mbedgt: test case summary: 1 pass, 0 failures
mbedgt: all tests finished!
mbedgt: shuffle seed: 0.4463017990
mbedgt: test suite report:
| target                | platform_name | test suite                      | result | elapsed_time (sec) | copy_method |
|-----------------------|---------------|---------------------------------|--------|--------------------|-------------|
| NUCLEO_F429ZI-GCC_ARM | NUCLEO_F429ZI | tests-mbed_drivers-reset_reason | OK     | 16.13              | default     |
mbedgt: test suite results: 1 OK
mbedgt: test case report:
| target                | platform_name | test suite                      | test case                       | passed | failed | result | elapsed_time (sec) |
|-----------------------|---------------|---------------------------------|---------------------------------|--------|--------|--------|--------------------|
| NUCLEO_F429ZI-GCC_ARM | NUCLEO_F429ZI | tests-mbed_drivers-reset_reason | tests-mbed_drivers-reset_reason | 1      | 0      | OK     | 16.13              |
mbedgt: test case results: 1 OK
mbedgt: completed in 16.62 sec

mbedgt version

linux-20xk:~/ws/mbed-os> mbedgt --version
1.6.5
@0xc0170

This comment has been minimized.

Copy link
Member

commented Aug 29, 2019

There was an error also in CI, let me restart tests, however as noted above, this is not for 5.14 is it? 5.14 jobs are prioritized.

This reverts commit bda7678.
@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 29, 2019

@0xc0170 Problem solved - I reverted the part where NVStore is set as disabled. Rest should go in. I'll update the description of this PR.

@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 29, 2019

@0xc0170 PR's description updated. Now it should be ready for 5.14. @bulislaw , @JanneKiiskila Fine by you?

@VeijoPesonen VeijoPesonen changed the title Disable NVStore & Enable building feature storage with a bare metal profile Enable building feature storage with a bare metal profile Aug 29, 2019
@VeijoPesonen

This comment has been minimized.

Copy link
Contributor Author

commented Aug 29, 2019

Btw, this is not a breaking change anymore.

@JanneKiiskila

This comment has been minimized.

Copy link
Contributor

commented Aug 29, 2019

@0xc0170 - this is definitely for 5.14, please tag it matchingly.

@0xc0170

This comment has been minimized.

Copy link
Member

commented Aug 29, 2019

CI started

@mbed-ci

This comment has been minimized.

Copy link

commented Aug 29, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 2
Build artifacts

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Aug 29, 2019
@0xc0170 0xc0170 merged commit 9203c88 into ARMmbed:master Aug 29, 2019
25 checks passed
25 checks passed
continuous-integration/jenkins/pr-head This commit looks good
Details
jenkins-ci/build-ARM Success
Details
jenkins-ci/build-GCC_ARM Success
Details
jenkins-ci/build-IAR Success
Details
jenkins-ci/cloud-client-test Success
Details
jenkins-ci/dynamic-memory-usage RTOS ROM(-1556 bytes) RAM(+8 bytes)
Details
jenkins-ci/exporter Success
Details
jenkins-ci/greentea-test Success
Details
jenkins-ci/mbed2-build-ARM Success
Details
jenkins-ci/mbed2-build-GCC_ARM Success
Details
jenkins-ci/mbed2-build-IAR Success
Details
jenkins-ci/unittests Success
Details
travis-ci/astyle Success!
Details
travis-ci/docs Success!
Details
travis-ci/doxy-spellcheck Success!
Details
travis-ci/events Success! Runtime is 8624 cycles.
Details
travis-ci/gitattributestest Success!
Details
travis-ci/include_check Success!
Details
travis-ci/licence_check Success!
Details
travis-ci/littlefs Success! Code size is 8464B.
Details
travis-ci/psa-autogen Success!
Details
travis-ci/tools-py2.7 Success!
Details
travis-ci/tools-py3.5 Success!
Details
travis-ci/tools-py3.6 Success!
Details
travis-ci/tools-py3.7 Success!
Details
@VeijoPesonen VeijoPesonen deleted the VeijoPesonen:disable_nvstore branch Aug 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.