(FACT-1416) Use the -c flag when calling zoneadm for Solaris zone facts

[whopper]

Previously, Facter called zoneadm using only the -p flag when
querying the system for zones, which excluded non-running zones.
This is a breaking change from Facter 2, where Facter reported
all zones.

This commit updates the zone resolver to use the -c flag when
calling zoneadm, as we did in Facter 2. In addition, the zone
regex has been modified to match a '-' at the beginning of a
zone entry, as is the output for non-running zones.

[kylog]

LGTM.

[puppetcla]

CLA signed by all contributors.

[whopper]

root@ns9ptobpve8wrtu:~# zoneadm list -cp
0:global:running:/::solaris:shared:-:none
-:testzone:configured:/zones/testzone::solaris:excl:-:
-:testzone2:configured:/zones/testzone2::solaris:excl:-:

Output with Facter 2:

timezone => PDT
zone_global_brand => solaris
zone_global_id => 0
zone_global_iptype => shared
zone_global_name => global
zone_global_path => /
zone_global_status => running
zone_global_uuid =>
zone_testzone2_brand => solaris
zone_testzone2_id => -
zone_testzone2_iptype => excl
zone_testzone2_name => testzone2
zone_testzone2_path => /zones/testzone2
zone_testzone2_status => configured
zone_testzone2_uuid =>
zone_testzone_brand => solaris
zone_testzone_id => -
zone_testzone_iptype => excl
zone_testzone_name => testzone
zone_testzone_path => /zones/testzone
zone_testzone_status => configured
zone_testzone_uuid =>

With this change in Facter 3:

root@jnhqf8ibhpj9098:~# facter solaris_zones
{
  current => "global",
  zones => {
    global => {
      brand => "solaris",
      id => "0",
      ip_type => "shared",
      path => "/",
      status => "running"
    },
    testzone => {
      brand => "solaris",
      id => "-",
      ip_type => "excl",
      path => "/zones/testzone",
      status => "configured"
    },
    testzone2 => {
      brand => "solaris",
      id => "-",
      ip_type => "excl",
      path => "/zones/testzone2",
      status => "configured"
    }
  }
}