state.archive recursively set group

[russellballestrini]

recursively set group owner, archive_user to user

modified:   salt/states/archive.py

[jfindlay]

@russellballestrini, since this involves an API change ('archive_user' -> 'user'), the change needs to be formally deprecated until some future release. See a443680 as an example. Also, there are lint errors.

[russellballestrini]

@jfindlay thank you for the hints, I will bring this PR into spec for contribution by adding warnings about deprecation of archive_user in favor of the standardized user parameter, update the change log, and work out the issues with lint.

[jfindlay]

@russellballestrini, no problem, and thank you for the fixes.

[russellballestrini]

@jfindlay please review.

[jfindlay]

@russellballestrini, excellent, thank you.

[russellballestrini]

@jfindlay thank you! How do we target this for v2014.7.2 and/or v2015.2.0rc1

[jfindlay]

@russellballestrini, the 2014.7 branch is almost retired as 2015.2 is now in the rc phase. If you want changes to go into release branches, the best thing to do is submit pull requests against that branch rather than develop. Since this is already merged, we can backport it, but it is cleaner and easier to submit changes against the feature branch as they will be merged into develop every few days.