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

[STK-114] Add bootstrap.sh and update dockerfile #7

Merged
merged 3 commits into from
Feb 27, 2018

Conversation

waseem-h
Copy link
Contributor

This PR updates the dockerfile and adds a bootstrap.sh file which is executed upon container creation

@waseem-h waseem-h self-assigned this Feb 27, 2018
@waseem-h waseem-h requested review from asim42 and hazim1093 February 27, 2018 10:55
Dockerfile Outdated
@@ -4,11 +4,10 @@ LABEL author="stakater"
ARG SRC_DIR="ingress-monitor-controller"
ENV SRC_DIR=${SRC_DIR}

ADD bootstrap.sh /bootstrap.sh

RUN chmod +x /bootstrap.sh
Copy link
Member

Choose a reason for hiding this comment

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

No need to run this here, run this locally on the script so the file is committed with executable permissions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@waseem-h waseem-h changed the title Add bootstrap.sh and update dockerfile [STK-114] Add bootstrap.sh and update dockerfile Feb 27, 2018
@@ -0,0 +1,13 @@
#!/usr/bin/env sh

Copy link
Member

Choose a reason for hiding this comment

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

Add "echos" just to tell what's happening so the logs are understandable

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@hazim1093 hazim1093 merged commit 182d8c4 into master Feb 27, 2018
@hazim1093 hazim1093 deleted the update-dockerfile branch February 27, 2018 11:03
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.

2 participants