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.

Sign up for GitHub

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

Jump to bottom

Add e2e test to check for tablet healthcheck cache correctness #9109

Closed
wants to merge 2 commits into from
Closed

Add e2e test to check for tablet healthcheck cache correctness #9109

wants to merge 2 commits into from

Conversation

mattlord
Copy link
Contributor

Description

Specifically to confirm zombie tablets do not occur

Confirming that this fails on main...

Related Issue(s)

#9106

@mattlord mattlord mentioned this pull request Oct 28, 2021
Merged
3 tasks
@mattlord mattlord force-pushed the StaleHeathCheckCacheTest branch 2 times, most recently from 232539f to 40ad5cc Compare October 28, 2021 18:37
@mattlord mattlord force-pushed the StaleHeathCheckCacheTest branch 17 times, most recently from 0f675b7 to cfcc692 Compare October 29, 2021 05:15
@mattlord
Add e2e test to check for tablet healthcheck cache correctness
8f4a399 
Specifically to confirm zombie tablets do not occur

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord self-assigned this Oct 29, 2021
@mattlord
Minor improvements to tablet healthcheck correctness test
f2504fa 
Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord closed this Nov 1, 2021
@mattlord mattlord deleted the StaleHeathCheckCacheTest branch November 1, 2021 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GuptaManan100 GuptaManan100 Awaiting requested review from GuptaManan100 GuptaManan100 will be requested when the pull request is marked ready for review GuptaManan100 is a code owner

@frouioui frouioui Awaiting requested review from frouioui frouioui will be requested when the pull request is marked ready for review frouioui is a code owner

@rohit-nayak-ps rohit-nayak-ps Awaiting requested review from rohit-nayak-ps rohit-nayak-ps will be requested when the pull request is marked ready for review rohit-nayak-ps is a code owner

@deepthi deepthi Awaiting requested review from deepthi deepthi will be requested when the pull request is marked ready for review deepthi is a code owner

@harshit-gangal harshit-gangal Awaiting requested review from harshit-gangal harshit-gangal will be requested when the pull request is marked ready for review harshit-gangal is a code owner

@systay systay Awaiting requested review from systay systay will be requested when the pull request is marked ready for review systay is a code owner

Assignees

@mattlord mattlord

Labels
Component: Query Serving Do Not Merge Type: Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mattlord