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

Wrangler tests: Return a fake tablet in the wrangler test dialer to avoid tablet picker errors spamming the test logs #7863

Merged
merged 1 commit into from
Apr 14, 2021

Conversation

rohit-nayak-ps
Copy link
Contributor

Signed-off-by: Rohit Nayak rohit@planetscale.com

Description

The wrangler test framework was incorrectly creating a lot of unnecessary logs like shown below. The fake tablets it creates don't always contain a valid QueryService implementation since the tests don't need it. The tablet picker started logging these as warnings when it was reimplemented resulting in inaccurate logs.

This PR returns a fake queryserivice in such cases so that the picker doesn't log them. Some residual logs are also removed.

In WranglerTest dialer
In WranglerTest dialer
In WranglerTest dialer
In WranglerTest dialer
W0413 14:28:22.422144   31135 tablet_picker.go:133] unable to connect to tablet for alias cell:"cell1" uid:10 
W0413 14:28:22.422121   31135 tablet_picker.go:133] unable to connect to tablet for alias cell:"cell1" uid:10 
In WranglerTest dialer
W0413 14:28:22.422185   31135 tablet_picker.go:133] unable to connect to tablet for alias cell:"cell1" uid:10 
W0413 14:28:22.422253   31135 tablet_picker.go:133] unable to connect to tablet for alias cell:"cell1" uid:10 

Related Issue(s)

Checklist

  • Should this PR be backported?
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

Impacted Areas in Vitess

Components that this PR will affect:

  • Query Serving
  • VReplication
  • Cluster Management
  • Build/CI
  • VTAdmin

…r errors spamming the test logs

Signed-off-by: Rohit Nayak <rohit@planetscale.com>
Copy link
Member

@deepthi deepthi left a comment

Choose a reason for hiding this comment

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

LGTM

@rohit-nayak-ps rohit-nayak-ps merged commit 0104b4b into vitessio:master Apr 14, 2021
@rohit-nayak-ps rohit-nayak-ps deleted the rn-wrangler-tests branch April 14, 2021 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants