Skip to content

Add sleep to object_store CI#3189

Merged
tustvold merged 2 commits into
apache:masterfrom
tustvold:add-sleep-object-store-ci
Nov 25, 2022
Merged

Add sleep to object_store CI#3189
tustvold merged 2 commits into
apache:masterfrom
tustvold:add-sleep-object-store-ci

Conversation

@tustvold

@tustvold tustvold commented Nov 25, 2022

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #.

Rationale for this change

Frequently this job fails with

* TCP_NODELAY set
* Connected to localhost (::1) port 4443 (#0)
> POST /storage/v1/b HTTP/1.1
> Host: localhost:4443
> User-Agent: curl/7.68.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 22
> 
} [22 bytes data]
* upload completely sent off: 22 out of 22 bytes
* Empty reply from server

100    22    0     0  100    22      0  22000 --:--:-- --:--:-- --:--:-- 22000

https://github.com/apache/arrow-rs/actions/runs/3546438851/jobs/5955482110

Where success looks like

* TCP_NODELAY set
* Connected to localhost (::1) port 4443 (#0)
> POST /storage/v1/b HTTP/1.1
> Host: localhost:4443
> User-Agent: curl/7.68.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 22
> 
} [22 bytes data]
* upload completely sent off: 22 out of 22 bytes
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Content-Type: application/json
< Date: Fri, 25 Nov 2022 07:06:50 GMT
< Content-Length: 151
< 
{ [151 bytes data]

It is possible this is some sort of startup race, so lets add a sleep and see if it helps

What changes are included in this PR?

Are there any user-facing changes?

@tustvold tustvold added the development-process Related to development process of arrow-rs label Nov 25, 2022
alamb
alamb approved these changes Nov 25, 2022
View reviewed changes
Comment thread .github/workflows/object_store.yml
@tustvold @alamb
Update .github/workflows/object_store.yml
36d8240 
Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
@tustvold tustvold merged commit 3998bed into apache:master Nov 25, 2022
@ursabot

ursabot commented Nov 25, 2022

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = 4c37ec2 and contender = 3998bed. 3998bed is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Skipped ⚠️ Benchmarking of arrow-rs-commits is not supported on ec2-t3-xlarge-us-east-2] ec2-t3-xlarge-us-east-2
[Skipped ⚠️ Benchmarking of arrow-rs-commits is not supported on test-mac-arm] test-mac-arm
[Skipped ⚠️ Benchmarking of arrow-rs-commits is not supported on ursa-i9-9960x] ursa-i9-9960x
[Skipped ⚠️ Benchmarking of arrow-rs-commits is not supported on ursa-thinkcentre-m75q] ursa-thinkcentre-m75q
Buildkite builds:
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alamb alamb alamb approved these changes

Assignees

No one assigned

Labels

development-process Related to development process of arrow-rs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tustvold @ursabot @alamb