[MODULES-1333] Add explicit dependencies for mysql_database and mysql_user types #571
Conversation
mysql_grant has an autorequire()'d dependency on the .my.cnf file used by the provider to talk to the database. I've added this to mysql_database and mysql_user too since logically these also need the file to be in place. I've hit this bug because of a slightly unusual edge case in our own manifests, but I think this fix belongs upstream regardless.
|
any chance to add a test case for those edge cases of yours? |
|
I can add tests here, but the edge case feels sufficiently unusual that it may well not add sufficient value to others for it to be included in the repo (and might serve to confuse instead). We've used our own (non-open) MySQL class for a long time, and have started to use parts of the puppetlabs-mysql class alongside it. In particular we wanted to use the types and providers for managing users, grants etc, but not (for now) the whole |
|
nod |
[MODULES-1333] Add explicit dependencies for mysql_database and mysql_user types
Add an autorequire() dependency on /root/.my.cnf to the mysql_database and mysql_user types.