[master] Make file.manage_file permission, user and group atomic

[m-czernek]

What does this PR do?

In this PR, I tweak the internal behavior of file.manage_file to improve the file permission and owner consistency while the function is executing. See the linked PR for discussion of why this might be useful to users.

In the end, while modifying the code, I found this approach to be the simplest and cleanest; however, I am open to discussions and modifying the approach.

What issues does this PR fix or reference?

Fixes: #65651

Previous Behavior

When the module copied a file, the file temporarily used the 600 perms with the user of the current running process.

New Behavior

Currently, the module attempts to modify the basic file attributes as soon as the file is copied to minimize the risk of race conditions.

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

• Docs
• Changelog - https://docs.saltproject.io/en/master/topics/development/changelog.html
• Tests written/updated

Commits signed with GPG?

Yes

[welcome]

Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey.
Please be sure to review our Code of Conduct. Also, check out some of our community resources including:

• Community Wiki
• Salt’s Contributor Guide
• Join our Community Slack
• IRC on LiberaChat
• Salt Project YouTube channel
• Salt Project Twitch channel

There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar.
If you have additional questions, email us at saltproject@vmware.com. We’re glad you’ve joined our community and look forward to doing awesome things with you!

[m-czernek]

Ping @dwoz @felippeb, any chance for a review? :)

[twangboy]

Please create a changelog and write a test for this. Since this is such a heavily used module, we need to make sure it is well tested.

[twangboy]

Please rebase and address conflicts

[m-czernek]

PR rebased, should be mergeable now.

[twangboy]

Test failures