-
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
Start moving over to official templates #99433
Conversation
6cc6b72
to
408ad80
Compare
e7356ff
to
8cb76a6
Compare
/azp run runtime-nativeaot-outerloop |
Azure Pipelines failed to run 1 pipeline(s). |
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.
Looks great!
@@ -165,12 +165,19 @@ jobs: | |||
# Official Build Linux Pool | |||
${{ if and(or(in(parameters.osGroup, 'linux', 'freebsd', 'android', 'tizen'), eq(parameters.jobParameters.hostedOs, 'linux')), ne(variables['System.TeamProject'], 'public')) }}: | |||
name: $(DncEngInternalBuildPool) | |||
demands: ImageOverride -equals Build.Ubuntu.2204.Amd64 | |||
demands: ImageOverride -equals 1es-ubuntu-2204 |
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 Build.Ubuntu.2204.Amd64 image should get the 1ES prerequisites in today's rollout: dotnet/dnceng#2113 ?
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'll leave this for later cleanup rather than depend on the rollout.
* Start moving over to official templates * Parameterize runtime-inner-loop * Use default false for isOfficialBuild in pipeline-with-resources * Respond to PR comments * Code review comments * Use default template args
I believe this successfully ports over all major scenarios. It's certainly possible some of the outerloops will break. I'm happy to do some up-front work to scout the outerloops, but I'm not going to block on them all being done before moving us over.