Skip to content

{CI} Pin to Python 3.8 for "Verify src/azure-cli/requirements.*.Windows.txt" #15691

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.

Sign up for GitHub

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

Jump to bottom
Merged
jiasli merged 1 commit into from
Oct 28, 2020
Merged

{CI} Pin to Python 3.8 for "Verify src/azure-cli/requirements.*.Windows.txt" #15691

jiasli merged 1 commit into from
Oct 28, 2020

Conversation

@jiasli
Copy link
Member

@jiasli jiasli commented Oct 28, 2020

Description
Pin to Python 3.8 for CI task "Verify src/azure-cli/requirements.*.Windows.txt".

Using the default 3.x will make ADO use 3.9 and fail:

...
    File "c:\hostedtoolcache\windows\python\3.9.0\x64\lib\distutils\dist.py", line 985, in run_command
      cmd_obj.run()
    File "setup.py", line 159, in run
      raise Exception("ERROR: The 'make' utility is missing from PATH")
  Exception: ERROR: The 'make' utility is missing from PATH
  ----------------------------------------
  ERROR: Failed building wheel for PyNaCl
ERROR: Could not build wheels for cryptography, PyNaCl which use PEP 517 and cannot be installed directly

Not sure if bumping cryptography (#15687) can solve this issue.

@jiasli
Copy link
Member Author

jiasli commented Oct 28, 2020

Should we pin the Python version for Linux and MacOS as well?

@yonzhan yonzhan added this to the S178 milestone Oct 28, 2020
@jiasli jiasli merged commit 6f25ca6 into Azure:dev Oct 28, 2020
@jiasli jiasli deleted the pin-python branch October 28, 2020 07:10
@jiasli jiasli mentioned this pull request Nov 5, 2020
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bim-msft bim-msft Awaiting requested review from bim-msft

@jsntcy jsntcy Awaiting requested review from jsntcy

@arrownj arrownj Awaiting requested review from arrownj

3 more reviewers

@qwordy qwordy qwordy approved these changes

@haroldrandom haroldrandom haroldrandom approved these changes

@fengzhou-msft fengzhou-msft fengzhou-msft approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jiasli jiasli

Labels

None yet

Projects

None yet

Milestone

S178

Development

Successfully merging this pull request may close these issues.

5 participants

@jiasli @qwordy @haroldrandom @fengzhou-msft @yonzhan