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

Test Python Package Installation tests are flaky #1122

Closed
aeisenberg opened this issue Jun 24, 2022 · 3 comments · Fixed by #1124
Closed

Test Python Package Installation tests are flaky #1122

aeisenberg opened this issue Jun 24, 2022 · 3 comments · Fixed by #1124

Comments

@aeisenberg
Copy link
Contributor

These tests often fail for external reasons. We should add some sort of retry on failure for these tests.

@aeisenberg
Copy link
Contributor Author

On a deeper look, this is not a flaky test, it is just failing. Python2 + poetry is not a combination that can be easily supported. Best suggestion is to remove that combination and state that we can't run on it.

More information: actions/setup-python#374

@RasmusWL
Copy link
Member

I looked over actions/setup-python#374 a bit, and I don't think that's the cause of it, since we're not using the setup-python action in those tests at all.

Instead I think we're running into a similar problem, the poetry installation (under Python 3) on windows is not able to find the Python 2 executable.

From the logs of a failure.

Found poetry.lock, will install packages with poetry
+ C:\hostedtoolcache\windows\Python\3.10.5\x64\python.exe -m poetry install --no-root
...
The currently activated Python version 3.10.5 is not supported by the project (^2.7).
Trying to find and use a compatible version. 
  NoCompatiblePythonVersionFound
  Poetry was unable to find a compatible version. If you have one, you can explicitly use it via the "env use" command.

@aeisenberg
Copy link
Contributor Author

Right, I don't think it is caused by actions/setup-python#374, but it is the same underlying issue. Thanks for taking a deeper look here.

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 a pull request may close this issue.

2 participants