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

grpcvtgateconn: Add fuzzer #7689

Merged
merged 1 commit into from
Mar 30, 2021
Merged

grpcvtgateconn: Add fuzzer #7689

merged 1 commit into from
Mar 30, 2021

Conversation

AdamKorcz
Copy link
Contributor

@AdamKorcz AdamKorcz commented Mar 15, 2021

Description

This PR adds a fuzzer.
The actual fuzzing happens at line 105-114. Everything prior to that is setup.
The file is named _flaky_test.go because the fuzzer uses some test utils and interferes with unit tests. Naming it _flaky_test.go is a recommendation by the CI and removes the breakages of unit tests.

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

@AdamKorcz AdamKorcz force-pushed the fuzz2 branch 18 times, most recently from 9dad70e to 735b326 Compare March 20, 2021 22:18
@AdamKorcz AdamKorcz marked this pull request as draft March 20, 2021 22:55
Signed-off-by: AdamKorcz <adam@adalogics.com>
@AdamKorcz AdamKorcz marked this pull request as ready for review March 25, 2021 16:27
@systay systay merged commit b5177cd into vitessio:master Mar 30, 2021
@askdba askdba added this to the v10.0 milestone Apr 1, 2021
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.

3 participants