Update puppet_metadata to ~> 4.0 and voxpupuli-acceptance to ~> 3.0

[archanaserver]

No description provided.

[ekohl]

Looks like an update that starts to require PostgreSQL 13 was done and we don't set that up on EL8 (EL9 defaults to PG 13):

  Notice: /Stage[main]/Pulpcore::Database/Pulpcore::Admin[migrate --noinput]/Exec[pulpcore-manager migrate --noinput]/returns: django.db.utils.NotSupportedError: PostgreSQL 12 or later is required (found 10.23).

So that isn't related to this PR, but something we should address.

[ekohl]

Oh, I know why again: the Hiera hierarchy changed in the major version. It's now https://github.com/voxpupuli/voxpupuli-acceptance?tab=readme-ov-file#hiera so you'll need to move spec/acceptance/hieradata/os. Effectively reverting c16ddb8

[archanaserver]

Looks like an update that starts to require PostgreSQL 13 was done and we don't set that up on EL8 (EL9 defaults to PG 13):
Notice: /Stage[main]/Pulpcore::Database/Pulpcore::Admin[migrate --noinput]/Exec[pulpcore-manager migrate --noinput]/returns: django.db.utils.NotSupportedError: PostgreSQL 12 or later is required (found 10.23).
So that isn't related to this PR, but something we should address.

For the new version of PostgreSQL 13, how we can do that? Do we have to define new variable based on OS release or how you preferred to do this @ekohl ?

[ekohl]

Please also squash it into a single commit since this is a single change.

[ekohl]

Please follow the hierarchy as specified in https://github.com/voxpupuli/voxpupuli-acceptance/?tab=readme-ov-file#hiera. Now it's not in the right path which means it isn't loaded and causing failures.

[ehelms]

LGTM. @ekohl ?

[ekohl]

This is not what I meant. I've already tried telling you in #347 (review) that you need to move https://github.com/theforeman/puppet-pulpcore/blob/master/spec/acceptance/hieradata/os/RedHat/8.yaml because version 3 of voxpupuli-acceptance ends up using a different hierarchy. Rather than overriding the hierarchy to the old style, you should move the file.

In practice what you need to do is exactly the reverse of c16ddb8.

[archanaserver]

@ekohl I am not sure what I was missing out but isn't my this change archanaserver@9f884fb was exactly reverting 4248c03 one?

Also I'm not sure if I did the right thing now, but i did revert this specific c16ddb8 and pushed it and I can see this now "Skipping EOL operating system CentOS 8"

[ekohl]

As discussed elsewhere: c16ddb8 used a dash in RedHat-8.yaml but according to https://github.com/voxpupuli/voxpupuli-acceptance?tab=readme-ov-file#hiera it needs to be a file in the subdirectory (so a slash).

[archanaserver]

@ekohl it is passing all the tests now. thanks!