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

[Feature Request] salt-api not installed in onedir #2012

Closed
Foorack opened this issue Jul 23, 2024 · 2 comments · Fixed by #2013
Closed

[Feature Request] salt-api not installed in onedir #2012

Foorack opened this issue Jul 23, 2024 · 2 comments · Fixed by #2013
Assignees
Labels

Comments

@Foorack
Copy link

Foorack commented Jul 23, 2024

Description of Issue/Question

There is no way to tell bootstrap to install salt-api when using the onedir method.

This is effectively a breakout of #609 (comment) to be able to track it.

There isn't a flag currently in the bootstrap script that handles installing the salt-api package. This needs to be added for all distros that split out the salt-api package, which I believe is most (if not all) of the distros packaged and hosted on repo.saltstack.com.

Steps to Reproduce Issue

curl -L https://bootstrap.saltstack.com -o install_salt.sh
sh install_salt.sh -d -M -X onedir 3007.1
salt-api      # COMMAND DOES NOT EXIST
@amalaguti
Copy link

Just had the same issue with 3006.8

@dmurphy18 dmurphy18 changed the title salt-api not installed in onedir [Feature Request] salt-api not installed in onedir Jul 23, 2024
@dmurphy18
Copy link
Contributor

dmurphy18 commented Jul 23, 2024

@Foorack Wow, quoting an issue from 2015, bootstrap appear not to have this functionality, so treating this as a Feature Request. Need to pick an option flag to install salt_api and ensure salt-master is installed if selected, same for syndic

Note, the salt-api won't be enabled at boot, given the amount of times this comment appears

Skip salt-api since the service should be opt-in and not necessarily started on boot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants