-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Add helper functions to sync auth modules located in salt://_auth to master/extmods/auth #51938
Conversation
It makes sense to update the docs (both the release notes and the module list https://docs.saltstack.com/en/develop/topics/development/modules/#module-types), add some tests |
@cro ping. |
ping @dwoz. This should be ready now. |
What does this PR do?
This PR adds the helper functions necessary to sync auth modules from
salt://_auth
to/var/cache/salt/master/extmods/auth
. Auth modules are already loaded by the loader, so manually putting them in the cache directory would cause Salt to "do the right thing" and load them when needed, there was just no way to get them there with Salt utilities.What issues does this PR fix or reference?
No filed issue that I could see.
Previous Behavior
salt-run saltutil.sync_all
would not copy auth modules. There was nosalt-run saltutil.sync_auth
at all.New Behavior
salt-run saltutil.sync_all
now copies auth modules to/var/cache/salt/master/extmods/auth
where the loader will pick them up. There is a correspondingsaltutil.sync_auth
as well.Commits signed with GPG?
Yes