inactive domains are not listed (virt_domains_inactive)

[sls83]

Hello carlasouza,

first, thank you for this module ! :o)
The puppet fact "virt_domains_inactive" does not list inactive kvm-domains.
I guess the search fail, because inactive domains do not have an "domid"

See here:

root@kvm-host:~# virsh list --inactive
 Id    Name                           Status
----------------------------------------------------
 -     VM_01                 switched-off
 -     VM_02                   switched-off
 -     VM_03                     switched-off

I`ve changed the search-base to "name", please see my diff from libvirt.rb:

<       libvirt_connect.list_defined_domains.each do |domid|
<         domains.concat([ libvirt_connect.lookup_domain_by_id(domid.to_i).name ])
<       end

---
>       libvirt_connect.list_defined_domains.each do |name|
>       domains.concat([ name ])
>     end

For my purposes this change works:

facter --puppet | grep virt_domains_inactive
virt_domains_inactive => VM_01,VM_02,VM_03

There are also other inactive objects in libvirt that have no ID,
but these should be recognized by this custom fact.

To reflect this there is a need to change from ID to Name at several places.

Best regards,
Stefan L.-S.