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
salt/modules/file.py: rm trailing whitespace

************* Module salt.modules.file
salt/modules/file.py:953: [C0303(trailing-whitespace), ] Trailing whitespace
Merge pull request #13229 from cvrebert/patch-1

salt/modules/file.py: rm trailing whitespace
Fix for #13008

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

```
save_load(load['jid'], load)
```
Fix white-space related lint issues
Replace already defaults to True

This allows users to pass replace: False to file.recurse.

closes #13212
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)