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

Flaky tests #42

Open
rvagg opened this issue May 26, 2020 · 6 comments
Open

Flaky tests #42

rvagg opened this issue May 26, 2020 · 6 comments

Comments

@rvagg
Copy link
Member

rvagg commented May 26, 2020

We're getting this pretty regularly in node-gyp CI: https://travis-ci.com/github/nodejs/node-gyp/jobs/339883554

The command "npm test" exited with 0.
10714.02s$ GYP_MSVS_VERSION=2015 GYP_MSVS_OVERRIDE_PATH="C:\\Dummy" python -m pytest
1072============================= test session starts ==============================
1073platform linux -- Python 3.5.6, pytest-4.6.6, py-1.7.0, pluggy-0.13.1
1074rootdir: /home/travis/build/nodejs/node-gyp
1075collected 27 items                                                             
1076
1077gyp/pylib/gyp/MSVSSettings_test.py F.F.....                              [ 29%]
...

I don't really know how to interpret the log output that follows, but it's been consistent. I can restart it in Travis and it should pass again, this shows up in maybe 50% of Travis runs and it seems to be most common on Node.js 12 + Python 3.5, but I don't have numbers to back that up.

@ryzokuken
Copy link
Contributor

Thanks for reporting this. @targos can you reproduce this fail locally?

@targos
Copy link
Member

targos commented May 26, 2020

It's probably the same problem as #7

@targos
Copy link
Member

targos commented May 26, 2020

I don't have python 3.5 to test locally

@ryzokuken
Copy link
Contributor

Good catch, it is infact that problem. It'd be a shame to not support Python 3.5. I'd try to look into it in a week or so.

@cclauss
Copy link
Contributor

cclauss commented May 26, 2020

Python 3.5 EOLs in September https://devguide.python.org/#status-of-python-branches

@cclauss
Copy link
Contributor

cclauss commented Jun 22, 2021

Can this issue be closed?

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

No branches or pull requests

4 participants