Skip to content

[ZEPPELIN-5270] Implementing a soft shutdown for K8s interpreters#4064

Closed
Reamer wants to merge 1 commit intoapache:masterfrom
Reamer:k8s_potwatcher
Closed

[ZEPPELIN-5270] Implementing a soft shutdown for K8s interpreters#4064
Reamer wants to merge 1 commit intoapache:masterfrom
Reamer:k8s_potwatcher

Conversation

@Reamer
Copy link
Contributor

@Reamer Reamer commented Mar 1, 2021

What is this PR for?

This PR adds a softshutdown for K8s interpreters. Also changed the implementation from multiple API calls to a watcher implementation.

What type of PR is it?

  • Improvement

What is the Jira issue?

How should this be tested?

  • CI

Questions:

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

@Reamer Reamer force-pushed the k8s_potwatcher branch 2 times, most recently from aa7ff3e to 3b0b68e Compare March 9, 2021 14:36
@Reamer
Copy link
Contributor Author

Reamer commented Mar 12, 2021

All tests are passed. I will merge this PR into master and branch-0.9 on Monday if no further comments are received.

@asfgit asfgit closed this in 95a0f43 Mar 15, 2021
asfgit pushed a commit that referenced this pull request Mar 15, 2021
### What is this PR for?
This PR adds a softshutdown for K8s interpreters. Also changed the implementation from multiple API calls to a watcher implementation.

### What type of PR is it?
- Improvement

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

### How should this be tested?
* CI

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

Author: Philipp Dallig <philipp.dallig@gmail.com>

Closes #4064 from Reamer/k8s_potwatcher and squashes the following commits:

95ebd59 [Philipp Dallig] Implement softshutdown in K8s with K8s watcher

(cherry picked from commit 95a0f43)
Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
@Reamer Reamer deleted the k8s_potwatcher branch March 15, 2021 06:17
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.

1 participant