salt.states.file.replace should honor user and group

[ruimarinho]

Currently, salt.states.file.replace changes the ownership of the file that is being updated. I believe this state should honor the original user/group and additionally support the user and group parameters as most of the other file states.

I worked around this issue by using a file.managed state later on the process to manage the ownership.

[terminalmage]

I've noticed this as well. The reason for this is that the old file is renamed to filename.bak and salt creates a new file. It should be also doing a chown after the fact to properly set ownershop. Should not be too difficult to fix.

[terminalmage]

@ruimarinho Check out #8417, this should take care of it. Thanks again for reporting this. Like I said, this is something I had noticed, but I just hadn't gotten around to doing anything about it.