Code review comment for lp:~a.skwar/mylvmbackup/zfs

Where are you backing up InnoDB data? You keep referring to DataDir but that is only for MyISAM.

Here is how we layout our customers:

basedir = /usr
socket = /var/lib/mysql/mysql.sock
tmpdir = /var/lib/mysql/tmp
datadir = /var/lib/mysql/data
log-error = /var/lib/mysql/log/error.log
log-bin = /var/lib/mysql/binlog/binlog
relay-log = /var/lib/mysql/relaylog/relaylog
innodb-data-home-dir = /var/lib/mysql/innodb
innodb-log-group-home-dir = /var/lib/mysql/innodb

If we only back'd up the datadir, we'd be missing out on tons of data necessary to completely rebuild a customer's database from the ground up.