foreman_config_entry type does not deal well with array data

[jhoblitt]

The foreman_config_entry type only supports scalar values. Rather amazingly, it seems that it is possible to set an "array" of values if passed a stringified JSON array. However, this appears to not be idempotent:

Notice: /Stage[main]/Profile::Core::Puppet_master/Foreman_config_entry[excluded_facts]/value: value changed '["lo","en*v*","usb*","vnet*","macvtap*",";vdsmdummy;","veth*","tap*","qbr*","qvb*","qvo*","qr-*","qg-*","vlinuxbr*","vovsbr*","br-int","vif*","load_averages::*","memory::swap::available*","memory::swap::capacity","memory::swap::used*","memory::system::available*","memory::system::capacity","memory::system::used*","memoryfree","memoryfree_mb","swapfree","swapfree_mb","uptime_hours","uptime_days"]' to '["lo", "en*v*", "usb*", "vnet*", "macvtap*", ";vdsmdummy;", "veth*", "tap*", "qbr*", "qvb*", "qvo*", "qr-*", "qg-*", "vlinuxbr*", "vovsbr*", "br-int", "vif*", "load_averages::*", "memory::swap::available*", "memory::swap::capacity", "memory::swap::used*", "memory::system::available*", "memory::system::capacity", "memory::system::used*", "memoryfree", "memoryfree_mb", "swapfree", "swapfree_mb", "uptime_hours", "uptime_days"]' (corrective)

It would be more convenient if the foreman_config_entry type directly supported array values.