You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by 1ambda <gi...@git.apache.org> on 2017/09/15 13:08:30 UTC
[GitHub] zeppelin pull request #2590: [ZEPPELIN-2938] Can't build docker image for bi...
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2590
[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
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/1ambda/zeppelin fix/install-wget-for-docker-container-dep
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2590.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2590
----
commit 66463343f3829ec658c1e1436a9aab919794091f
Author: 1ambda <1a...@gmail.com>
Date: 2017-09-15T12:42:55Z
fix: Install wget
----
---
[GitHub] zeppelin issue #2590: [ZEPPELIN-2938] Can't build docker image for bin due t...
Posted by 1ambda <gi...@git.apache.org>.
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2590
CI Failed but irrelevant. Merge if no more discussion.
---
[GitHub] zeppelin pull request #2590: [ZEPPELIN-2938] Can't build docker image for bi...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2590
---
[GitHub] zeppelin issue #2590: [ZEPPELIN-2938] Can't build docker image for bin due t...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2590
LGTM
---