(#2538) Puppet::Util::Execution.execute should also return exit status

Many of the commit for #23124 also fix #2538, but this fixes other
places that were using $CHILD_STATUS that had called execute().

They are now using the exit status of the command, which closes 2538.

Paired with Andrew Parker <andy@puppetlabs.com>