NodeUtility::WriteNodeConfigObjects(): avoid unneccessary Utility::SetFileOwnership()

[Al2Klimov]

fixes #8743

[Al2Klimov]

@cla-bot check

[julianbrost]

In the current state, the call definitely doesn't make sense. It chowns the file just before it's replaced with a new version. However, I think it could have been the intention of the code to actually chown tempFilename instead of filename (or rather filename after the move), especially given that this is done on the directory as well. Is this code always executed as the icinga user? Then the chown would be pointless as the new file is owned by that user already anyways.

[Al2Klimov]

Is this code always executed as the icinga user?

Yes. This is node-setup-only code and the respective CLI commands drop permissions.