docker run helper script #6270

Merged
merged 1 commit into from Jan 8, 2017

Projects

None yet

2 participants

@dagar
Member
dagar commented Jan 7, 2017

This is a script to make running docker a little easier. I previously tried to incorporate it directly in the Makefile, but in practice that had some issues.

./Tools/docker_run.sh 'make tests'

@LorenzMeier @AndreasAntener FYI the recently updated mixer_test fails when built with -O0, which is what was happening in #6245 the other day. For now I've just increased the optimization level for the code coverage build.

@dagar
Member
dagar commented Jan 7, 2017

Mixer failing intermittently - https://circleci.com/gh/PX4/Firmware/2741
3 successful builds, 1 failure

INFO  [tests] RUNNING TEST: loadIOPass
INFO  [tests] TEST PASSED: loadIOPass
INFO  [tests] RUNNING TEST: loadQuadTest
ERROR [modules__unit_test] Compare failed: check number of mixers loaded - (mixer_group.count():4) (expected_count:5) (../src/systemcmds/tests/test_mixer.cpp:353)
ERROR [tests] Mixer load failed with chunk size 58
ERROR [tests] TEST FAILED: loadQuadTest
INFO  [tests] RUNNING TEST: loadVTOL1Test
ERROR [modules__unit_test] Compare failed: check number of mixers loaded - (mixer_group.count():3) (expected_count:4) (../src/systemcmds/tests/test_mixer.cpp:353)
ERROR [tests] Mixer load failed with chunk size 61
ERROR [tests] TEST FAILED: loadVTOL1Test
INFO  [tests] RUNNING TEST: loadVTOL2Test
ERROR [modules__unit_test] Compare failed: check number of mixers loaded - (mixer_group.count():5) (expected_count:6) (../src/systemcmds/tests/test_mixer.cpp:353)
ERROR [tests] Mixer load failed with chunk size 61
ERROR [tests] TEST FAILED: loadVTOL2Test
INFO  [tests] RUNNING TEST: loadComplexTest
INFO  [tests] TEST PASSED: loadComplexTest
INFO  [tests] RUNNING TEST: loadAllTest
INFO  [tests] Testing all mixers in ROMFS/px4fmu_common/mixers
INFO  [tests] TEST PASSED: loadAllTest
INFO  [tests] RUNNING TEST: mixerTest
INFO  [tests] TEST PASSED: mixerTest
ERROR [modules__unit_test] SOME TESTS FAILED
INFO  [modules__unit_test]   Tests passed :      4
INFO  [modules__unit_test]   Tests failed :      3
INFO  [modules__unit_test]   Tested assertions : 4268
mixer FAILED
Tools/docker_run.sh
+ -v ${SRC_DIR}:${SRC_DIR}:rw \
+ -v /tmp/.X11-unix:/tmp/.X11-unix:ro \
+ -v ${HOME}/.ccache:${HOME}/.ccache:rw \
+ -e CCACHE_DIR=${HOME}/.ccache \
@AndreasAntener
AndreasAntener Jan 7, 2017 Member

@dagar this reminds me, I have trouble with the current containers compiling because the default cache directory is owned by root.

@dagar
Member
dagar commented Jan 7, 2017

Moving the code coverage/mixer test optimization problem to another issue.

@dagar
Member
dagar commented Jan 7, 2017

I'll work on restoring the code coverage build and mixer test failure here - #6273

@dagar dagar requested a review from AndreasAntener Jan 7, 2017
@dagar dagar docker run helper script
eca1512
@dagar dagar merged commit defaf7f into master Jan 8, 2017

5 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/sitl/pr Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
semaphoreci The build passed on Semaphore.
Details
@dagar dagar deleted the docker_wrapper branch Jan 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment