server crashes when shutting down in smoke tests
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
High
|
Alan Griffiths | ||
mir (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
The "smoke tests" run each of the demo clients for 3 seconds and ensure they and the server all exit cleanly. But we've seen an instance where this fails:
https:/
05:09:56 Starting
05:09:56 libevdev error in fix_invalid_
05:09:56 Mir chose pixel format 1
05:09:56 Surface created
05:09:59 Signal 15 received. Good night.
05:10:08 /home/phablet/
05:10:08 I: [FAILED] mir_demo_
This looks like the server crashed between signalling the client and shutting down.
All 21 of the other smoke tests pass:
05:11:48 I: The following clients failed to execute successfully:
05:11:48 I: mir_demo_
05:11:48 I: Smoke testing complete with returncode -1
Related branches
- Mir CI Bot: Approve (continuous-integration)
- Daniel van Vugt: Approve
- Brandon Schaefer (community): Approve
- Alberto Aguirre (community): Approve
- Cemil Azizoglu (community): Approve
-
Diff: 301 lines (+32/-29)18 files modifiedinclude/server/mir/scene/session.h (+1/-1)
include/test/mir/test/doubles/stub_session.h (+1/-1)
src/include/server/mir/compositor/buffer_stream.h (+1/-1)
src/server/compositor/buffer_queue.cpp (+7/-8)
src/server/compositor/buffer_queue.h (+3/-1)
src/server/compositor/buffer_stream_surfaces.cpp (+3/-3)
src/server/compositor/buffer_stream_surfaces.h (+1/-1)
src/server/compositor/stream.cpp (+1/-1)
src/server/compositor/stream.h (+1/-1)
src/server/scene/application_session.cpp (+3/-3)
src/server/scene/application_session.h (+1/-1)
src/server/scene/session_manager.cpp (+1/-1)
tests/include/mir/test/doubles/mock_buffer_stream.h (+2/-1)
tests/include/mir/test/doubles/mock_scene_session.h (+2/-1)
tests/include/mir/test/doubles/stub_buffer_stream.h (+1/-1)
tests/integration-tests/compositor/test_buffer_stream.cpp (+1/-1)
tests/mir_test_framework/stub_session.cpp (+1/-1)
tests/unit-tests/compositor/test_buffer_stream.cpp (+1/-1)
Changed in mir: | |
status: | Fix Committed → Fix Released |
The reason I think the server core dumps is that one would expect to see "mirserver: Stopping" one second after "Signal 15 received. Good night".