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 build-docker-image dependency for load tests #8127

Merged
merged 4 commits into from
Aug 29, 2022

Conversation

molinamelendezj
Copy link
Contributor

nightly runs are still broken since now we are building arm64 images, and requires the binfmt_misc hack. So proposing we don't rebuild docker images in different places. and only use the same job and make nightly load depend on that.

nightly load will now use same image build job instead of rebuilding in its script. we should consider using the built in requires in circle ci and get rid of wait_for_docker_images.sh the ci tests too

@molinamelendezj molinamelendezj changed the title move load test to ci workflow conditionally runs add build-docker-image dependency for load tests Aug 28, 2022
Copy link
Contributor

@chainlink chainlink left a comment

Choose a reason for hiding this comment

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

Looks good! FYI We don't use don't require the docker-build to be complete before running the test suites so we can parallelize starting up minikube and building the images

@codecov
Copy link

codecov bot commented Aug 29, 2022

Codecov Report

Merging #8127 (ed41f41) into master (72d079f) will decrease coverage by 20.98%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #8127       +/-   ##
==========================================
- Coverage   22.63%   1.65%   -20.99%     
==========================================
  Files         426     172      -254     
  Lines      113608   71993    -41615     
==========================================
- Hits        25717    1188    -24529     
+ Misses      83489   70705    -12784     
+ Partials     4402     100     -4302     
Impacted Files Coverage Δ
src/server/auth/db.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/watch/op.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/miscutil/log.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/miscutil/math.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/randutil/rand.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/grpcutil/error.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/randutil/unique.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/testutil/naming.go 0.00% <0.00%> (-100.00%) ⬇️
src/internal/storage/track/tracker.go 3.70% <0.00%> (-96.30%) ⬇️
src/internal/task/etcd_queue.go 0.00% <0.00%> (-93.55%) ⬇️
... and 380 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@molinamelendezj molinamelendezj merged commit c0e4365 into master Aug 29, 2022
molinamelendezj added a commit that referenced this pull request Aug 30, 2022
* move load test to ci workflow conditionally runs

* add build-docker-image dependency for load tests

* keep load test step as they were
molinamelendezj added a commit that referenced this pull request Aug 30, 2022
* move load test to ci workflow conditionally runs

* add build-docker-image dependency for load tests

* keep load test step as they were
@molinamelendezj molinamelendezj deleted the molinamelendezj/nightly-load-test-fix branch September 6, 2022 14:29
robert-uhl pushed a commit that referenced this pull request Oct 17, 2022
* move load test to ci workflow conditionally runs

* add build-docker-image dependency for load tests

* keep load test step as they were
robert-uhl pushed a commit that referenced this pull request Oct 17, 2022
* move load test to ci workflow conditionally runs

* add build-docker-image dependency for load tests

* keep load test step as they were
robert-uhl pushed a commit that referenced this pull request Oct 26, 2022
* move load test to ci workflow conditionally runs

* add build-docker-image dependency for load tests

* keep load test step as they were
robert-uhl pushed a commit that referenced this pull request Oct 27, 2022
* move load test to ci workflow conditionally runs

* add build-docker-image dependency for load tests

* keep load test step as they were
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.

None yet

2 participants