audio_settings before/after suspend test is too sensitive to minor changes in alsa mixer settings
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox |
Fix Released
|
Medium
|
Brendan Donegan |
Bug Description
The current test for checking audio settings before and after suspend works by diff'ing the output of alsa_mixer between suspends cycles. Unfortunately this is very flaky and there will end up being a lot of failures which are of little consequence, such as below:
5c5 < Mono: Playback 40 [62%] [-18.00dB] [on] --- > Mono: Playback 40 [62%] [-18.00dB] [off] 11,12c11,12 < Front Left: Playback 64 [100%] [0.00dB] [on] < Front Right: Playback 64 [100%] [0.00dB] [on] --- > Front Left: Playback 64 [100%] [0.00dB] [off] > Front Right: Playback 64 [100%] [0.00dB] [off] 18,19c18,19 < Front Left: Playback 64 [100%] [0.00dB] [on] < Front Right: Playback 64 [100%] [0.00dB] [on] --- > Front Left: Playback 64 [100%] [0.00dB] [off] > Front Right: Playback 64 [100%] [0.00dB] [off]
It's very difficult to parse this failure. A better solution would be to use the 'store' function of the audio_settings script to dump the basic settings such as the currently selected source and sink and the volume level and mute setting on those. If those change between suspends, we can say there is definitely a problem.
Related branches
- Zygmunt Krynicki (community): Approve
-
Diff: 37 lines (+7/-2)2 files modifieddebian/changelog (+4/-0)
jobs/suspend.txt.in (+3/-2)
- Ubuntu branches: Pending requested
-
Diff: 114866 lines (+22292/-18992)125 files modifiedcheckbox_qt/qt_interface.py (+7/-2)
debian/changelog (+51/-0)
debian/po/ast.po (+2/-2)
debian/po/cs.po (+2/-2)
debian/po/de.po (+2/-2)
debian/po/en_AU.po (+2/-2)
debian/po/en_GB.po (+2/-2)
debian/po/es.po (+2/-2)
debian/po/fr.po (+2/-2)
debian/po/he.po (+2/-2)
debian/po/hu.po (+2/-2)
debian/po/id.po (+2/-2)
debian/po/it.po (+2/-2)
debian/po/ja.po (+2/-2)
debian/po/nl.po (+2/-2)
debian/po/oc.po (+2/-2)
debian/po/pl.po (+2/-2)
debian/po/pt_BR.po (+2/-2)
debian/po/ro.po (+2/-2)
debian/po/ru.po (+2/-2)
debian/po/tr.po (+2/-2)
debian/po/uk.po (+2/-2)
debian/po/zh_CN.po (+2/-2)
debian/po/zh_TW.po (+2/-2)
jobs/audio.txt.in (+6/-6)
jobs/networking.txt.in (+4/-1)
jobs/suspend.txt.in (+5/-4)
patches/0.14.2 (+2/-1)
po/ace.po (+199/-198)
po/af.po (+199/-198)
po/am.po (+199/-198)
po/ar.po (+219/-203)
po/ast.po (+199/-198)
po/az.po (+199/-198)
po/be.po (+199/-198)
po/bg.po (+199/-198)
po/bn.po (+199/-198)
po/bo.po (+199/-198)
po/br.po (+199/-198)
po/bs.po (+199/-198)
po/ca.po (+199/-198)
po/ca@valencia.po (+199/-198)
po/ckb.po (+199/-198)
po/cs.po (+199/-198)
po/cy.po (+199/-198)
po/da.po (+199/-198)
po/de.po (+179/-178)
po/dv.po (+199/-198)
po/el.po (+199/-198)
po/en_AU.po (+415/-414)
po/en_CA.po (+199/-198)
po/en_GB.po (+891/-504)
po/eo.po (+202/-201)
po/es.po (+462/-216)
po/et.po (+199/-198)
po/eu.po (+199/-198)
po/fa.po (+199/-198)
po/fi.po (+202/-201)
po/fr.po (+1576/-489)
po/ga.po (+199/-198)
po/gd.po (+199/-198)
po/gl.po (+261/-208)
po/he.po (+199/-198)
po/hi.po (+199/-198)
po/hr.po (+199/-198)
po/hu.po (+199/-198)
po/hy.po (+199/-198)
po/id.po (+199/-198)
po/is.po (+199/-198)
po/it.po (+199/-198)
po/ja.po (+199/-198)
po/jbo.po (+199/-198)
po/ka.po (+199/-198)
po/kk.po (+199/-198)
po/km.po (+199/-198)
po/kn.po (+199/-198)
po/ko.po (+199/-198)
po/ku.po (+199/-198)
po/ky.po (+199/-198)
po/lt.po (+199/-198)
po/lv.po (+199/-198)
po/mk.po (+199/-198)
po/ml.po (+199/-198)
po/mr.po (+199/-198)
po/ms.po (+666/-251)
po/my.po (+199/-198)
po/nb.po (+199/-198)
po/nds.po (+199/-198)
po/ne.po (+199/-198)
po/nl.po (+199/-198)
po/nn.po (+199/-198)
po/oc.po (+204/-199)
po/pl.po (+199/-198)
po/ps.po (+199/-198)
po/pt.po (+266/-257)
po/pt_BR.po (+626/-236)
po/ro.po (+199/-198)
po/ru.po (+629/-222)
po/sd.po (+199/-198)
po/shn.po (+199/-198)
po/si.po (+199/-198)
po/sk.po (+199/-198)
po/sl.po (+288/-225)
po/sq.po (+217/-198)
po/sr.po (+199/-198)
po/sv.po (+199/-198)
po/ta.po (+199/-198)
po/te.po (+199/-198)
po/th.po (+199/-198)
po/tr.po (+204/-203)
po/ug.po (+199/-198)
po/uk.po (+218/-217)
po/ur.po (+199/-198)
po/uz.po (+199/-198)
po/vi.po (+199/-198)
po/zh_CN.po (+199/-198)
po/zh_HK.po (+199/-198)
po/zh_TW.po (+199/-198)
qt/frontend/qtfront.ui (+2/-2)
scripts/audio_settings (+12/-2)
scripts/color_depth_info (+1/-1)
scripts/frequency_governors_test (+5/-2)
scripts/graphics_driver (+4/-4)
scripts/graphics_modes_info (+1/-1)
scripts/network_device_info (+95/-44)
Changed in checkbox: | |
status: | New → In Progress |
assignee: | nobody → Brendan Donegan (brendan-donegan) |
importance: | Undecided → Medium |
Changed in checkbox: | |
status: | In Progress → Fix Released |