Bert Van Vreckem

Thanks for catching this! The code is becoming so complex that it's easy to miss these mistakes...

Ok, so that would mean a separate variable `bind_allow_transfer` which is a list just like `bind_allow_query` and maybe a boolean `bind_allow_transfer_to_acls` that, when true, implements the curren behaviour? Does that...

Ok, that makes sense, I think I understand now!

`bind_zone_dir` is defined in vars, not in defaults. Consequently, it cannot be changed by your own playbook. Would your issue be resolved if we moved it to defaults, or if...

Very sorry, but I don't understand your question. It's a bit too concise. Could you explain in a bit more detail? What is the expected behaviour? Remark that managing logrotate...

These changes cause ansible-lint warnings. Can you look into this? You make use of the `shell:` module, which can break idempotency of this role. Could you look into this? Often,...

Seems like a useful addition, thanks! Can you update the README with some documentation?

Can you give a concrete example? The handler that restarts the service is only called on configuration file changes. I can't imagine that a server restart is not necessary in...

Thank you! ansible-lint indicates several issues, can you check these? See the [Travis-CI build log](https://travis-ci.org/github/bertvv/ansible-role-mariadb/jobs/700306658)

The Travis CI build fails, after applying the role, the database server does not seem to be available to external users. Can you check this? Also, I have a few...