puppetlabs / puppet_operational_dashboards

Apache License 2.0
5 stars 22 forks source link

Revert "Fix collection of postgres metrics on FOSS Puppet" #207

Closed Aaronoftheages closed 4 months ago

Aaronoftheages commented 5 months ago

https://github.com/puppetlabs/puppet_operational_dashboards/pull/206

Previous PR was merged erroneously and acceptance tests did not trigger, reverting and recreating branch to address test failings.

m0dular commented 5 months ago

I think it would be better to add the fixtures in a new PR rather than reverting this. A revert means that the reverted patches won't ever be applied again, so we'd have to rebase or something if we wanted to re-merge them. I opened #208, hopefully the tests pass there.