Strip out BOM in minion IDs.

Closes #12296
Remove shadowed id kwarg and make valid and invalid functions private
Don't convert, just remove BOM
Remove chadet dep. Not needed.
pylint fixes
Let boto_elb manage cnames via route53
Followup for route53 in boto_elb
Fix for #13008

This is a somewhat silly fix since everywhere that calls "save_load" does something like:

```
save_load(load['jid'], load)
```
Merge pull request #13234 from jacksontj/develop

Fix for #13008
Update sentry_mod.py

Pass DSN to raven client. fix for issue 13172. Thanks cachedout
Merge pull request #13237 from deuscapturus/develop

Update sentry_mod.py
Merge pull request #13203 from cachedout/12296

Strip BOM in minion ids
Merge pull request #13204 from rallytime/pylints

Remove shadowed id kwarg and make valid and invalid functions private
Merge pull request #13211 from lyft/boto-elb-update-route53

 Let boto_elb manage cnames via route53
Merge pull request #13213 from gtmanfred/develop

Replace already defaults to True
Use correct case for requisites (even in titles)
Revamp the ``watch`` requisite docs
Small clarification fix
Merge pull request #13242 from basepi/requisitesdocs

Clarify the watch requisite docs
fixes #13243