fix file ownerships

[ceriath]

Pull Request (PR) description

Currently most exporters have weird permissions after being installed (e.g. 3434:3434 ownership). For the exporters installed via daemon, there is already a patch applied, which sets the owner and group to root:0 for the executables, which still leaves files like "LICENSE" etc. with the wrong ownerships. I am adding a patch for all files, replacing the old patch.

This Pull Request (PR) fixes the following issues

Fixes #111

[baurmatt]

I could only find files like LICENSE, NOTICE, README.md with weird permissions. Those files are irrelevant to the actual exporter functionality. As fixing those permissions requires dirty exec workarounds, could you please elaborate why you think this is important?

[baurmatt]

I've talked about this with @kubicgruenfeld. The best solution for this seems to be the user/group parameter of the archive resource:

-> https://github.com/voxpupuli/puppet-prometheus/blob/873ce7842554b98bfe3e286b1623f6672d637a53/manifests/daemon.pp
-> https://github.com/voxpupuli/puppet-archive/blob/master/REFERENCE.md#user
-> https://github.com/voxpupuli/puppet-archive/blob/master/REFERENCE.md#group

e.g.

archive { "/opt/${name}-${version}.${os}-${arch}/${name}":
  user  => 'root',
  group => 0,
}

Could you please test if this works? :)

[vox-pupuli-tasks]

Dear @ceriath, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks