openSUSE / salt

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

Avoid explicit reading of /etc/salt/minion (bsc#1220357) #683

Closed vzhestkov closed 3 weeks ago

vzhestkov commented 3 weeks ago

What does this PR do?

Parcial backport of https://github.com/saltstack/salt/pull/65696 with https://github.com/saltstack/salt/pull/65696/commits/d54407ba6dc664e5e5f3f613e27ae24f828c9648 from upstream PR only.

Upstream PR is messy and contains a list of changes not related to the original description. I'm closing previous PR https://github.com/openSUSE/salt/pull/641 in favor of this one.

In some cases salt-call on the salt-ssh client side is rereading /etc/salt/minion explicitly what doesn't make any sense.

What issues does this PR fix or reference?

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

Previous Behavior

In some cases salt-call on the salt-ssh client side is rereading /etc/salt/minion.

New Behavior

Prevent salt-call of rereading /etc/salt/minion with no reason.

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.