Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zabbix 4.0 support #104

Open
q1x opened this issue Jan 14, 2019 · 3 comments
Open

Zabbix 4.0 support #104

q1x opened this issue Jan 14, 2019 · 3 comments

Comments

@q1x
Copy link

q1x commented Jan 14, 2019

Hi all,

I'd like to add support for the Zabbix 4.0.x versions as this is the new LTS release.
However, I don't know the procedure that was used to generate the mysql create files that are shipped in this formula (Zabbix sources contain 3 separate files).

Does anyone have a step-by-step handy?

@hatifnatt
Copy link
Collaborator

hatifnatt commented Jan 18, 2019

First of all I think it's better to stop providing SQL files with formula. Since Zabbix 3.0 Zabbix package provide gzipped SQL file with schema and data, IMO it's better to just use this file. PostgreSQL part of this formula already using this approach.

If you need to concatenate 3 SQL files, schema file must be first, concatenation can be done with simple cat

cat schema.sql data.sql images.sql > create.sql

To use your own sql file.

  1. Put your file into folder /salt/fileserver/root/dir/zabbix/files/default/zabbix4.sql on your salt master
  2. Add pillar
zabbix-mysql:
  sql_file: /zabbix4.sql

To install Zabbix 4 - add pillar

zabbix:
  lookup:
    version_repo: '4.0'

@xenadmin
Copy link
Contributor

First of all I think it's better to stop providing SQL files with formula. Since Zabbix 3.0 Zabbix package provide gzipped SQL file with schema and data, IMO it's better to just use this file. PostgreSQL part of this formula already using this approach.

+1 for this.
Instead we should maybe think about changing the Version default from 2.2 to a more recent version that is actually supported.

@myii
Copy link
Member

myii commented Mar 31, 2020

Should be solved by #123 once the implementation is resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants