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

Fix flaky importlib error on Ubuntu Python 3.7 conda CI job #423

Closed
CAM-Gerlach opened this issue Apr 8, 2023 · 2 comments · Fixed by #425
Closed

Fix flaky importlib error on Ubuntu Python 3.7 conda CI job #423

CAM-Gerlach opened this issue Apr 8, 2023 · 2 comments · Fixed by #425
Assignees
Labels
Milestone

Comments

@CAM-Gerlach
Copy link
Member

The Test ubuntu-latest Python 3.7 conda=Yes CI job fails approximately every other time with the error ERROR module 'importlib_metadata' has no attribute 'distributions': Example Run. We should probably look into that and do something about it.

@ccordoba12
Copy link
Member

3.7 is about to reach end-of-life, so I'd suggest to drop it from our CIs instead of investing time trying to fix this

@CAM-Gerlach
Copy link
Member Author

CAM-Gerlach commented Apr 9, 2023

After looking in to this further I found conda/conda#12390 which explains what's going on; basically, the issue is that the Conda solver solution can sometimes be unstable and not fully accurate when only using repodata_current.json; in this case it's happening to the Python 3.7 job, but it could theoretically happen to others as well, or if we bumped the version.

However, the solution is to just use the Mamba solver instead, which not only ensures a more stable solve, but speeds up the (currently slower) conda=yes jobs by an average of approximately 2x (especially when combined with some other optimizations, particularly doing a one-pass env creation and installation rather than installing the Qt bindings separately, which also further improves stability and reduces the chance for issues). Plus the various tweaks end up making our CI scripts less duplicative and more maintainable too.

See PR #425 for those fixes and improvements.

@ccordoba12 ccordoba12 added this to the v2.4.0 milestone Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants