-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Make sure traefik runs in an openshift workspace #7955
Conversation
dockerfiles/dev/Dockerfile
Outdated
@@ -60,12 +62,17 @@ RUN mkdir /home/user/traefik ;\ | |||
wget -O /home/user/traefik/traefik "https://github.com/containous/traefik/releases/download/v1.4.3/traefik_linux-amd64"; \ | |||
chmod +x /home/user/traefik/traefik | |||
COPY traefik.toml /home/user/traefik/ | |||
RUN sudo chgrp -R 0 ~/traefik;\ | |||
sudo chmod -R g+rwX ~/traefik |
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.
can we group RUN instructions ?
dockerfiles/dev/Dockerfile
Outdated
|
||
|
||
RUN sudo mkdir /var/run/sshd | ||
RUN sudo ssh-keygen -t rsa -f /etc/ssh/ssh_host_rsa_key -N '' && \ | ||
sudo ssh-keygen -t rsa -f /etc/ssh/ssh_host_ecdsa_key -N '' && \ | ||
sudo ssh-keygen -t rsa -f /etc/ssh/ssh_host_ed25519_key -N '' && \ | ||
npm install -g typescript@2.5.3 typescript-language-server@0.1.4 | ||
|
||
ADD entrypoint.sh /home/user/entrypoint.sh | ||
RUN sudo chmod +x /home/user/entrypoint.sh |
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.
do we need to chmod the file if file is committed with execute permissions ?
dockerfiles/dev/Dockerfile
Outdated
WORKDIR /projects | ||
ENTRYPOINT ["/home/user/entrypoint.sh"] | ||
CMD tail -f /dev/null |
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.
there is a missing new line
@@ -0,0 +1,32 @@ | |||
#!/bin/bash |
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.
copyright header is missing
What does this PR do?
Sets CHE_API for Traefik according to infra, adds sufficient permissions for tomcat and traefik