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

AWS API Definitions Updated #453

Merged
merged 2 commits into from
Sep 28, 2021
Merged

AWS API Definitions Updated #453

merged 2 commits into from
Sep 28, 2021

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

@github-actions github-actions bot force-pushed the create-pull-request/patch branch from be5faf6 to 9aa6d11 Compare September 23, 2021 06:13
@github-actions github-actions bot force-pushed the create-pull-request/patch branch from 9aa6d11 to 54583cd Compare September 24, 2021 06:12
@github-actions github-actions bot force-pushed the create-pull-request/patch branch from 54583cd to 3b9c6a6 Compare September 25, 2021 06:12
@omus
Copy link
Member

omus commented Sep 27, 2021

I'm going to merge this before #457

@omus
Copy link
Member

omus commented Sep 27, 2021

bors try

bors bot added a commit that referenced this pull request Sep 27, 2021
@bors
Copy link
Contributor

bors bot commented Sep 28, 2021

try

Build failed:

@omus
Copy link
Member

omus commented Sep 28, 2021

Fixing the CI bug in: #463

bors bot added a commit that referenced this pull request Sep 28, 2021
463: Flush I/O before reading from test file r=omus a=omus

This failure occurred when running the CI tests on #453:
```
 high-level s3: Test Failed at /Users/runner/work/AWS.jl/AWS.jl/test/AWS.jl:844
  Expression: read(f, String) == body
   Evaluated: "" == "sample-file-body"
Stacktrace:
 [1] (::var"#41#43"{String, String, String})(f::String, io::IOStream)
   @ Main ~/work/AWS.jl/AWS.jl/test/AWS.jl:844
 [2] mktemp(fn::var"#41#43"{String, String, String}, parent::String)
   @ Base.Filesystem ./file.jl:703
 [3] mktemp(fn::Function)
   @ Base.Filesystem ./file.jl:701
 [4] macro expansion
   @ ~/work/AWS.jl/AWS.jl/test/AWS.jl:842 [inlined]
 [5] macro expansion
   @ /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.6/Test/src/Test.jl:1151 [inlined]
 [6] macro expansion
   @ ~/work/AWS.jl/AWS.jl/test/AWS.jl:801 [inlined]
 [7] macro expansion
   @ /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.6/Test/src/Test.jl:1151 [inlined]
 [8] top-level scope
   @ ~/work/AWS.jl/AWS.jl/test/AWS.jl:800
WARNING: replacing module S3.
WARNING: replacing module S3.
Backend: HTTPBackend: Test Failed at /Users/runner/work/AWS.jl/AWS.jl/test/minio.jl:45
  Expression: read(f, String) == "Hi from Minio"
   Evaluated: "" == "Hi from Minio"
Stacktrace:
  [1] (::var"#162#163")(f::String, io::IOStream)
    @ Main ~/work/AWS.jl/AWS.jl/test/minio.jl:45
  [2] mktemp(fn::var"#162#163", parent::String)
    @ Base.Filesystem ./file.jl:703
  [3] mktemp(fn::Function)
    @ Base.Filesystem ./file.jl:701
  [4] top-level scope
    @ ~/work/AWS.jl/AWS.jl/test/minio.jl:43
  [5] include
    @ ./client.jl:444 [inlined]
  [6] macro expansion
    @ ~/work/AWS.jl/AWS.jl/test/runtests.jl:66 [inlined]
  [7] macro expansion
    @ /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.6/Test/src/Test.jl:1226 [inlined]
  [8] macro expansion
    @ ~/work/AWS.jl/AWS.jl/test/runtests.jl:59 [inlined]
  [9] macro expansion
    @ /Users/julia/buildbot/worker/package_macos64/build/usr/share/julia/stdlib/v1.6/Test/src/Test.jl:1151 [inlined]
 [10] top-level scope
    @ ~/work/AWS.jl/AWS.jl/test/runtests.jl:52
```
– https://github.com/JuliaCloud/AWS.jl/runs/3726649750

As it turns out I already noticed this test as part of #458 but [originally thought it was a minor difference with `AWS.Response`](#458 (comment)). It turns out that it is just a stochastic when using the HTTPBackend. 

Co-authored-by: Curtis Vogt <[email protected]>
@omus
Copy link
Member

omus commented Sep 28, 2021

bors r+

bors bot added a commit that referenced this pull request Sep 28, 2021
453: AWS API Definitions Updated r=omus a=github-actions[bot]

Automated changes by [create-pull-request](https://github.com/peter-evans/create-pull-request) GitHub action

Co-authored-by: mattBrzezinski <[email protected]>
@omus
Copy link
Member

omus commented Sep 28, 2021

Unless I circumvent bors I can't merge #463 so I'll attempt to merge this PR and hope it doesn't fail

@bors
Copy link
Contributor

bors bot commented Sep 28, 2021

Build failed:

@omus
Copy link
Member

omus commented Sep 28, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 28, 2021

@bors bors bot merged commit 46d5521 into master Sep 28, 2021
@bors bors bot deleted the create-pull-request/patch branch September 28, 2021 03:48
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.

2 participants