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

Optionally enable salt-minion #239

Merged
merged 6 commits into from Mar 20, 2024
Merged

Conversation

cdalvaro
Copy link
Owner

@cdalvaro cdalvaro commented Mar 12, 2024

This PR adds support for running a salt-minion inside the container.

The service is disabled by default, but it can be enabled using the environment variable SALT_MINION_ENABLED=True.

Additionally, this PR renames the env variable SALT_API_SERVICE_ENABLED to SALT_API_ENABLED (for consistency with the new env variable). Backward compatibility will be provided until 3007.2.

More details will be provided as new commits are added...

Closes #237

@cdalvaro cdalvaro added enhancement New feature or request suggestions-accepted Suggestions are welcome docker Pull requests that update Docker code labels Mar 12, 2024
@cdalvaro cdalvaro self-assigned this Mar 12, 2024
@cdalvaro cdalvaro force-pushed the feature/optionally-enable-salt-minion branch 2 times, most recently from 65fff4e to 2b4a6f0 Compare March 15, 2024 12:00
@cdalvaro cdalvaro marked this pull request as ready for review March 15, 2024 12:01
@cdalvaro
Copy link
Owner Author

cdalvaro commented Mar 15, 2024

Hi @abhi1693, could you build this branch and test it with your settings to make sure this development covers your requirements?

Edit: Simply clone this branch and execute: make build to build the image.

@cdalvaro cdalvaro force-pushed the feature/optionally-enable-salt-minion branch from 2b4a6f0 to 0b18e45 Compare March 15, 2024 17:55
@cdalvaro cdalvaro force-pushed the feature/optionally-enable-salt-minion branch from 0b18e45 to ed4281c Compare March 19, 2024 22:24
@cdalvaro cdalvaro merged commit c8a2666 into main Mar 20, 2024
9 checks passed
@cdalvaro cdalvaro deleted the feature/optionally-enable-salt-minion branch March 20, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Pull requests that update Docker code enhancement New feature or request suggestions-accepted Suggestions are welcome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Optionally enable salt-minion
1 participant