Skip to content

Add new chaos test suite. #14417

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
Merged
merged 1 commit into from
Jul 14, 2021
Merged

Add new chaos test suite. #14417

merged 1 commit into from
Jul 14, 2021

Conversation

mpoeter
Copy link
Contributor

@mpoeter mpoeter commented Jun 29, 2021

Scope & Purpose

Backport of #14406

Add a chaos test suite with a cluster-chaos-load test which revealed several issues in the replication system.

  • 💩 Bugfix (requires CHANGELOG entry)
  • 🍕 New feature (requires CHANGELOG entry, feature documentation and release notes)
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification
  • 📖 CHANGELOG entry made

Backports:

  • Backports required for: 3.8, 3.7

Testing & Verification

  • This change is a trivial rework / code cleanup without any test coverage.
  • The behavior in this PR was manually tested
  • This change is already covered by existing tests, such as (please describe tests).
  • This PR adds tests that were used to verify all changes:
    • Added new C++ Unit tests
    • Added new integration tests (e.g. in shell_server / shell_server_aql)
    • Added new resilience tests (only if the feature is impacted by failovers)
  • There are tests in an external testing repository:
  • I ensured this code runs with ASan / TSan or other static verification tools

@mpoeter mpoeter added this to the 3.8 milestone Jun 29, 2021
@mpoeter mpoeter requested review from jsteemann and dothebart June 29, 2021 09:35
@mpoeter mpoeter mentioned this pull request Jun 29, 2021
Merged
@KVS85
Copy link
Contributor

KVS85 commented Jun 29, 2021 •
edited
Loading

http://jenkins01.arangodb.biz:8080/job/arangodb-ANY-matrix/45/

jsteemann
jsteemann approved these changes Jun 29, 2021
View reviewed changes
Copy link
Contributor

@jsteemann jsteemann left a comment

Choose a reason for hiding this comment

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

LGTM

@KVS85 KVS85 merged commit f1043d9 into 3.8 Jul 14, 2021
@KVS85 KVS85 deleted the feature-3.8/cluster-chaos-load-test branch July 14, 2021 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsteemann jsteemann jsteemann approved these changes

@dothebart dothebart dothebart approved these changes

Assignees

@mpoeter mpoeter

Labels
3 - Testing 3 Cluster
Projects
None yet
Milestone
3.8
Development

Successfully merging this pull request may close these issues.
4 participants
@mpoeter @KVS85 @jsteemann @dothebart