puppetlabs-postgresql icon indicating copy to clipboard operation
puppetlabs-postgresql copied to clipboard

List Debian 12 as supported system

Open deric opened this issue 2 years ago • 6 comments

Summary

Debian 12 has been removed from supported systems (f10ee71a28b7b727d9bffb869ecc1a76d7e98b57). Now since https://github.com/puppetlabs/litmusimage/pull/62 has been merged, there should be a docker image available.

deric avatar Aug 31 '23 08:08 deric

Hi @deric Thanks for raising the PR. I have checked yesterday after taking on https://github.com/puppetlabs/litmusimage/pull/62 and got to know that the Agent is not yet compatible with Debian12, so don't think so make sense to proceed with this change unless we get official support for the Debian 12. But will keep eye on this one and take it forward once we get official support of Debian 12.

Ramesh7 avatar Aug 31 '23 08:08 Ramesh7

Also if you look at https://github.com/puppetlabs/puppetlabs-postgresql/actions/runs/6034903052/job/16374201436?pr=1488 (GitHub CI run) its still skipping Debian12.

Warning: Cannot find image for Debian-12
Screenshot 2023-08-31 at 2 16 24 PM

Ramesh7 avatar Aug 31 '23 08:08 Ramesh7

@Ramesh7 Thanks for looking into this. The job definition is here: https://github.com/puppetlabs/cat-github-actions/blob/main/.github/workflows/module_ci.yml#L47C35-L47C35, right? But where is the method matrix_from_metadata_v2 defined?

deric avatar Aug 31 '23 09:08 deric

The matrix_from_metadata_v2 is part of the puppet_litmus gem. With https://github.com/puppetlabs/puppet_litmus/pull/523/files Debian-12 has been added so this one should now work? Needs a rerun of the workflow to verify.

cruelsmith avatar Sep 07 '23 12:09 cruelsmith

Oh the version bump for the puppet_lintmus gem that adds it is still open puppetlabs/puppet_litmus#524

cruelsmith avatar Sep 07 '23 13:09 cruelsmith

Oh the version bump for the puppet_lintmus gem that adds it is still open puppetlabs/puppet_litmus#524

@cruelsmith @deric wanted to update here that the Debian 12 OS is not yet officially supported by Puppet, so will not recommend to use with Debian12 OS. The PR in litmus to add Debian12 got raised for experimental purpose only.

Ramesh7 avatar Sep 12 '23 04:09 Ramesh7

I rebased this against main

bastelfreak avatar Apr 10 '24 11:04 bastelfreak

This PR also contains #1592

bastelfreak avatar Apr 10 '24 15:04 bastelfreak