-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Added cluster setup using go processes #5346
Merged
Merged
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
ba3d1c2
Added cluster setup using go processes
arindamnayak ee6a939
added etcdctl as link
arindamnayak 06cc912
Merge branch 'master' into cluster_for_test
arindamnayak fa27ea8
revert bootstrap.sh change
arindamnayak f5b3737
Added alternative for etcdctl
arindamnayak 0bd72fe
Added alternative for etcdctl
arindamnayak 8e60a47
Merge branch 'cluster_for_test' of github.com:planetscale/vitess into…
ajeetj ef7139d
externalize etcd peer port and tmp directory
arindamnayak f3969f8
separated cluster test to shard 2
arindamnayak c7adf4c
Transaction mode and svchema e2e GO test cases
ajeetj 5ac032e
exclude cluster test from e2e_race
arindamnayak c2627e7
updated health check for vtgate and vtctld
arindamnayak 316dabf
Merge branch 'tal_test_master' into cluster_for_test
arindamnayak 976242f
Signed-off-by: Arindam Nayak <[email protected]>
arindamnayak c88c678
fix tablet teardown
arindamnayak e79b845
Merge remote-tracking branch 'upstream/master' into cluster_for_test
morgo b7672bd
Add GitHub action for building new test
morgo 24fabc9
moved cluster test to not to execute in travis, as it will be managed…
arindamnayak bb51e19
added rand.seed when cluster instance is built
arindamnayak File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
name: e2e Test Cluster | ||
on: [push, pull_request] | ||
jobs: | ||
|
||
build: | ||
name: Build | ||
runs-on: ubuntu-latest | ||
steps: | ||
|
||
- name: Set up Go | ||
uses: actions/setup-go@v1 | ||
with: | ||
go-version: 1.13 | ||
|
||
- name: Check out code | ||
uses: actions/checkout@v1 | ||
|
||
- name: Get dependencies | ||
run: | | ||
sudo apt-get install -y mysql-server mysql-client make unzip g++ etcd curl git wget | ||
sudo service mysql stop | ||
sudo service etcd stop | ||
sudo ln -s /etc/apparmor.d/usr.sbin.mysqld /etc/apparmor.d/disable/ | ||
sudo apparmor_parser -R /etc/apparmor.d/usr.sbin.mysqld | ||
go mod download | ||
|
||
- name: Run bootstrap.sh | ||
run: | | ||
echo "Copying new bootstrap over location of legacy one." | ||
cp .github/bootstrap.sh . | ||
./bootstrap.sh | ||
|
||
- name: Build | ||
run: | | ||
GOBIN=$PWD/bin make build | ||
|
||
- name: Run e2e test cluster | ||
run: | | ||
export PATH=$PWD/bin:$PATH | ||
source ./dev.env | ||
VTDATAROOT=/tmp/vtdataroot VTTOP=$PWD VTROOT=$PWD tools/e2e_test_cluster.sh |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@morgo how do we know which version of mysql is being used here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is the one included in
ubuntu-latest
, so likely 8.0. I can change it to MySQL 5.7 from Oracle repos. For the local-example it does the same, but a matrix build probably makes more sense so we catch both.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I looked into this further, it is 5.7:
The
ubuntu-latest
is 18.04. Assuming GitHub plans to bump this with new lts releases, this will change to 20.04, and MySQL 8.0 in April 2020. The fix then is to switch to the MySQL team repos, or consider upgrading our testsuite. Since MySQL 8.0 initializes slower, we will most likely chose the repo switch: