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

VReplication e2e tests: Simplify and split tests to overcome current CI resource constraints #6730

Merged
merged 3 commits into from
Sep 17, 2020

Conversation

rohit-nayak-ps
Copy link
Contributor

@rohit-nayak-ps rohit-nayak-ps commented Sep 16, 2020

VReplication e2e tests never ran in CI. We traced the errors to insufficient resources: either file handles or aio handles. These are set as 64K in /etc/security/limits.conf and /proc/sys/fs/aio-max-nr and they cannot be overwritten by sudo.

For now, until we can figure out to increase the hard limits:

  • The basic test runs only with masters (replicas set to 0)
  • The three tests have been separated into different test "shards" (22/23/24 atm).

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

@rohit-nayak-ps rohit-nayak-ps changed the title Only use master tablets for the Basic test since CI images run out of resources with too many tablets VReplication e2e tests: Simplify and split tests to overcome current CI resource constraints Sep 17, 2020
… resources with too many tablets

Signed-off-by: Rohit Nayak <rohit@planetscale.com>
Signed-off-by: Rohit Nayak <rohit@planetscale.com>
Signed-off-by: Rohit Nayak <rohit@planetscale.com>
@deepthi deepthi merged commit d58a0fe into vitessio:master Sep 17, 2020
@deepthi deepthi deleted the rn-ci-test2 branch September 17, 2020 20:39
@askdba askdba added this to the v8.0 milestone Oct 6, 2020
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.

3 participants