Revert a few dict usage cleanups from #17080

Turns out our test coverage is missing a few things, upon closer inspection that change broke a few things that we don't seem to have test coverage for.
Merge branch '2014.7' of github.com:saltstack/salt into 2014.7
Add tests for SREQ class

revert keys() change to socket dicts since we will mutate the dict on iteration
remove comment from local debugging
Pylint fixes
Fix error in lvcreate (pv=None)
Re-enable setuptools `console_scripts`
factorize call
Merge pull request #17691 from makinacorpus/7

factorize call
Merge pull request #17305 from jacksontj/2014.7

2014.7
Merge pull request #17682 from fessoga5/2014.7

Fix error in lvcreate (pv=None)
integration test: known JID through netapi #15763
pre-emptive pylint fix0r
ipaddr: remove now-redundant&unnecessary import
Detect chroot virtualization subtype

Add a check against PID 1 root and current root for chrooted environment.
Fix spelling in raet doc
need initializer for ret incase no minions return
removed salt_call as a share destination. Now that salt-call with raet with standup minion
is owrking looks like don't need separate destination
Only read first line of minion_id, Fixes #17687
Merge pull request #17953 from rallytime/spelling_fix

Fix spelling in raet doc