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

CI: fix the tests worfklow #5498

Merged
merged 7 commits into from
Nov 1, 2022
Merged

CI: fix the tests worfklow #5498

merged 7 commits into from
Nov 1, 2022

Conversation

maximlt
Copy link
Member

@maximlt maximlt commented Oct 28, 2022

It's really annoying but Python itself gets updated in doit develop_install, despite the channels being correctly constrained, leading to this bug we already encountered a few times now:

2022-10-28T17:04:57.6857340Z The following packages will be DOWNGRADED:
2022-10-28T17:04:57.6857490Z 
2022-10-28T17:04:57.6857770Z   cryptography                        38.0.2-py37hbf3704f_1 --> 38.0.2-py37h1818b49_1 None
2022-10-28T17:04:57.6858210Z   libarchive                               3.5.2-hbf7dfe4_3 --> 3.5.2-hde4784d_3 None
2022-10-28T17:04:57.6858630Z   openssl                                  3.0.5-hfd90126_2 --> 1.1.1q-hfd90126_1 None
2022-10-28T17:04:57.6859050Z   python                        3.7.12-hf3644f1_100_cpython --> 3.7.12-haf480d7_100_cpython None
2022-10-28T17:04:57.6859470Z   sigtool                                  0.1.3-h88f4db0_0 --> 0.1.3-h57ddcff_0 None

@maximlt
Copy link
Member Author

maximlt commented Oct 29, 2022

I'll try a hack we found last year in #5082.

@codecov-commenter
Copy link

codecov-commenter commented Oct 29, 2022

Codecov Report

Merging #5498 (9a85982) into master (a2d003b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5498   +/-   ##
=======================================
  Coverage   88.09%   88.09%           
=======================================
  Files         302      302           
  Lines       62269    62269           
=======================================
  Hits        54858    54858           
  Misses       7411     7411           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jlstevens
Copy link
Contributor

Yay, macos tests are finally green again!

It's really annoying but Python itself gets updated in doit develop_install ...

I agree, wouldn't really expect that command to change python itself...

Ready to merge?

@jlstevens jlstevens merged commit b66d493 into master Nov 1, 2022
@jlstevens jlstevens deleted the ci_fix_tests_macos branch November 1, 2022 15:49
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants