-
Notifications
You must be signed in to change notification settings - Fork 641
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
feat(install): Install packer when rosco is installed. #189
Conversation
I'm going to add a check if |
rosco-web/pkg_scripts/postInstall.sh
Outdated
@@ -11,4 +11,13 @@ if [ -z `getent passwd spinnaker` ]; then | |||
useradd --gid spinnaker spinnaker -m --home-dir /home/spinnaker | |||
fi | |||
|
|||
# install packer | |||
pushd | |||
mkdir /packer && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think /packer
is the right way to do this. I'd create a tempdir and install there, removing the tempdir after.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Dockerfile
installs to /packer
and then appends this to ENV
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dockerfiles are a bit of a special case, since it's more OK to pollute the /
directory in a containerized environment. This should end up in /usr/bin
.
rosco-web/pkg_scripts/postInstall.sh
Outdated
pushd | ||
mkdir /packer && \ | ||
cd /packer \ | ||
wget https://releases.hashicorp.com/packer/0.12.1/packer_0.12.1_linux_amd64.zip && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull this version into a top-level env var so it's easy to update.
Updated & tested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.