Bash-related fixes.

[dumol]

Problem

When run directly, the Zabbix backup script works as expected and only the relevant DBs are backed-up. But when run through backupninja, it backups up the entire Zabbix DB.

Solution

Use $BASH_SOURCE instead of $0 to account for instances when the script is invoked by another script, as is the case with backupninja scripts. Otherwise this happens when populating DATA_TABLES from the end section of the script:

+++ DATA_TABLES=()
+++ read line
++++ sed '0,/^__DATA__$/d' /usr/sbin/backupninja
++++ tr -s ' '

Drive-by fix:

• changed the shebang to one that doesn't assume the bash binary to be available in /bin, which is not the case for the BSDs and some other Unixes.

[endersonmaia]

The realpath programa is not installed by default in Ubuntu 12.04 nor 14.04.

I think a mention to this requirement is made necessary, or maybe another solution.

[dumol]

It seems you are right. Perhaps DUMPDIR should be initiated with .? Can't see any reason for the current, seemingly over-engineered, situation.

I suggest you try your hand at a pull request for this problem. Hope your PR will be luckier than this one… ;-]

[jva5007]

Hi,

I got the following error while running the backup script.

"ERROR: The number of large data tables configured in this script is less than 5."

Zabbix server version is 2.2.8.

Please let me know how can I fix this error.

Thank You.

[maxhq]

@dumol I am very sorry for the (almost) 1st anniversary of your pull request. I haven't been working with Zabbix for a long time. Thank you very much for your changes though!

[dumol]

Thanks anyway! Perhaps next time I'll have changes to your script I'll fork it and continue to maintain it… ;-]