You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by mi...@apache.org on 2017/09/16 14:25:18 UTC
zeppelin git commit: [ZEPPELIN-2938] Can't build docker image for bin
due to missing wget cmd (master, branch-0.7)
Repository: zeppelin
Updated Branches:
refs/heads/master 9c1fca182 -> d25639caa
[ZEPPELIN-2938] Can't build docker image for bin due to missing wget cmd (master, branch-0.7)
### What is this PR for?
Can't build docker image for bin due to missing wget cmd
![image](https://user-images.githubusercontent.com/4968473/30482870-b6c9344a-9a5f-11e7-913f-b67d67a49a66.png)
### What type of PR is it?
[Bug Fix]
### What is the Jira issue?
[ZEPPELIN-2938](https://issues.apache.org/jira/browse/ZEPPELIN-2938)
### How should this be tested?
- `cd scripts/docker/zeppelin/bin/`
- `docker build . -t test`
- should fail here
![image](https://user-images.githubusercontent.com/4968473/30483594-5ee7fc72-9a62-11e7-8472-1fdf3115fa0f.png)
### Screenshots (if appropriate)
#### Before
![image](https://user-images.githubusercontent.com/4968473/30482783-5d344d70-9a5f-11e7-8407-de039ccfd16a.png)
#### After
### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO
Author: 1ambda <1a...@gmail.com>
Closes #2590 from 1ambda/fix/install-wget-for-docker-container-dep and squashes the following commits:
66463343 [1ambda] fix: Install wget
Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/d25639ca
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/d25639ca
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/d25639ca
Branch: refs/heads/master
Commit: d25639caaed4a23f1623f3439568d7e4666addfb
Parents: 9c1fca1
Author: 1ambda <1a...@gmail.com>
Authored: Fri Sep 15 21:42:55 2017 +0900
Committer: Mina Lee <mi...@apache.org>
Committed: Sat Sep 16 16:24:59 2017 +0200
----------------------------------------------------------------------
scripts/docker/zeppelin/bin/Dockerfile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/zeppelin/blob/d25639ca/scripts/docker/zeppelin/bin/Dockerfile
----------------------------------------------------------------------
diff --git a/scripts/docker/zeppelin/bin/Dockerfile b/scripts/docker/zeppelin/bin/Dockerfile
index 9c8cfa3..da5a677 100644
--- a/scripts/docker/zeppelin/bin/Dockerfile
+++ b/scripts/docker/zeppelin/bin/Dockerfile
@@ -33,7 +33,7 @@ RUN echo "$LOG_TAG update and install basic packages" && \
apt-get install -y build-essential
RUN echo "$LOG_TAG install tini related packages" && \
- apt-get install -y curl grep sed dpkg && \
+ apt-get install -y wget curl grep sed dpkg && \
TINI_VERSION=`curl https://github.com/krallin/tini/releases/latest | grep -o "/v.*\"" | sed 's:^..\(.*\).$:\1:'` && \
curl -L "https://github.com/krallin/tini/releases/download/v${TINI_VERSION}/tini_${TINI_VERSION}.deb" > tini.deb && \
dpkg -i tini.deb && \