Skip to content

Conversation

@Reamer
Copy link
Contributor

@Reamer Reamer commented Apr 29, 2020

What is this PR for?

With this PR, we render the K8s-Service name into the Zeppelin interpreter start command for the initial connection.
The K8s-Service name should be variable, because the name can vary from environment to environment.

What type of PR is it?

  • Bug Fix

Todos

  • - Task

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

@Leemoonsoo
Copy link
Member

LGTM. Merge to master and branch-0.9 if no further comment.

@asfgit asfgit closed this in 4359193 Apr 30, 2020
asfgit pushed a commit that referenced this pull request Apr 30, 2020
### What is this PR for?
With this PR, we render the K8s-Service name into the Zeppelin interpreter start command for the initial connection.
The K8s-Service name should be variable, because the name can vary from environment to environment.
### What type of PR is it?
 - Bug Fix

### Todos
* [ ] - Task

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

### How should this be tested?
* Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/680915013

### 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: Philipp Dallig <[email protected]>

Closes #3753 from Reamer/k8s_service and squashes the following commits:

74be24f [Philipp Dallig] Use K8s service to connect Zeppelin interpreter with Zeppelin server
7ecee5d [Philipp Dallig] Some cleanup
84e4e33 [Philipp Dallig] Add ".factorypath" to .gitignore it's created by eclipse

(cherry picked from commit 4359193)
Signed-off-by: Lee moon soo <[email protected]>
@Reamer Reamer deleted the k8s_service branch May 4, 2020 07:30
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