-
-
Notifications
You must be signed in to change notification settings - Fork 386
Use Windows 2022 rather than deprecated Windows 2019 on azure #1438
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
Conversation
Reviewer's GuideThis PR migrates all Windows Azure VM images from the deprecated 2019 version to 2022, refactors conda environment creation and build steps into a more unified workflow, and updates Ubuntu job definitions to use Ubuntu 24.04. Sequence Diagram: Updated Windows Build ProcesssequenceDiagram
participant AzurePipelines
participant WindowsVM2022 as "Windows VM (2022)"
AzurePipelines->>WindowsVM2022: Trigger Build Job
WindowsVM2022->>WindowsVM2022: Initial Setup (Add conda to PATH, Create 'build_env')
opt If OS is Windows 2022
WindowsVM2022->>WindowsVM2022: Call vcvars64.bat (VS 2022)
WindowsVM2022->>WindowsVM2022: Set CC=cl.exe, CXX=cl.exe
WindowsVM2022->>WindowsVM2022: Activate 'build_env' (using 'call activate')
WindowsVM2022->>WindowsVM2022: Execute conda-build
end
WindowsVM2022-->>AzurePipelines: Build Complete
Flow Diagram: Refactored Conditional Build Execution Logicgraph TD
Start["Start CI Job"] --> CondaConfig["Conda Config & Info"]
CondaConfig --> CreateEnv["Create Anaconda Environment ('build_env')"]
CreateEnv --> CheckOS{"Is VM Windows?"}
CheckOS -- "Yes (Windows 2022)" --> WinScript_Header["Windows Build Steps"]
WinScript_Header --> WinScript_vcvars["call vcvars64.bat (VS 2022)"]
WinScript_vcvars --> WinScript_setenv["set CC=cl.exe, CXX=cl.exe"]
WinScript_setenv --> WinScript_activate["call activate build_env"]
WinScript_activate --> WinScript_build["conda-build"]
WinScript_build --> EndBuild["End Build"]
CheckOS -- "No (e.g., Ubuntu)" --> NonWinScript_Header["Non-Windows Build Steps"]
NonWinScript_Header --> NonWin_activate["source activate build_env"]
NonWin_activate --> NonWin_build["conda-build"]
NonWin_build --> EndBuild
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @tpaviot - I've reviewed your changes - here's some feedback:
- Verify that 'ubuntu-24.04' is actually supported in your Azure DevOps agent pool—if not, those new Ubuntu jobs will fail to spin up.
- The env: block in the windows-2022 build step looks misaligned under the script—ensure CC, CXX, PYTHONBUFFERED, etc. are properly indented as top-level env mappings, not inside the script body.
- Consider extracting the common conda-build invocation into a shared template or step to reduce duplication between Windows and non-Windows jobs.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
where cl.exe | ||
set CC=cl.exe | ||
set CXX=cl.exe | ||
call activate build_env |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: Prefer 'conda activate' over 'activate' on Windows
'conda activate' is the recommended approach for newer Conda versions and ensures future compatibility.
call activate build_env | |
conda activate build_env |
Summary by Sourcery
Switch CI to use Windows Server 2022 instead of the deprecated 2019 image, migrate all Ubuntu builds to 24.04, and refactor the conda-build workflow to consolidate environment setup and build steps.
Enhancements:
CI: