search

internap / virtualpdu

A virtual power distribution unit (PDU) for controlling libvirt virtual machines via SNMP commands.
Apache License 2.0
3 stars 6 forks source link

Fixes internap/virtualpdu#30 #31

Closed idjaw closed 7 years ago

idjaw commented 7 years ago

Exception handling has been added to the libvirt driver's power_off feature. This fixes the issue where an uncaught exception would raise when a power_off command would be issued when the PDU is already off.

mat128 commented 7 years ago

Good job, but go to bed now we'll check this Monday

idjaw commented 7 years ago

I was bored. It seemed easy enough. :)

idjaw commented 7 years ago

Agreed about the integration test. I'll work on adding one. In the meantime will close this, so I can collect my other commits without triggering the PR again.