-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[release/6.0-staging] Fix #103485 #103935
[release/6.0-staging] Fix #103485 #103935
Conversation
Tagging subscribers to this area: @dotnet/runtime-infrastructure |
1efaa5b
to
03cf81c
Compare
ping @agocke @jkoritzinsky -- test failures are known. Build Analysis seems to be blocked by some infra issues that look unrelated to this change. |
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.
LGTM, but I'm not too familiar with the crossdac build.
|
checkDownloadedFiles: true | ||
- template: /eng/pipelines/common/templates/global-build-step.yml | ||
parameters: | ||
buildArgs: -s linuxdac+alpinedac -c $(_BuildConfig) |
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.
the only thing is the xdac's used to also be uploaded to the crossDacArtifactsContainer. This is not blocking since they matter the most for PRs and they are there.
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.
I see the installers and cross-dac artifacts in the test official build, LGTM
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.
LGTM. Approved. Please go ahead and merge.
CC .@carlossanlop |
89447d5
into
dotnet:release/6.0-staging
This PR adds back jobs for building Linux installers and CrossDAC packages as part of the .NET 6 official build by surgically merging various dependent changes. There are a couple changes unrelated to this work, but otherwise needed to unblock official .NET 6 builds (like upgrading to macOS 12 images). Fixes #103485. Official build
@jkoritzinsky @agocke PTAL; thanks!