Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update net7.0 branch from main #3410

Merged
merged 24 commits into from
Jun 27, 2022
Merged

Update net7.0 branch from main #3410

merged 24 commits into from
Jun 27, 2022

Conversation

alanwest
Copy link
Member

No description provided.

notcool11 and others added 23 commits June 8, 2022 09:16
@notcool11
Added Jaeger Propagator to Opentelemetry.Extensions.Propagators (#3309)
1da5b86
@pellared @cijothomas
Remove unnecessary bullet in CHANGELOG.md (#3352)
91d7e83 
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
@alanwest
Fix OTLP test (#3357)
c079179 
* Show that test is not doing what you might think it does

* More asserts the merrier

* Show this little test that it has potential
@TimothyMothra
improve test coverage: InMemoryExporter & IDeferredMeterProviderBuild…
2ab6281 
…er (#3345)
@CodeBlanch @cijothomas
[SDK] Circular buffer tweaks + cpu pressure test (#3349)
a80f1f2 
* CircularBuffer tweaks and cpu pressure test.

* Switch to Volatile.Read.

* Perf tweaks.

* Remove race check in debug after doing more testing.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
@CodeBlanch
Fix event name logic + support null categoryname. (#3359)
8b1521f
@CodeBlanch
In-memory Exporter: Buffer log scopes (#3360)
b680ec6 
* Buffer log scopes when using in-memory exporter.

* CHANGELOG update.

* Code review.

* Tests.

* CHANGELOG tweak.
@CodeBlanch
SDK: Forward SetParentProvider to children of CompositeProcessor (#3368)
a58c7a3
@CodeBlanch
Examples: Fix ParentProvider not being set on MyFilteringProcessor (#…
870eb92 
…3370)

* Fix ParentProvider not being set on MyFilteringProcessor example.

* Added XML comments.

* Tweak.

* Typo.
@CodeBlanch
Logs: Add helper ctors & forceflush on OpenTelemetryLoggerProvider (#…
17e6d76 
…3364)

* Add helper ctors & forceflush on OpenTelemetryLoggerProvider.

* CHANGELOG update.

* Unit tests.

* Code review.

* Code review.

* Tweak.
@CodeBlanch
SDK: Nullable annotations for base classes & batch + shims to enable …
ee11de9 
…compiler features (#3374)

* Nullable annotations and shims for SDK base classes & batch.

* Target updates.

* Remove System.Collections.Immutable ref.

* ApiCompat attribute exclusions.
@cijothomas
ASPNETCore instrumentation to populate httpflavor tag (#3372)
fd24811
@TimothyMothra
improve test coverage: InMemoryExporter SnapshotMetric (#3344)
6053c03
@cijothomas
Fix AspNetCore metrics to use correct value for http.flavor (#3379)
3c38200 
* Fix AspNetCore metrics to use correct value for http.flavor

* word better
@CodeBlanch
Logs: Add LogRecordData (#3378)
274dc44 
* Add LogRecordData and hook up to LogRecord.

* CHANGELOG update.

* Code review.
@cijothomas
Remove SetHttpFlavor from Http instrumentations (#3381)
d74ac9e
@cijothomas
Asp.Net Core trace instrumentation to populate http schema tag (#3392)
cc5083e
@cijothomas
Try asp.net core tests with inproc server (#3394)
9bb7aaf
@reyang
Dedupe IsPackable (#3398)
2e334df
@utpilla
Remove AspNet and AspNet.TelemetryHttpModule instrumentation projects (…
1f4d97e 
…#3397)
@alanwest
Handle possible exception when initializing the default service name (#…
b7a3a83 
…3405)
@alanwest
HttpClient: Invoke Enrich when SocketException = HostNotFound (#3407)
98b4fde
@Oberon00
Add & use ConfigureResource API. (#3307)
34efdbf
@codecov
Copy link

codecov bot commented Jun 27, 2022 •
edited
Loading

Codecov Report

Merging #3410 (7d007ab) into net7.0 (0f0a96d) will increase coverage by 0.34%.
The diff coverage is 83.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           net7.0    #3410      +/-   ##
==========================================
+ Coverage   85.77%   86.11%   +0.34%     
==========================================
  Files         267      258       -9     
  Lines        9368     9254     -114     
==========================================
- Hits         8035     7969      -66     
+ Misses       1333     1285      -48
Impacted Files Coverage Δ
...mentation.Http/HttpClientInstrumentationOptions.cs 100.00% <ø> (ø)
src/OpenTelemetry/BaseExportProcessor.cs 86.36% <ø> (ø)
src/OpenTelemetry/BatchExportProcessor.cs 81.90% <ø> (ø)
...c/OpenTelemetry/Logs/OpenTelemetryLoggerOptions.cs 78.57% <0.00%> (+5.84%) ⬆️
src/OpenTelemetry/SimpleExportProcessor.cs 70.00% <ø> (ø)
...elemetry/Metrics/MeterProviderBuilderExtensions.cs 63.82% <33.33%> (-7.60%) ⬇️
...AspNetCore/Implementation/HttpInMetricsListener.cs 65.71% <50.00%> (ø)
...ntation.AspNetCore/Implementation/HttpTagHelper.cs 50.00% <50.00%> (ø)
src/OpenTelemetry/Logs/LogRecord.cs 84.00% <64.51%> (-12.97%) ⬇️
src/OpenTelemetry/Logs/LogRecordData.cs 65.62% <65.62%> (ø)
... and 33 more

@alanwest alanwest marked this pull request as ready for review June 27, 2022 19:41
@alanwest alanwest requested a review from a team June 27, 2022 19:41
@alanwest alanwest merged commit 3a90287 into net7.0 Jun 27, 2022
@alanwest alanwest deleted the alanwest/main-to-net70 branch June 27, 2022 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reyang reyang reyang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@alanwest @reyang @notcool11 @pellared @TimothyMothra @CodeBlanch @cijothomas @utpilla @Oberon00