cirrax / puppet-libvirt

puppet module for libvirt
GNU General Public License v3.0
8 stars 25 forks source link

Make defaults from Ubuntu 18.04 new global Ubuntu defaults #45

Closed raoulbhatia closed 3 years ago

raoulbhatia commented 3 years ago
git mv Ubuntu.yaml Ubuntu-16.04.yaml
git mv Ubuntu-18.04.yaml Ubuntu.yaml
codecov-io commented 3 years ago

Codecov Report

Merging #45 (188ed31) into master (8d1fae6) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #45   +/-   ##
======================================
  Coverage    1.62%   1.62%           
======================================
  Files          10      10           
  Lines         430     430           
======================================
  Hits            7       7           
  Misses        423     423           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8d1fae6...188ed31. Read the comment docs.

trefzer commented 3 years ago

Hi Thanks for you're request. I basically agree, but 2 things: First, according to metadata.json, we support: { "operatingsystem": "Ubuntu", "operatingsystemrelease": [ "14.04", "16.04", "17.10", "18.04" ] }, So we should duplicate the original config also to the other older versions.

Second, why do you delete libvirt::qemu_hook_packages in the default ubuntu.yaml ?

raoulbhatia commented 3 years ago

Thanks for the review.

Ad 1. Supported Releases: I pushed additional commits that I hope will cover this: 0e1ff5e and 188ed31

Ad 2. libvirt::qemu_hook_packages: I was creating my fix in a rush. I pushed 3f1cef4 to fix this.

Please let me know if this PR looks better now.

Cheers, Raoul

raoulbhatia commented 3 years ago

@trefzer please let me know if you'd like to see any further changes.

trefzer commented 3 years ago

there was only a minor comment I changed. Thanks for the contribution. The patches are now merged and available in version 3.4.0.

raoulbhatia commented 3 years ago

Thanks 🇨🇭

For the records, fixed via a2cfd64118658c59ebb680e7a609bc19d4c03359