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)
Fix spelling in raet doc
Only read first line of minion_id, Fixes #17687
Merge pull request #17953 from rallytime/spelling_fix

Fix spelling in raet doc
Merge pull request #17968 from basepi/issue_17687

Only read first line of minion_id
Add explanation of "refresh" arg to saltutil.sync_all
Merge pull request #18061 from terminalmage/sync_all-docstring

Add explanation of "refresh" arg to saltutil.sync_all
Merge branch '2014.7' into merge-forward

Conflicts:
	salt/transport/__init__.py
Merge pull request #18063 from basepi/merge-forward

Merge forward from 2014.7 to develop