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

fix docker start with jib : added a sync before starting /entrypoint.sh #8468

Merged
merged 1 commit into from
Oct 6, 2018

Conversation

hcouplet
Copy link

@hcouplet hcouplet commented Oct 5, 2018

When running pod ok k8s, I have an error : (on my local docker it's fine)
sh: /entrypoint.sh : Text file busy

image

Looking in moby/moby#9547, it seems that the script can't start because the file is still busy with chmod.
I added a sync between the chmod and the script startup.
After that, all my microservices started correctly on kubernetes.

Changed in the pom.xml :
chmod +x /entrypoint.sh && /entrypoint.sh
with
chmod +x /entrypoint.sh && sync && /entrypoint.sh

When running pod ok k8s, I have an error :
   sh: /entrypoint.sh : Text file busy

Looking in moby/moby#9547, I added a sync between the chmod and the script startup.
After that, all my microservices started correctly
@jdubois
Copy link
Member

jdubois commented Oct 5, 2018

This is really good, I love this as it is the kind of contribution that is nearly impossible to figure out without such an awesome community! Thanks @hcouplet !

@deepu105
Copy link
Member

deepu105 commented Oct 5, 2018

@hcouplet can you also check the Gradle build?

@jdubois jdubois merged commit 0072a9a into jhipster:master Oct 6, 2018
@jdubois jdubois added this to the 5.4.2 milestone Oct 6, 2018
ruddell added a commit to ruddell/generator-jhipster that referenced this pull request Oct 6, 2018
@ruddell ruddell mentioned this pull request Oct 6, 2018
4 tasks
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