-
Notifications
You must be signed in to change notification settings - Fork 7.4k
Patch Tuesday for March 2025 #44354
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
Merged
BillyONeal
merged 20 commits into
microsoft:master
from
BillyONeal:patch-tuesday-march-2025
Mar 18, 2025
Merged
Patch Tuesday for March 2025 #44354
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
55c719b
Remove ImageOverride now that January- image is gone.
BillyONeal a1ded03
Add Dockerfile for Linux
BillyONeal 4048f0c
Update VS
BillyONeal 6a63561
Move docker .sh out of android since we use it on Linux too.
BillyONeal 23f6565
Remove clean: resources steps for runs which use blank VMs every time…
BillyONeal ea9e5f5
Update Android docker image.
BillyONeal cee532e
Upgrade Android build to Gen2 VM.
BillyONeal 06274f5
Save a docker invocation by running boostrap in mariner.
BillyONeal c9c34f5
Update Windows image.
BillyONeal 79ec960
Dockerize Linux runs in AzDO.
BillyONeal 1a27801
Wire up tool SHA for Android.
BillyONeal 582b00a
Give up on the 'gen2' thing. I guess 1ES just really hates that
BillyONeal 0a75939
Try moving docker stuff to the temp disk
BillyONeal 869d75a
Fix building vcpkg-tool from source.
BillyONeal 00b2169
Fix mnn when downloads and packages are on different volumes.
BillyONeal 6a5c2db
[robotraconteur] Add missing zlib dependency
BillyONeal 4c18b31
Turn off tensorflow. 3 years out of date and fails in Docker
BillyONeal 5537d84
Passing, remove from fail lists.
BillyONeal c137f84
fix copy typo
BillyONeal 90aa711
[robotraconteur] Remove spurious pthread rt z dl from OpenSSL depende…
BillyONeal File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| diff --git a/RobotRaconteurCore/CMakeLists.txt b/RobotRaconteurCore/CMakeLists.txt | ||
| index fbef9b4..85054f5 100644 | ||
| --- a/RobotRaconteurCore/CMakeLists.txt | ||
| +++ b/RobotRaconteurCore/CMakeLists.txt | ||
| @@ -336,12 +336,12 @@ if(UNIX AND NOT APPLE AND NOT ANDROID) | ||
|
|
||
| target_include_directories(RobotRaconteurCore PRIVATE ${DBUS_INCLUDE_DIR} ${DBUS_INCLUDE_ARCH_DIR} | ||
| ${LIBUSB_INCLUDE_DIR}) | ||
| - target_link_libraries(RobotRaconteurCore PRIVATE OpenSSL::SSL OpenSSL::Crypto pthread rt z dl) | ||
| + target_link_libraries(RobotRaconteurCore PRIVATE OpenSSL::SSL OpenSSL::Crypto) | ||
| target_compile_definitions(RobotRaconteurCore PRIVATE ROBOTRACONTEUR_USE_OPENSSL) | ||
| else() | ||
| target_include_directories(RobotRaconteurCore PRIVATE ${DBUS_INCLUDE_DIR} ${DBUS_INCLUDE_ARCH_DIR} | ||
| ${LIBUSB_INCLUDE_DIR}) | ||
| - target_link_libraries(RobotRaconteurCore PUBLIC OpenSSL::SSL OpenSSL::Crypto pthread rt z dl) | ||
| + target_link_libraries(RobotRaconteurCore PUBLIC OpenSSL::SSL OpenSSL::Crypto) | ||
| target_compile_definitions(RobotRaconteurCore PUBLIC ROBOTRACONTEUR_USE_OPENSSL) | ||
| endif() | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,6 +3,10 @@ | |
| # | ||
|
|
||
| parameters: | ||
| - name: vcpkgToolSha | ||
| displayName: 'Custom SHA of vcpkg-tool to use rather than bootstrap' | ||
| type: string | ||
| default: 'use default' | ||
| - name: jobName | ||
| type: string | ||
| - name: tripletPattern | ||
|
|
@@ -15,10 +19,7 @@ jobs: | |
| condition: and(succeeded(), contains('^${{ replace(parameters.jobName, '_', '-') }}$', '${{ parameters.tripletPattern }}')) | ||
| pool: | ||
| name: PrAnd-WUS | ||
| workspace: | ||
| clean: resources | ||
| timeoutInMinutes: 1440 # 1 day | ||
| cancelTimeoutInMinutes: 1 | ||
| variables: | ||
| - name: WORKING_ROOT | ||
| value: /mnt/vcpkg-ci | ||
|
|
@@ -27,7 +28,9 @@ jobs: | |
| - name: ANDROID_NDK_HOME | ||
| value: /android-ndk-r27c | ||
| - name: ANDROID_DOCKER_IMAGE | ||
| value: 'vcpkgandroidwus.azurecr.io/vcpkg-android:2025-02-14' | ||
| value: 'vcpkgandroidwus.azurecr.io/vcpkg-android:2025-03-12' | ||
| - name: LINUX_DOCKER_IMAGE | ||
| value: 'vcpkgandroidwus.azurecr.io/vcpkg-linux:2025-03-12' | ||
| steps: | ||
| # Note: /mnt is the Azure machines' temporary disk. | ||
| - bash: | | ||
|
|
@@ -37,27 +40,50 @@ jobs: | |
| sudo rm -rf ${{ variables.WORKING_ROOT }}/failure-logs | ||
| sudo mkdir ${{ variables.WORKING_ROOT }}/failure-logs -m=777 | ||
| sudo mkdir ${{ variables.VCPKG_DOWNLOADS }} -m=777 | ||
| # Move the docker layers to the temp disk. | ||
| sudo mkdir -p /etc/docker | ||
| echo '{"data-root": "/mnt/docker"}' | sudo tee /etc/docker/daemon.json | ||
| sudo systemctl restart docker | ||
| exit 0 | ||
| displayName: 'Create working directories' | ||
| - bash: ./bootstrap-vcpkg.sh -skipDependencyChecks | ||
| displayName: 'Bootstrap vcpkg' | ||
| condition: eq('use default', '${{ parameters.vcpkgToolSha }}') | ||
| - task: AzureCLI@2 | ||
| displayName: 'Build vcpkg with CMake' | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A nice side effect of having a "builds for Linux" container available is we can fix this missing feature with it. |
||
| condition: ne('use default', '${{ parameters.vcpkgToolSha }}') | ||
| inputs: | ||
| azureSubscription: 'VcpkgPrFleet' | ||
| scriptType: bash | ||
| scriptLocation: 'inlineScript' | ||
| inlineScript: | | ||
| # This is a second pull but the vcpkgToolSha setting is used rarely. | ||
| USER=$(id --user) | ||
| az acr login --name vcpkgandroidwus | ||
| docker pull ${{ variables.LINUX_DOCKER_IMAGE }} | ||
| docker run --init -i --rm \ | ||
| -a stderr \ | ||
| -a stdout \ | ||
| --user $USER \ | ||
| --mount type=bind,source=$(Build.Repository.LocalPath),target=/vcpkg \ | ||
| --workdir /vcpkg \ | ||
| ${{ variables.LINUX_DOCKER_IMAGE }} \ | ||
| /vcpkg/scripts/azure-pipelines/bootstrap-from-source.sh ${{ parameters.vcpkgToolSha }} | ||
| - task: AzureCLI@2 | ||
| displayName: '*** Test Modified Ports' | ||
| inputs: | ||
| azureSubscription: 'VcpkgPrFleet' | ||
| scriptType: bash | ||
| scriptLocation: 'inlineScript' # Be very very careful that the exit code from the last pwsh is reported correctly | ||
| inlineScript: | | ||
| az acr login --name vcpkgandroidwus | ||
| docker pull ${{ variables.ANDROID_DOCKER_IMAGE }} | ||
| docker run --rm \ | ||
| --mount type=bind,source=$(Build.Repository.LocalPath),target=/vcpkg \ | ||
| ${{ variables.ANDROID_DOCKER_IMAGE }} \ | ||
| ./bootstrap-vcpkg.sh | ||
| end=`date -u -d "2 days" '+%Y-%m-%dT%H:%MZ'` | ||
| assetSas=`az storage container generate-sas --name cache --account-name vcpkgassetcachewus --as-user --auth-mode login --https-only --permissions rcl --expiry $end -o tsv` | ||
| binarySas=`az storage container generate-sas --name cache --account-name vcpkgbinarycachewus --as-user --auth-mode login --https-only --permissions rclw --expiry $end -o tsv` | ||
| echo Minting SAS tokens valid through $end | ||
| USER=$(id --user) | ||
| docker run --init -i \ | ||
| az acr login --name vcpkgandroidwus | ||
| docker pull ${{ variables.ANDROID_DOCKER_IMAGE }} | ||
| docker run --init -i --rm \ | ||
| -a stderr \ | ||
| -a stdout \ | ||
| --user $USER \ | ||
|
|
@@ -66,9 +92,10 @@ jobs: | |
| --mount type=bind,source=/mnt/vcpkg-ci,target=/mnt/vcpkg-ci \ | ||
| --env X_VCPKG_ASSET_SOURCES="x-azurl,https://vcpkgassetcachewus.blob.core.windows.net/cache,$assetSas,readwrite" \ | ||
| --env ANDROID_NDK_HOME="${{ variables.ANDROID_NDK_HOME }}" \ | ||
| --workdir /vcpkg \ | ||
| ${{ variables.ANDROID_DOCKER_IMAGE }} \ | ||
| pwsh \ | ||
| -File /vcpkg/scripts/azure-pipelines/test-modified-ports.ps1 \ | ||
| -File scripts/azure-pipelines/test-modified-ports.ps1 \ | ||
| -Triplet ${{ replace(parameters.jobName, '_', '-') }} \ | ||
| -BuildReason $(Build.Reason) \ | ||
| -BinarySourceStub "x-azblob,https://vcpkgbinarycachewus.blob.core.windows.net/cache,$binarySas" \ | ||
|
|
||
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,45 @@ | ||
| # Create Docker images for vcpkg | ||
|
|
||
| function Build-Image { | ||
| param( | ||
| [string]$Location, | ||
| [string]$ImageName, | ||
| [string]$ContainerRegistryName, | ||
| [string]$Date | ||
| ) | ||
|
|
||
| Push-Location $Location | ||
| try { | ||
|
|
||
| docker build . -t $ImageName | ||
|
|
||
| $remote = [string]::Format('{0}.azurecr.io/{1}:{2}', $ContainerRegistryName, $ImageName, $Date) | ||
| docker tag $ImageName $remote | ||
|
|
||
| docker push $remote | ||
|
|
||
| Write-Host "Remote: $remote" | ||
| } finally { | ||
| Pop-Location | ||
| } | ||
| } | ||
|
|
||
| $Date = (Get-Date -Format 'yyyy-MM-dd') | ||
| $ResourceGroupName = "PrAnd-WUS" | ||
| $ContainerRegistryName = "vcpkgandroidwus" | ||
| $ErrorActionPreference = 'Stop' | ||
|
|
||
| $registry = Get-AzContainerRegistry -ResourceGroupName $ResourceGroupName -Name $ContainerRegistryName | ||
| Connect-AzContainerRegistry -Name $registry.Name | ||
|
|
||
| docker builder prune -f --filter "until=24h" | ||
| Build-Image -Location "$PSScriptRoot/android" ` | ||
| -ImageName "vcpkg-android" ` | ||
| -ContainerRegistryName $ContainerRegistryName ` | ||
| -Date $Date | ||
| Build-Image -Location "$PSScriptRoot/linux" ` | ||
| -ImageName "vcpkg-linux" ` | ||
| -ContainerRegistryName $ContainerRegistryName ` | ||
| -Date $Date | ||
|
|
||
| docker logout |
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| # syntax=docker/dockerfile:1.4 | ||
| # DisableDockerDetector "Used to build the container deployed to Azure Container Registry" | ||
| FROM ubuntu:jammy-20250126 | ||
| ADD provision-image.sh /provision-image.sh | ||
| RUN apt-get update && \ | ||
| apt-get install --no-install-recommends -y curl gnupg ca-certificates | ||
| RUN chmod +x /provision-image.sh && /provision-image.sh |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Next time we 'roll' the docker host images I will merge this into those
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.
Submitted changes to make 1ES capable of handling this for us: https://devdiv.visualstudio.com/XlabImageFactory/_git/XlabImageFactory/pullrequest/620203