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

Support sqLite db for proxies #6

Closed
jrg72 opened this issue Aug 7, 2014 · 2 comments
Closed

Support sqLite db for proxies #6

jrg72 opened this issue Aug 7, 2014 · 2 comments

Comments

@jrg72
Copy link

jrg72 commented Aug 7, 2014

Hello, and thank you for this Puppet module, it's quite excellent. I have run into an issue with proxy configuration, however. Proxies support use of sqLite for the dbtype in Zabbix, and it looks like there is mention of this in your docs also:

https://github.com/dj-wasabi/puppet-zabbix/blob/master/manifests/proxy.pp#L11

[dbtype]

Type of database. Can use the following 3 databases:

- postgresql

- mysql

- sqlite

However, setting the dbtype to `sqlite' seems to fail with the following error:

Error: Unrecognized database type for proxy: sqlite at /etc/puppet/modules/zabbix/manifests/proxy.pp:295

Farther down in the proxy manifest, it looks like there is no case switch to set `sqlite' as an option for the dbtype:

https://github.com/dj-wasabi/puppet-zabbix/blob/master/manifests/proxy.pp#L286

Use the correct db.

case $dbtype {
'postgresql': {
$db = 'pgsql'
}
'mysql': {
$db = 'mysql'
}
default: {
fail("Unrecognized database type for proxy: ${dbtype}")
}
}

Is this an option that can be fully implemented at some point? Thank you.

@dj-wasabi
Copy link
Contributor

Hi jrg72,

Thanks for making time for creating an issue. At the beginning, the plan was to create support for this type of database with the proxy and server classes. But lateron I thought that sqlite3 wasn't an popular choice as backend, so I didn't implemented it.

At this moment, I'm busy with some zabbixapi stuff and when this is done, I'll take a look at it. I can't say when this will happen tough. Why would you use sqlite3 for this if I may ask? I'm not an database engineer, but using MySQL or Postgresql isn't that hard.

Kind regards,
Werner

@jrg72
Copy link
Author

jrg72 commented Aug 29, 2014

Hello again Werner,

Thanks for the response! We ended up just switching to MySQL - I think sqLite was being used for simplicity's sake, but it's no big deal to just use MySQL instead.

Thanks again,

Joe

@jrg72 jrg72 closed this as completed Aug 29, 2014
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

2 participants