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

Suggestions for the salt report upload runner #56

Open
bochi opened this issue Jul 20, 2020 · 3 comments
Open

Suggestions for the salt report upload runner #56

bochi opened this issue Jul 20, 2020 · 3 comments

Comments

@bochi
Copy link
Contributor

bochi commented Jul 20, 2020

Hi,
I have some suggestions for packaging the salt report upload runner. When manually copying it to salt://_runners it could be that when there are updates or fixes they are missed because its not included in the package.
So I'd rather have it installed via package to some location and documented that one has to add whatever directory it is installed to to runner_dirs in the salt master config, ie:

runner_dirs:
  - /wherever/the/package/puts/the/runner

This also eliminates the need to sync the runners.

Same goes for the reactor, it should be installed to the same directory and referenced directly (with full path) in the reactor's config. Reactors or runners do not have to reside in file_roots.

@sbernhard
Copy link
Contributor

Thanks. I will have a look at this "soon".

@bochi
Copy link
Contributor Author

bochi commented Jul 23, 2020

@nadjaheitmann
Copy link
Contributor

Has this been resolved here theforeman/foreman-packaging@b38523b ?

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

No branches or pull requests

3 participants