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

appveyor fails pkg test suspiciously often #13436

Closed
tkelman opened this issue Oct 3, 2015 · 8 comments
Closed

appveyor fails pkg test suspiciously often #13436

tkelman opened this issue Oct 3, 2015 · 8 comments
Labels
packages Package management and loading system:windows Affects only Windows test This change adds or pertains to unit tests

Comments

@tkelman
Copy link
Contributor

tkelman commented Oct 3, 2015

example: https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.9756/job/2iqu8xwxflr4225e

INFO: Cloning METADATA from https://github.com/JuliaLang/METADATA.jl
ERROR: LoadError: LoadError: GitError(Code:ERROR, Class:OS, Failed to receive response: The server returned an invalid or unrecognized response
)
 [inlined code] from libgit2/error.jl:96
 in clone at libgit2/repository.jl:95
 in clone at libgit2.jl:303
 in anonymous at pkg/dir.jl:49
 in cd at file.jl:32
 in path at pkg/dir.jl:17
 in temp_pkg_dir at C:\projects\julia\test\pkg.jl:9
 in include at boot.jl:261
 in include_from_node1 at loading.jl:304
 [inlined code] from util.jl:179
 in runtests at C:\projects\julia\test\testdefs.jl:178
 in anonymous at multi.jl:684
 in run_work_thunk at multi.jl:645
 in remotecall_fetch at multi.jl:734
 [inlined code] from multi.jl:1430
 in anonymous at task.jl:1429
while loading C:\projects\julia\test\pkg.jl, in expression starting on line 169
while loading C:\projects\julia\test\runtests.jl, in expression starting on line 13

This has been happening a lot, and while it might just be connection issues, the fact that it only seems to happen on 32 bit leads me to believe it might be a bug in integer sizes or something similar.

@tkelman tkelman added system:windows Affects only Windows test This change adds or pertains to unit tests packages Package management and loading system:32-bit Affects only 32-bit systems labels Oct 3, 2015
@wildart
Copy link
Member

wildart commented Oct 5, 2015

It relates to handling a HTTP requests. It could be just appveyor glitch.

@tkelman
Copy link
Contributor Author

tkelman commented Oct 5, 2015

If it happened as often on 64 bit as 32 bit I would agree with you. It feels like there may be a datatype size bug hiding somewhere.

@wildart
Copy link
Member

wildart commented Oct 5, 2015

It's WinHttpReceiveResponse call error.

@tkelman
Copy link
Contributor Author

tkelman commented Oct 5, 2015

Ah, interesting. So maybe this falls under the category of "try to reproduce in C, report upstream" if we can get the same issue to show up in a standalone case.

@wildart
Copy link
Member

wildart commented Oct 5, 2015

It looks like https response decryption failed.

@tkelman
Copy link
Contributor Author

tkelman commented Oct 5, 2015

we might be the first users to heavily exercise the winhttp functionality (edit: in libgit2) with mingw compiler, it's possible i or the original author had a bug in the pr upstream that first got that working.

@wildart
Copy link
Member

wildart commented Oct 5, 2015

Not likely, it's WinAPI. It could be solved by proper OS/IIS settings. I could not find any particulars, an error message is too vague and quite common.

@tkelman tkelman changed the title 32 bit appveyor fails pkg test suspiciously often appveyor fails pkg test suspiciously often Oct 9, 2015
@tkelman tkelman removed the system:32-bit Affects only 32-bit systems label Oct 14, 2015
@tkelman
Copy link
Contributor Author

tkelman commented Oct 14, 2015

This hasn't happened for a few days, maybe it was a transient network thing. I'll reopen if it starts coming back frequently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages Package management and loading system:windows Affects only Windows test This change adds or pertains to unit tests
Projects
None yet
Development

No branches or pull requests

2 participants