Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure object store plugin processes are cleaned up after restore and after BSL validation during server start up #2041

Merged
merged 2 commits into from
Nov 5, 2019

Conversation

betta1
Copy link
Contributor

@betta1 betta1 commented Nov 4, 2019

This fixes #2039

@skriss
Copy link
Contributor

skriss commented Nov 5, 2019

@betta1 this looks good to me - I'll do some testing today as well. Thanks for the PR!

@skriss
Copy link
Contributor

skriss commented Nov 5, 2019

FYI, basic testing LGTM - can confirm it fixes the issue, and I didn't encounter any other problems

@skriss
Copy link
Contributor

skriss commented Nov 5, 2019

@carlisia @nrb @prydonius PTAL as well, thanks!

Copy link
Contributor

@skriss skriss left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@carlisia carlisia left a comment

Choose a reason for hiding this comment

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

💯 lgtm!

@carlisia carlisia merged commit 121dc02 into vmware-tanzu:master Nov 5, 2019
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.

Plugin processes for object stores not cleared out after restore
4 participants