openSUSE / salt

openSUSE and SUSE patches and backports for SaltStack
Apache License 2.0
22 stars 52 forks source link

Remove redundant `_file_find` call to the master #645

Closed vzhestkov closed 5 months ago

vzhestkov commented 5 months ago

What does this PR do?

This PR is intended to improve the performance in the large scale deployments as there is a useless call to get stats of the syncronized files which is not used. Seem it's a leftover of removing the code with https://github.com/saltstack/salt/pull/40609 due to https://github.com/advisories/GHSA-xcx4-5wq7-g5g7 introduced with https://github.com/saltstack/salt/commit/73a156d28cafd45dcc6c2aac9f1ac3ad905695e3 (https://github.com/saltstack/salt/pull/34807 ???)

What issues does this PR fix or reference?

Tracks: https://github.com/SUSE/spacewalk/issues/23526

Previous Behavior

Redundant _file_find calls in combination with _file_hash.

New Behavior

No redundant _file_find calls on files syncronization.

Merge requirements satisfied?

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

Commits signed with GPG?

Yes/No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.