-
Notifications
You must be signed in to change notification settings - Fork 293
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
[repo] Bump SDK to 9.0.100 #2316
[repo] Bump SDK to 9.0.100 #2316
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2316 +/- ##
==========================================
- Coverage 73.91% 72.04% -1.88%
==========================================
Files 267 359 +92
Lines 9615 13771 +4156
==========================================
+ Hits 7107 9921 +2814
- Misses 2508 3850 +1342 |
@@ -10,7 +10,7 @@ | |||
<NetMinimumSupportedVersion>net8.0</NetMinimumSupportedVersion> | |||
<NetStandardMinimumSupportedVersion>netstandard2.0</NetStandardMinimumSupportedVersion> | |||
<GenerateDocumentationFile>true</GenerateDocumentationFile> | |||
<AnalysisLevel>latest-all</AnalysisLevel> | |||
<AnalysisLevel>8.0</AnalysisLevel> |
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.
If I don't set the AnalysisLevel to 8.0
, there will be tons of syntax suggestions showing as errors and failing build: #2313.
…ore.App.Ref with version (= 8.0.11)`. However 8.0.11 was released less than 20 min ago. Maybe the restore step had a delay and didn't get the new version yet? D:\a\opentelemetry-dotnet-contrib\opentelemetry-dotnet-contrib\src\OpenTelemetry.Exporter.Geneva\OpenTelemetry.Exporter.Geneva.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.App.Ref with version (= 8.0.11) D:\a\opentelemetry-dotnet-contrib\opentelemetry-dotnet-contrib\src\OpenTelemetry.Exporter.Geneva\OpenTelemetry.Exporter.Geneva.csproj : error NU1102: - Found 128 version(s) in NuGet [ Nearest version: 9.0.0-preview.1.24081.5 ] D:\a\opentelemetry-dotnet-contrib\opentelemetry-dotnet-contrib\src\OpenTelemetry.Exporter.Geneva\OpenTelemetry.Exporter.Geneva.csproj : error NU1102: - Found 0 version(s) in C:\Program Files\dotnet\library-packs Failed to restore D:\a\opentelemetry-dotnet-contrib\opentelemetry-dotnet-contrib\src\OpenTelemetry.Exporter.Geneva\OpenTelemetry.Exporter.Geneva.csproj (in 20.62 sec).
…uild-test-exporter-geneva-integration / build-test (ubuntu-24.04, net8.0) runs on Ubuntu 22.04.1 LTS. If not set, the version is newer. For example, build-test-exporter-influxdb / build-test (ubuntu-latest, net8.0) runs on Ubuntu 22.04.5 LTS.
…fail in GitHub CI for now, both for confluentkafka-integration and stackexchangeredis-integration. The current latest SDK in mcr.microsoft.com/dotnet/sdk:9.0 is 9.0.100-rc.2.24474.11, as shown in CI. I'll revert global.json to use rc.2 for now.
…ow to be inspected later.
@@ -50,8 +50,8 @@ public UnixUserEventsDataTransportTests(ITestOutputHelper testOutputHelper) | |||
this.testOutputHelper = testOutputHelper; | |||
} | |||
|
|||
[SkipUnlessPlatformMatchesFact(TestPlatform.Linux, requireElevatedProcess: true)] | |||
public void UserEvents_Enabled_Succes_Linux() | |||
[Fact(Skip = "This would fail on Ubuntu. Skipping for now.")] |
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.
These two tests are failing (only on Ubuntu) after upgrading to .NET 9. Disabling them for now to be inspected 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.
Consider creating separate issue and link them in the code.
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.
Created #2326.
… stable version now. docker run --rm mcr.microsoft.com/dotnet/sdk:9.0 dotnet --info .NET SDK: Version: 9.0.100 Commit: 59db016f11 Workload version: 9.0.100-manifests.3068a692 MSBuild version: 17.12.7+5b8665660 The image info is: `Digest: sha256:7d24e90a392e88eb56093e4eb325ff883ad609382a55d42f17fd557b997022ca`. Earlier today, the version was rc.2: docker run --rm mcr.microsoft.com/dotnet/sdk:9.0@sha256:c6a6d7995042bc8905825c2f1e2f8717733e90738a09fb0d30f917596f4bcb10 dotnet --info .NET SDK: Version: 9.0.100-rc.2.24474.11 Commit: 315e1305db Workload version: 9.0.100-manifests.cea95dba MSBuild version: 17.12.0-preview-24473-03+fea15fbd1
@@ -50,8 +50,8 @@ public UnixUserEventsDataTransportTests(ITestOutputHelper testOutputHelper) | |||
this.testOutputHelper = testOutputHelper; | |||
} | |||
|
|||
[SkipUnlessPlatformMatchesFact(TestPlatform.Linux, requireElevatedProcess: true)] | |||
public void UserEvents_Enabled_Succes_Linux() | |||
[Fact(Skip = "This would fail on Ubuntu. Skipping for now.")] |
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.
Consider creating separate issue and link them in the code.
@@ -104,8 +104,6 @@ jobs: | |||
with: | |||
project-name: Component[OpenTelemetry.Exporter.Geneva] | |||
code-cov-name: Exporter.Geneva | |||
os-list: '[ "ubuntu-24.04" ]' # Note: This may be switched to latest once ubuntu-latest has a kernel version >= 6.8.0-1014-azure | |||
tfm-list: '[ "net8.0" ]' # Note: Should be able to remove this once the above is using ubuntu-latest |
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.
There is couple other occurrences of tfm-list
which should be updated in this file (extended by net9.0)
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 don't think we need to do that until projects actually have net9.0
targets added.
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
Towards #2319.
Changes
AnalysisLevel
to8.0
to minimize changes in this PR.Merge requirement checklist
[ ] Unit tests added/updated[ ] Appropriate: components are not upgraded. Only building with new SDK version.CHANGELOG.md
files updated for non-trivial changes[ ] Changes in public API reviewed (if applicable)