sous-chefs / mysql

Development repository for the mysql cookbook
https://supermarket.chef.io/cookbooks/mysql
Apache License 2.0
337 stars 691 forks source link

Add dependency on apparmor cookbook #662

Closed ramereth closed 3 years ago

ramereth commented 3 years ago

This fixes #660.

We test for node['apparmor']['disable'] in libraries/mysql_service_base.rb and fails on systems which doesn't include the apparmor cookbook. If we're depending on this attribute, we should also be depending on the cookbook which uses it, so this adds that dependency.

In addition, update the comment in the test for why we disable apparmor for our integration tests.

Signed-off-by: Lance Albertson lance@osuosl.org

Description

[Describe what this change achieves]

Issues Resolved

[List any existing issues this PR resolves]

Check List

cgunther commented 3 years ago

That fixes it! Thanks for the quick fix.

kitchen-porter commented 3 years ago

Released as: 10.1.0