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

Changing the original Wazuh docker image to allow adding code in entrypoint.sh #151

Merged
merged 2 commits into from
Apr 17, 2019

Conversation

Phandora
Copy link
Contributor

Hi team!

This PR changes the official Wazuh docker image to facilitate the process of adding code in the entrypoint.sh without using commands like sed or echo.

The changes are as follows:

  • 00-wazuh.sh contains the same code from the older entrypoint.sh but the call to the init service.
  • entrypoint-scripts folder is created in order to contain every configuration script.
    (e.g. 00-wazuh.sh, 10-cloud-backups.sh)
  • entrypoint.sh runs every .sh script located in entrypoint-scripts folder in lexicographical order, and finally it calls the init service.

Best regards,
Mayte Ariza

#
# It will run every .sh script located in entrypoint-scripts folder in lexicographical order
for script in `ls /entrypoint-scripts/*.sh | sort -n`; do
bash "$script" -H
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What means the "-H" argument?.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enable ! style history substitution. This option is on by default when the shell is interactive.```

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have removed "-H" argument

@manuasir manuasir merged commit 3d8d612 into k8s Apr 17, 2019
@manuasir manuasir deleted the k8s-cloud branch April 17, 2019 14:05
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.

None yet

3 participants