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

Dockerfile: Include an entrypoint script for ejabberdctl #4071

Closed
wants to merge 1 commit into from
Closed

Dockerfile: Include an entrypoint script for ejabberdctl #4071

wants to merge 1 commit into from

Conversation

sando38
Copy link
Contributor

@sando38 sando38 commented Aug 3, 2023

Hi @badlop

The entrypoint script serves as a placeholder, so that admins can replace it with an actual entrypoint script, e.g. for initiating ejabberd clusters, through mounting it to the path /usr/local/bin/run.sh.

This PR does not change anything of the existing functions of the image.

Motivation:
I would like to use the image as basis for the helm-chart with these scripts and not rely on a custom image in the end.

This serves as a placeholder, so that custom images can use an actual entrypoint script, e.g. for initiating ejabberd clusters.
@coveralls
Copy link

Coverage Status

coverage: 32.991% (+0.007%) from 32.984% when pulling ea157ba on sando38:master into 20a8654 on processone:master.

@sando38
Copy link
Contributor Author

sando38 commented Aug 12, 2023

Deleted the branch by accident, will open again if necessary.

@sando38 sando38 closed this Aug 12, 2023
@Neustradamus
Copy link
Contributor

@badlop, @prefiks, @mremond: @sando38 needs to create a new PR?

@sando38
Copy link
Contributor Author

sando38 commented Aug 18, 2023

I think, I don't. For my particular case, I discovered two options to override entrypoint and cmd:
https://www.ithands-on.com/2021/11/kubernetes-101-overriding-entrypoint.html

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.

3 participants