search

voxpupuli / puppet_metadata

The gem intends to provide an abstraction over Puppet's metadata.json file. Its API allow easy iteration over its illogical data structures.
Apache License 2.0
3 stars 13 forks source link

Add Debian 12 #102

Closed evgeni closed 11 months ago

bastelfreak commented 11 months ago

In the past we waited with adding a new OS until puppet provides AIO packages for the platform. I don't mind adding Debian 12 already but then we need to ensure that we use the distro packages instead.

evgeni commented 11 months ago

In the past we waited with adding a new OS until puppet provides AIO packages for the platform. I don't mind adding Debian 12 already but then we need to ensure that we use the distro packages instead.

They still… Okay…

How do I tell it to use distro packages, is that even in this repo? Do I really want to know?

ekohl commented 11 months ago

It is in this repository: https://github.com/voxpupuli/puppet_metadata/commit/0dcad5388994a21805a3eaa5446bf61efc716ed0 is how it was done for Arch Linux.

It was in my wish list to support this at some point. We probably need some special casing here:

https://github.com/voxpupuli/puppet_metadata/blob/5ffbf7905b5b9ab1be5cf39d94291ba7b0f7a409/lib/puppet_metadata/github_actions.rb#L63-L67

We probably need something for Fedora too there.

ekohl commented 11 months ago

We probably need something for Fedora too there.

https://github.com/voxpupuli/puppet_metadata/pull/104 does that. And I've taken the liberty of taking your PR and rebase it on top of it, doing the same for Debian 12.

ekohl commented 11 months ago

Testing it out in https://github.com/voxpupuli/puppet-example/pull/37.

ekohl commented 11 months ago

It includes https://github.com/voxpupuli/puppet_metadata/pull/104 but Fedora actually has an issue where puppet-agent isn't available, just puppet.

ekohl commented 11 months ago

Dependencies to merge & release this: