voxpupuli / puppet-zabbix

Puppet module for creating and maintaining zabbix components with puppet.
https://forge.puppet.com/puppet/zabbix
Apache License 2.0
79 stars 228 forks source link

Allow latest version of the PostgreSQL module #849

Closed teluq-pbrideau closed 1 year ago

teluq-pbrideau commented 1 year ago

Pull Request (PR) description

Update postgresql module dependency to < 9

teluq-pbrideau commented 1 year ago

I don’t think the postgresql module version makes the server.sql file unavailable...

  Notice: /Stage[main]/Zabbix::Database::Postgresql/Exec[zabbix_server_create.sql]/returns: psql: error: server.sql: No such file or directory
  Error: 'cd /usr/share/doc/zabbix-sql-scripts/postgresql/ && if [ -f server.sql.gz ]; then gunzip -f server.sql.gz ; fi && psql -h 'localhost' -U 'zabbix_server' -d 'zabbix_server' -f server.sql && touch /etc/zabbix/.schema.done' returned 1 instead of one of [0]
  Error: /Stage[main]/Zabbix::Database::Postgresql/Exec[zabbix_server_create.sql]/returns: change from 'notrun' to ['0'] failed: 'cd /usr/share/doc/zabbix-sql-scripts/postgresql/ && if [ -f server.sql.gz ]; then gunzip -f server.sql.gz ; fi && psql -h 'localhost' -U 'zabbix_server' -d 'zabbix_server' -f server.sql && touch /etc/zabbix/.schema.done' returned 1 instead of one of [0]

This should probably be fixed elsewhere

edit to add: is probably the issue #838

smortex commented 1 year ago

Just to be sure, can you rebase on top of master to confirm that integration tests pass?