acpi_sleep_attachement fails everytime

Bug #1316695 reported by Chris Gregan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
Medium
Unassigned

Bug Description

Build Version/Date: 1.9
Environment used for testing: Trusty

Summary: ss
ACPI sleep attachment fails each time without posting any error to stdout for analysis

Steps to Reproduce:
1) Run any suite

Expected result:
All attachments pass

Actual result:

Related branches

Chris Gregan (cgregan)
Changed in cdts:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Daniel Manrique (roadmr)
Revision history for this message
Daniel Manrique (roadmr) wrote :

plugin: attachment
id: acpi_sleep_attachment
command: [ -e /proc/acpi/sleep ] && cat /proc/acpi/sleep
estimated_duration: 0.5
_description: Attaches the contents of /proc/acpi/sleep if it exists.

The description lacks "if it does not exist, exit with error", which is the behavior of the command as is.

The command can be changed to

[ -e /proc/acpi/sleep ] && cat /proc/acpi/sleep || echo "No /proc/acpi/sleep found"

this way it will always pass, but we at least will know when the file is not present. I don't think its absence should constitute a failure, as systems even without it work and can sleep just fine.

OTOH, if /proc/acpi/sleep is no longer used for some reason, we should research why, when did it disappear, and whether we still care about it; if we no longer do, we should just remove the job.

Changed in cdts:
status: Confirmed → Triaged
Chris Gregan (cgregan)
information type: Proprietary → Public
Daniel Manrique (roadmr)
Changed in plainbox-provider-checkbox:
importance: Undecided → Medium
status: New → Triaged
Daniel Manrique (roadmr)
Changed in cdts:
status: Triaged → In Progress
Changed in plainbox-provider-checkbox:
status: Triaged → In Progress
Chris Gregan (cgregan)
Changed in cdts:
milestone: none → version2.0
Daniel Manrique (roadmr)
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Released
Changed in cdts:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.