Re-enable power management support. #216
Conversation
|
Works fine and the code is clean. What would be great though is a spread test to verify that a VM actually shuts down or reboots. Test VMs can either run a classic 16.04 or an all-snap OS. I think this could be doable by looking into tests lie https://github.com/snapcore/snapd/blob/master/tests/main/ubuntu-core-reboot/task.yaml or https://github.com/snapcore/spread/tree/master/tests/reboot |
|
I think that we should also add a bug somewhere so that Jamie can make snapweb autoconnect to shutdown right? |
justincan
force-pushed
the
power-management
branch
from
adc67be
to
e5541cd
Compare
Using shutdown interface on snapd.
justincan
force-pushed
the
power-management
branch
from
e5541cd
to
c5470cd
Compare
|
@AlexandreAbreu Do you know where that is, or is a request on the store? |
|
Ok, i filled https://bugs.launchpad.net/snapweb/+bug/1681693 to track the request to auto-connect the 'shutdown' interface. |
|
For reference, @justincan is proposing to enhance reboot/shutdown support in the spread test system, to implement a more realistic integration test. |
|
Not blocking on tests further. I re-tested the code and will now merge that in. |
Using shutdown interface on snapd.