Add clarification surrounding when the .rules file is copied & where it goes

[wbh1]

This PR adds a debug statement to warn users that their rules configuration won't be copied if they didn't provide a prometheus_alertmanager_config value but DID provide a prometheus_alert_rules value in defaults/main.yml

Also updates the README to reflect that the .rules file gets saved to ansible_managed.rules rather than basic.rules

Fixes #148

[paulfantom]

It might be better to move this to preflight.yml and fail the execution.

[wbh1]

@paulfantom Good call. I've updated it accordingly. Let me know what you think.

[paulfantom]

There is one more thing. By default this role creates some alerting rules but it doesn't set any alertmanager config, so this PR in current form will break default test scenario. We have two choices here:

1. comment-out default value of prometheus_alert_rules
2. revert to debug option and only notify user about lacking alertmanager section

I have no strong opinions on either, but I think that your initial concept is a better way.

[wbh1]

@paulfantom -- I've reverted back to using debug instead of fail :)

The build failed again, but the output seems to indicate that it's unrelated to my changes. Let me know if there's anything else that needs to be altered.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.