Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix __salt__ when used via __utils__ (bsc#1220357) #641

Conversation

vzhestkov
Copy link
Contributor

What does this PR do?

Backport of saltstack/salt#65696

In some cases salt-call on the salt-ssh client side is rereading /etc/salt/minion explicitly what doesn't make any sense.
The upstreram PR contains the fix of it in ef140fc

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.

@vzhestkov vzhestkov marked this pull request as ready for review April 15, 2024 10:04
@vzhestkov vzhestkov force-pushed the openSUSE/backport/3006.0/65696-fix-__salt__-used-via-__utils__ branch 3 times, most recently from 0cff225 to d30d308 Compare August 30, 2024 08:40
@vzhestkov vzhestkov changed the title Fix __salt__ when used via __utils__ Fix __salt__ when used via __utils__ (bsc#1220357) Aug 30, 2024
@vzhestkov vzhestkov force-pushed the openSUSE/backport/3006.0/65696-fix-__salt__-used-via-__utils__ branch from d30d308 to 4fd5395 Compare August 30, 2024 11:13
@vzhestkov
Copy link
Contributor Author

Closing in favor of #683

@vzhestkov vzhestkov closed this Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants