mir_demo_server --test-client [mir_demo_client_scroll|mir_demo_client_flicker] fails
Bug #1556205 reported by
Alan Griffiths
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
Low
|
Alan Griffiths | ||
mir (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
$ bin/mir_demo_server --test-timeout 3 --test-client bin/mir_
...
Connected
Surface created
[2016-03-11 17:19:54.463733] server_
[2016-03-11 17:19:54.463779] mirserver: Stopping
FAILED
$ bin/mir_demo_server --test-timeout 3 --test-client bin/mir_
...
Connected
Mir chose pixel format 3
Surface created
[2016-03-11 17:21:51.921994] server_
[2016-03-11 17:21:51.922045] mirserver: Stopping
FAILED
I suspect these clients ignore SIGTERM
Related branches
lp://qastaging/~alan-griffiths/mir/fix-1556205
- Mir CI Bot: Needs Fixing (continuous-integration)
- Daniel van Vugt: Approve
- Cemil Azizoglu (community): Approve
-
Diff: 30 lines (+13/-0)1 file modifiedexamples/scroll.cpp (+13/-0)
lp://qastaging/mir
- PS Jenkins bot (community): Approve (continuous-integration)
- Alan Griffiths: Approve
- Kevin DuBois (community): Approve
-
Diff: 1373 lines (+579/-161)23 files modified3rd_party/android-input/android/frameworks/base/services/input/MirLog.cpp (+157/-5)
include/test/mir_test_framework/display_server_test_fixture.h (+2/-0)
include/test/mir_test_framework/stub_client_connection_configuration.h (+35/-0)
include/test/mir_test_framework/testing_client_configuration.h (+43/-0)
include/test/mir_test_framework/testing_process_manager.h (+7/-8)
src/client/mir_client_library.cpp (+39/-17)
src/server/graphics/android/hwc_common_device.cpp (+15/-13)
src/server/options/program_option.cpp (+2/-1)
tests/acceptance-tests/test_client_authorization.cpp (+7/-0)
tests/acceptance-tests/test_client_focus_notification.cpp (+2/-9)
tests/acceptance-tests/test_client_input.cpp (+8/-15)
tests/acceptance-tests/test_client_library.cpp (+30/-17)
tests/acceptance-tests/test_focus_selection.cpp (+2/-9)
tests/acceptance-tests/test_server_shutdown.cpp (+2/-9)
tests/acceptance-tests/test_shell_control_of_surface_configuration.cpp (+2/-9)
tests/acceptance-tests/test_surfaceloop.cpp (+3/-10)
tests/acceptance-tests/test_surfaces_with_output_id.cpp (+2/-9)
tests/mir_test_framework/CMakeLists.txt (+1/-0)
tests/mir_test_framework/display_server_test_fixture.cpp (+9/-6)
tests/mir_test_framework/testing_client_options.cpp (+126/-0)
tests/mir_test_framework/testing_process_manager.cpp (+41/-1)
tests/unit-tests/graphics/android/test_hwc_device.cpp (+25/-1)
tests/unit-tests/logging/test_legacy_input_report.cpp (+19/-22)
Changed in mir: | |
importance: | Undecided → Low |
Changed in mir: | |
assignee: | nobody → Alan Griffiths (alan-griffiths) |
status: | New → In Progress |
Changed in mir: | |
milestone: | none → 0.21.0 |
Changed in mir: | |
status: | In Progress → Fix Committed |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Released → Fix Committed |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Released → Fix Committed |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir: | |
status: | Fix Released → Fix Committed |
Changed in mir: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
The main reason for caring is these prevent us automatically running all the clients in CI