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 && \