Skip to content

Conversation

@AsakerMohd
Copy link
Contributor

Description of changes:
Set current .NET SDK release for .NET E2E Nuget instead of the latest version because the new ADOT .NET SDK release will introduce breaking changes. This will fix the current Nuget E2E failure.

Copy link
Contributor

@vastin vastin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AsakerMohd AsakerMohd merged commit f6bc748 into main Nov 2, 2024
1 check passed
@AsakerMohd AsakerMohd deleted the nugetHardCode branch November 2, 2024 00:19
vastin added a commit that referenced this pull request Nov 12, 2024
*Description of changes:*
Set current .NET SDK release for .NET E2E tests instead of the latest
version because the new ADOT .NET SDK release will introduce breaking
changes. It will prevent E2E EKS test failures when new EKS addon (which
will use the new ADOT .NET SDK) is released. It is related to
#310
and
#309.

The EKS addon installation script is updated in `dotnet-sdk-1_3_2`
branch as
aws-observability/application-signals-demo@main...dotnet-sdk-1_3_2.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
AsakerMohd added a commit that referenced this pull request Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants