Skip to content

[ZEPPELIN-4236] Cluster mode support interpreter running in docker#3405

Closed
xunliu wants to merge 2 commits intoapache:masterfrom
xunliu:ZEPPELIN-4236
Closed

[ZEPPELIN-4236] Cluster mode support interpreter running in docker#3405
xunliu wants to merge 2 commits intoapache:masterfrom
xunliu:ZEPPELIN-4236

Conversation

@xunliu
Copy link
Copy Markdown
Member

@xunliu xunliu commented Jul 15, 2019

What is this PR for?

Now that zeppelin is set up in cluster mode, the interpreter will not work in docker mode.
We need to mix these two modes together.
This will allow more users to use the interpreter container.

What type of PR is it?

[Improvement]

What is the Jira issue?

How should this be tested?

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

Copy link
Copy Markdown
Member

@felixcheung felixcheung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any test for this?

@xunliu
Copy link
Copy Markdown
Member Author

xunliu commented Jul 15, 2019

Add Docker Mode & Local Mode test case.
And CI Pass

@xunliu
Copy link
Copy Markdown
Member Author

xunliu commented Jul 18, 2019

Will merge if no more comments

@asfgit asfgit closed this in 8f97164 Jul 19, 2019
srehmann pushed a commit to srehmann/zeppelin that referenced this pull request Jul 25, 2019
### What is this PR for?
Now that zeppelin is set up in cluster mode, the interpreter will not work in docker mode.
We need to mix these two modes together.
This will allow more users to use the interpreter container.

### What type of PR is it?
[Improvement]

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-4236

### How should this be tested?
* [CI pass](https://travis-ci.org/liuxunorg/zeppelin/builds/557797483)

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Xun Liu <liuxun@apache.org>

Closes apache#3405 from liuxunorg/ZEPPELIN-4236 and squashes the following commits:

0a9b34b [Xun Liu] Add `Docker Mode` and `Local Mode` test case.
b26195d [Xun Liu] [ZEPPELIN-4236] Cluster mode support interpreter running in docker
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants