-
Notifications
You must be signed in to change notification settings - Fork 93
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
Scripts jobs are failing #1699
Comments
Easy to reproduce locally
|
I see two possibilities
|
what do you mean? I guess that updating |
Even if you manage to fix the missing dll (via hack), there will be next one.
|
Checked the file version for Lower version of |
@rajkumar-rangaraj, As I remember, you have recommended to put into the store the latest version of required package, so we put the Microsoft.Extensions.Configuration.Abstractions v 7.0.0. Is it still valid recommendation or we should refer 3.1.0? |
How about fixing it by running TestApplication.Smoke instead of @dszmigielski Can you do it as part of #1604 as it is very related to each other? |
Testing with dotnet.exe may not be a good idea. It is a self-contained executable. Our implementation does not work for self-contained apps as they don't support AdditionalDeps. As @pellared recommended we should move away from To answer the other question, AdditionalDeps/RuntimeStore should always have the latest version of the package for the app to use the correct version of assembly. |
Fixed in scope of #1703 |
https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/actions/runs/3581979634
The text was updated successfully, but these errors were encountered: