Specify puppetmaster::gitclone missing ownerships

We need operations/private accessible by puppetmaster (started by
passenger. The hooks however dont need to be accessed by puppetmaster so
specify a reasonable default. While at it, make an effort to cleanup the
puppetmaster module file resources adding owner/group/mode when possible
Also fix a couple of places according to style guide

Change-Id: I17d2ab588ade67f8a98ee357f3367700dd706c6f