-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Refactored end-to-end tests fully orchestrated by pytest #1094
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
Merged
Merged
Changes from all commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
5a5b7f2
python to manage e2e dependencies
pyalex 51a7f11
all tests running
pyalex dafe895
run tests on github action
pyalex 0bd30cc
fixing e2e action
pyalex 3e93b5f
pass version as option
pyalex ef30da1
setup python
pyalex d054591
setup python
pyalex 43d3641
setup python
pyalex e16543c
pass feast version
pyalex 82ebdf3
python style
pyalex cba58d6
fix make python
pyalex 4c40307
fixes for 3.6
pyalex 4c6b9b4
add missing fixtures
pyalex 6e2a847
test with setuid
pyalex d198d2f
use provided postgres
pyalex 4959565
use postgres host
pyalex 6255772
debug
pyalex 51d39cc
debug
pyalex c29fd20
debug
pyalex 8596aad
try set uid
pyalex f5cb160
patch postgres executor
pyalex cbf0ef4
patch postgres executor
pyalex b8951ac
install postgres
pyalex 8a085bf
always set uid
pyalex 0fd02b8
su
pyalex 85fbb03
debug
pyalex 8cc59b6
debug
pyalex d1f4fb6
debug
pyalex 3389dbe
pass PATH
pyalex 96a6b50
pass HOME
pyalex e04cd9f
external services fixtures
pyalex e07f713
gcp tests
pyalex dbf1ad1
fix test requirements install
pyalex 93e9393
fixing tests requirements
pyalex 2b65977
fixing tests requirements
pyalex ee69301
make historical test storage agnostic
pyalex 07eaf08
unify test options
pyalex 6eadb67
fix noopredis init
pyalex 54946ac
fix streaming test
pyalex 1dd9768
gcp test
pyalex e5e983e
lint
pyalex adaa91c
test scopes
pyalex 6b16c7a
add dataproc to ci requiremenets
pyalex ce63eca
zookeper fixture
pyalex d5ab3dc
freeze grpcio
pyalex 6beb7dc
return pytest requirements to ci
pyalex 41793b9
freeze google sdk
pyalex e1dd22e
fix dataproc cancel
pyalex 26fd919
check kafka consumers before publish
pyalex 84833b3
enable tests with auth
pyalex b07c8a4
wait for job to stop
pyalex 3fbcb6a
unique kafka topic name per test
pyalex 9a18ed5
upgrade kafka python
pyalex e6ef09a
format
pyalex c5e3952
use redis cluster in gcp tests
pyalex ac26fb2
e2e redis internal lb
pyalex d063045
pass redis cluster config to serving
pyalex a58ed22
pass redis cluster config to serving
pyalex 031c8ba
handle kafka admin incorrect parsing
pyalex 5f706cf
format
pyalex 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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,18 @@ | ||
| externalAccess: | ||
| enabled: true | ||
| service: | ||
| loadBalancerIPs: | ||
| - $feast_kafka_ip | ||
| annotations: | ||
| cloud.google.com/load-balancer-type: Internal | ||
| loadBalancerSourceRanges: | ||
| - 10.0.0.0/8 | ||
| - 172.16.0.0/12 | ||
| - 192.168.0.0/16 | ||
|
|
||
| persistence: | ||
| enabled: false | ||
|
|
||
| zookeeper: | ||
| persistence: | ||
| enabled: false |
This file contains hidden or 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,17 @@ | ||
| cluster: | ||
| nodes: 3 | ||
| replicas: 0 | ||
| externalAccess: | ||
| enabled: true | ||
| service: | ||
| annotations: | ||
| cloud.google.com/load-balancer-type: Internal | ||
| loadBalancerIP: | ||
| - $feast_redis_1_ip | ||
| - $feast_redis_2_ip | ||
| - $feast_redis_3_ip | ||
|
|
||
| persistence: | ||
| enabled: false | ||
|
|
||
| usePassword: false |
This file contains hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.
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.
The self-hosted runners don't clean up. If you aren't using docker then all the state will remain. I dont think this test is safe, it's going to fall over for our current runners because they are already so unstable (long running stateful pods)
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 don't use docker. It's regular test, which github runner was designed for, hence there should be way to easily clean up. Right now for example python w/ environment (and installed packages) being cleaned up by python action
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.
Also, we're on preemptible nodes right now. So we have additional clean up
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.
ok, but self-hosted runners don't clean up right? state just hangs around as far as I know.