-
Notifications
You must be signed in to change notification settings - Fork 216
Add .NET6 to CI scripts #1605
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
Add .NET6 to CI scripts #1605
Changes from all commits
f68963f
bf00e6c
7c7befc
4077ffc
502f683
05194b5
1b21dd8
6b98b60
83ff6c1
5963785
9e96240
402d230
bf05f17
c6d9a24
8d1e958
7251ce1
eabfdd6
55f430f
a567704
f94f1bb
7db3d5a
06762db
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ $ErrorActionPreference = "Stop" | |
|
|
||
| # Install visualstudio2019 and workloads needed to build ASP.NET and .NET Core apps | ||
| Invoke-WebRequest -UseBasicParsing ` | ||
| -Uri "https://download.visualstudio.microsoft.com/download/pr/cb1d5164-e767-4886-8955-2df3a7c816a8/b9ff67da6d68d6a653a612fd401283cc213b4ec4bae349dd3d9199659a7d9354/vs_BuildTools.exe" ` | ||
| -Uri "https://download.visualstudio.microsoft.com/download/pr/ce8663b0-08ed-410a-9f5d-4f9469d1b2cb/73271b3d53a4e50e65e2e918a8c1100d2681c17bc418e03513c9f0554609ff8a/vs_BuildTools.exe" ` | ||
|
Contributor
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. This updates it to the vs 2022 version. |
||
| -OutFile "C:\tools\vs_BuildTools.exe" | ||
|
|
||
| Start-Process "C:\tools\vs_BuildTools.exe" -ArgumentList "--add", "Microsoft.VisualStudio.Component.NuGet", ` | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,28 @@ | ||
| :: | ||
| :: This script runs the dotnet build without the Full Framework projects | ||
| :: | ||
| dotnet sln remove sample/AspNetFullFrameworkSampleApp/AspNetFullFrameworkSampleApp.csproj | ||
| dotnet sln remove src/Elastic.Apm.AspNetFullFramework/Elastic.Apm.AspNetFullFramework.csproj | ||
| dotnet sln remove test/Elastic.Apm.AspNetFullFramework.Tests/Elastic.Apm.AspNetFullFramework.Tests.csproj | ||
| dotnet sln remove test/Elastic.Apm.SqlClient.Tests/Elastic.Apm.SqlClient.Tests.csproj | ||
| dotnet sln remove test/Elastic.Apm.EntityFramework6.Tests/Elastic.Apm.EntityFramework6.Tests.csproj | ||
| dotnet sln remove test/Elastic.Apm.MongoDb.Tests/Elastic.Apm.MongoDb.Tests.csproj | ||
|
|
||
| :: Remove startup hooks tests, which are tested separately- require agent zip to be built | ||
| dotnet sln remove test/Elastic.Apm.StartupHook.Tests/Elastic.Apm.StartupHook.Tests.csproj | ||
|
|
||
| :: Remove profiler tests, which are tested separately- require profiler to be built | ||
| dotnet sln remove test/Elastic.Apm.Profiler.Managed.Tests/Elastic.Apm.Profiler.Managed.Tests.csproj | ||
|
|
||
| :: TODO: Test only - building this seems to fail | ||
| dotnet sln remove test/Elastic.Apm.StaticImplicitInitialization.Tests/Elastic.Apm.StaticImplicitInitialization.Tests.csproj | ||
| dotnet sln remove test/Elastic.Apm.StaticExplicitInitialization.Tests/Elastic.Apm.StaticExplicitInitialization.Tests.csproj | ||
|
|
||
| :: | ||
| :: This script runs the tests and stored them in an xml file defined in the | ||
| :: LogFilePath property | ||
| :: | ||
| dotnet test -c Release --no-build ^ | ||
| dotnet test -c Release ^ | ||
|
Member
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. Out of curiosity, if the test.bat does also run the build, are the
Contributor
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. Yes, that's intentional. Those projects that we remove with I only know this from experience. If we don't remove those, it'll fail - I saw it :) . Prior to this PR removing happened in the
Member
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. Thanks Greg for the clarification :) |
||
| --verbosity normal ^ | ||
| --results-directory target ^ | ||
| --diag target\diag.log ^ | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -360,3 +360,4 @@ target/ | |
| Cargo.lock | ||
| expanded.rs | ||
|
|
||
| test/Elastic.Apm.Feature.Tests/SpecFlow/userid | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -358,9 +358,6 @@ pipeline { | |
| unstash 'source' | ||
| dir("${BASE_DIR}"){ | ||
| powershell label: 'Install test tools', script: '.ci\\windows\\test-tools.ps1' | ||
| retry(3) { | ||
| bat label: 'Build', script: '.ci/windows/dotnet.bat' | ||
|
Contributor
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. Instead of building it once and then running |
||
| } | ||
| withAzureCredentials(path: "${HOME}", credentialsFile: '.credentials.json') { | ||
| bat label: 'Test & coverage', script: '.ci/windows/test.bat' | ||
| } | ||
|
|
@@ -624,6 +621,7 @@ def dotnet(Closure body){ | |
| ./dotnet-install.sh --install-dir "\${DOTNET_ROOT}" -version '3.0.103' | ||
| ./dotnet-install.sh --install-dir "\${DOTNET_ROOT}" -version '3.1.100' | ||
| ./dotnet-install.sh --install-dir "\${DOTNET_ROOT}" -version '5.0.100' | ||
| ./dotnet-install.sh --install-dir "\${DOTNET_ROOT}" -version '6.0.100' | ||
| """) | ||
| withAzureCredentials(path: "${homePath}", credentialsFile: '.credentials.json') { | ||
| withTerraform(){ | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
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.
Took out verbose logging - it turned out to generate huge noise when I need to look at logs from a failed build. I'd prefer to keep it without verbose logs and only turn it on once we really need it.