Fix for #17051
Add regression test for #17051
Add "normalize_ret" function, so we don't have to copy/paste so much

Add checks for "changes" since we want to test cases where there aren't changes to be made
Quotations are not necessary when passing cmd arguments in as an array

- This also fixes the inability to clone mercurial repos on Ubuntu 12.04
Merge pull request #17054 from jacksontj/2014.7

Fix for #17051
adding tests for modules/linux_sysctl.py
change linux_sysctl.py to fail properly on misassign

The original check (4e1c367) for sysctl assign failure queries the
format returned by sysctl on stdout.  It was added evidently for some
container system called Virtuozzo.  A later commit (4e1c367) strengthens
the check into a full regex on sysctl stdout.

Unfortunately, it appears that on a contemporary conventional distro,
CentOS 7, sysctl repeats the config it is given on stdout whether or not
the config requested is valid.  Furthermore, an invalid config request
will not cause sysctl to return a nonzero status.  It does print an
error on stderr, so I've added an optional additional check of stderr.
Examples:

centos-7 # sysctl net.ipv4.ip_forward=1 ; echo $?
net.ipv4.ip_forward = 1
0

centos-7 # sysctl net.ipv4.ip_forward=backward ; echo $?
sysctl: setting key "net.ipv4.ip_forward": Invalid argument
net.ipv4.ip_forward = backward
0
remove redundant mock_open() in darwin_sysctl_test.py
pylint fix on linux_sysctl_test.py
Fix state_verbose: False output
Merge pull request #17073 from jfindlay/jmoney/linux_sysctl_test

Jmoney/linux_sysctl_test
Merge pull request #17064 from opdude/hg-quotations

Quotations are not necessary when passing cmd arguments in as an array
Merge pull request #17081 from lyft/fix-state-verbose

Fix state_verbose: False output
Update RC version in docs sidebar
Merge pull request #17084 from terminalmage/docs

Update RC version in docs sidebar