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

Add running integration tests to GitHub actions #66

Merged
merged 2 commits into from
Jul 27, 2022

Conversation

Gor027
Copy link
Contributor

@Gor027 Gor027 commented Jul 21, 2022

This PR adds running integration tests to GitHub actions. scylla-ccm is used for managing clusters and tests are run on Scylla 5.0.0. Tests are run with Valgrind which will detect memory corruption and will not show details of leaked memory. For now, only ClusterTests are enabled to be tested on GitHub actions.

@Gor027 Gor027 force-pushed the add_test_action branch from cf3a972 to 09add13 Compare July 21, 2022 07:51
@Gor027 Gor027 mentioned this pull request Jul 21, 2022
.github/pull_request_template.md Outdated Show resolved Hide resolved
.github/workflows/build.yml Outdated Show resolved Hide resolved
@Gor027 Gor027 force-pushed the add_test_action branch from 79967f2 to 7d3611b Compare July 21, 2022 10:09
@Gor027 Gor027 force-pushed the add_test_action branch from 7d3611b to e64d94f Compare July 26, 2022 15:31
@avelanarius avelanarius requested a review from jul-stas July 26, 2022 15:44
run: |
sudo sh -c "echo 'deb http://security.ubuntu.com/ubuntu xenial-security main' >> /etc/apt/sources.list"
sudo apt-get update
sudo apt-get install libssl1.0.0 libuv1-dev libkrb5-dev libc6-dbg
Copy link
Contributor

@jul-stas jul-stas Jul 26, 2022

Choose a reason for hiding this comment

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

Are libuv and krb5 really needed in -dev version?
EDIT: ok, apparently libuv-dev is necessary.

@jul-stas jul-stas merged commit af5854c into scylladb:master Jul 27, 2022
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