-
Notifications
You must be signed in to change notification settings - Fork 674
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
Adding E2E Tests to CircleCI #253
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
5eeb7cb
Merge pull request #1 from ava-labs/master
galenmarchetti 081ab11
defining basic e2e tests ci
galenmarchetti 3eb9efd
modifying travisci yaml
galenmarchetti 37b7440
pulling images
galenmarchetti d85a016
e2e tests must be runnable
galenmarchetti 0165827
removing osx build to debug kurtosis runs;
galenmarchetti 374062a
sleeping 90
galenmarchetti dea626a
putting in debugging statements
galenmarchetti 2036c72
building with docker deploy in scripts
galenmarchetti c695422
build image script
galenmarchetti 6fac7c2
fixing travis yml
galenmarchetti f8cea1d
travisyml now minimally modified to include kurtosis tests
galenmarchetti 72330d4
removing debugging statements
galenmarchetti 1ae9c76
isolating the kurtosis testing
galenmarchetti 76e7122
Merge pull request #2 from kurtosis-tech/gmarchetti/kurtosis-e2e-ci-i…
galenmarchetti e771c2f
Merge remote-tracking branch 'upstream/master'
3de0209
Merge pull request #3 from ava-labs/master
galenmarchetti 1be5daf
updating CI
galenmarchetti 4b22442
changing path to access ava-e2e-tests
galenmarchetti 5cc4aa4
adding debugging steps
galenmarchetti ea7b77c
adding correct cd
galenmarchetti 9ac4472
removing -d
galenmarchetti 878056d
pulling in gopath
galenmarchetti 0627c7c
turning GO111MODULE off to pull
galenmarchetti 2e16f20
cleaning up CI script
galenmarchetti a523fb1
cleaning CI run script
galenmarchetti 4cacb56
commenting controller image label from script
galenmarchetti 551e16f
checking build directory
galenmarchetti 0f0439f
calling executable raw
galenmarchetti 50fba75
defining just two tests
galenmarchetti f4a4283
corrected test-names arg
galenmarchetti 4acaea9
Merge pull request #4 from kurtosis-tech/gmarchetti/integrate-kurtosi…
galenmarchetti 6a37d26
replacing whitespace
galenmarchetti e2aea23
re-enabling osx
galenmarchetti fe8157f
Merge pull request #5 from kurtosis-tech/gmarchetti/reset-comments
galenmarchetti 62340e4
removing set x
galenmarchetti 81494fa
Merge pull request #6 from kurtosis-tech/gmarchetti/remove-set-x
galenmarchetti 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,16 @@ | ||
| SCRIPTS_PATH=$(cd $(dirname "${BASH_SOURCE[0]}"); pwd) | ||
| SRC_PATH=$(dirname "${SCRIPTS_PATH}") | ||
| # Build the runnable Gecko docker image | ||
| bash "${SRC_PATH}"/scripts/build_image.sh | ||
| GECKO_IMAGE=$(docker image ls --format="{{.Repository}}" | head -n 1) | ||
|
|
||
| # Turn off GO111MODULE to pull e2e test source code in order to get run script. | ||
| GO111MODULE=off go get -t -v github.com/kurtosis-tech/ava-e2e-tests/... | ||
| cd "${GOPATH}"/src/github.com/kurtosis-tech/ava-e2e-tests/ || exit | ||
|
|
||
| bash "./scripts/rebuild_initializer_binary.sh" | ||
| bash "./scripts/rebuild_controller_image.sh" | ||
| # TODO: Make the controller image label a parameter to rebuild_controller_image script | ||
| # Standard controller image label used by above scripts. | ||
| CONTROLLER_IMAGE="kurtosistech/ava-e2e-tests_controller:latest" | ||
| ./build/ava-e2e-tests --gecko-image-name="${GECKO_IMAGE}" --test-controller-image-name="${CONTROLLER_IMAGE}" --test-names="fiveStakingNodeGetValidatorsTest,fiveStakingNodeFullyConnectedTest" | ||
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
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.
I am not sure I understand why we need to switch off
GO111MODULEhere.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.
@holisticode in go 1.13, with GO111MODULE=on,
go getwill not pull source code. ( golang/go#31529 ) We have scripts that build necessary docker images which interact with the network, so we want to pull the source from the test repo to build those images.