Fix #5102
netbsd_sysctl: fix pylint W1401 (Anomalous backslash in string)
cpu_times -> times, to avoid shadowing same-named func
Merge pull request #5108 from cvrebert/patch-5

netbsd_sysctl: fix pylint W1401 (Anomalous backslash in string)
Merge pull request #5113 from hulu/cputimes-shadowing

cpu_times -> times, to avoid shadowing same-named func
clean up zpool and fix #5114
sentry_return: returner.connect_sentry() being called w/ too many args
fix pylint W1401 (Anomalous backslash in string) problems in grains.core
win_pkg mod: actually invoke refresh_db
netbsdservice mod: status -> service_status, to avoid shadowing status() func
win_useradd: info -> user_info, to avoid shadowing info() func
Merge pull request #5115 from hulu/sentry_return

sentry_return: returner.connect_sentry() being called w/ too many args
Merge pull request #5116 from hulu/grains-backslashes

fix pylint W1401 (Anomalous backslash in string) problems in grains.core
Merge pull request #5117 from cvrebert/patch-1

win_pkg mod: actually invoke refresh_db
Merge pull request #5118 from hulu/netbsdservice-shadowing

netbsdservice mod: status -> service_status, to avoid shadowing status()
Merge pull request #5119 from hulu/win_useradd-shadowing

win_useradd: info -> user_info, to avoid shadowing info() func
Fix #5121
file state: refactor to avoid shadowing comment() func; optimization too
Handle lower msgpack versions which can't handle `OrderedDict`'s. Fixes #5017.

Dictionary subclasses were only added to msgpack after 0.2.0(inclusive).
Since `isinstance(OrderedDict(), dict)` returns true, of course, msgpack's custom type handling cannot be used.
The solution is to iterate through the data and convert `OrderedDict`'s to `dict`'s since that's what msgpack also does.
For now, the iteration is a top level one, ie, only first child descendants are evaluated. We won't go deeper unless we get the same failure a little deeper in the nested data.
grains/core : fix fqdn / domain return on SunOS systems

getfqdn() on Solaris platforms returns the unqualified hostname. In
that case, using gethostname() could be a solution.

Tested on SmartOS / Solaris 11