Skip to content

Cellular: Disable Greentea tracing to consume less memory #7182

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
Jun 12, 2018

Conversation

AriParkkila
Copy link

Description

Cellular Greentea tests were failing with UBLOX_C027 target + IAR toolchain due to out-of-memory. This change disables tracing while running Greentea tests to consume less memory.

Fixes ARM internal ref IOTCELL-1082.

Pull request type

[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

@AriParkkila
Copy link
Author

@mirelachirica @jarvte @kimlep01 please review

Copy link
Contributor

@kjbracey kjbracey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sadly, tracing can occasionally break Greentea comms in general - it's not totally robust, unlike Icetea. Might want to do this more generally if any other of your tests have tracing enabled.

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 11, 2018

/morph build

Copy link

@kimlep01 kimlep01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are passing now in CI

@mbed-ci
Copy link

mbed-ci commented Jun 11, 2018

Build : SUCCESS

Build number : 2315
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7182/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Jun 11, 2018

@mbed-ci
Copy link

mbed-ci commented Jun 11, 2018

@cmonr cmonr merged commit af55378 into ARMmbed:master Jun 12, 2018
@0xc0170 0xc0170 removed the needs: CI label Jun 12, 2018
@AriParkkila AriParkkila deleted the cellular-greentea-fix branch September 10, 2018 08:26
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.

7 participants