Commit Graph

28 Commits

Author SHA1 Message Date
Timothy Mothra dfd1f25c95
Add `AnalysisLevel` to Common.props (#3960)
* add AnalysisLevel to Common.props

* fix CA2017
2022-12-02 12:42:56 -08:00
Mikel Blanchard e6d39b8b14
[Tracing] Improve dependency injection support in tracing build-up using SDK (#3533)
* TracerBuilder depedency injection improvements.

* Fixes and API updates.

* Update skipped test.

* Tweak.

* Tweaks.

* Test fix.

* Turn on options in TracerProviderBuilderBase.

* Updated CHANGELOGs.

* Added XML comments to AddOpenTelemetryTracing methods.

* Tweaks.

* Added export helpers and restored removed API from hosting library so that there is no breakage during upgrades.

* Export extension tweaks.

* Tweak.

* Tracer builder improvements.

* Fix hosting multiple configurations test.

* Cleanup.

* Cleanup and fixes.

* Added "UseOpenTelemetry" extensions.

* Cleanup.

* Code review.

* Code review.

* Rename SDK extension "Configure" and restore "Add" in the hosting lib.

* Doc clarification.

* Some test coverage.

* README updates.

* Fix double dispose of TracerProvider when using Sdk.CreateTracerProviderBuilder.

* AddExporter tests.

* More test coverage.

* More test coverage.

* Code review.
2022-08-29 13:14:24 -07:00
Mikel Blanchard 58ee541cff
[net7-main] Fixes to get solution building in VS (#3552)
* Fixes to get solution building in VS with net7 changes.

* More fixes + some code review.

* CONTRIBUTING update.

* Attempting to fix integration tests.

* Attempting to fix integration tests.

* Attempt to fix integration tests.

* Attempt to fix integration tests.
2022-08-05 10:49:06 -07:00
Travis Illig d55de344f1
Fix IDE0090: 'using' statement can be simplified. (#3017) 2022-03-10 21:32:41 -08:00
Travis Illig 2a97920ff0
Fix IDE0090: 'new' expression can be simplified. (#3012)
* Set rule for new() usage.

* Fix IDE0090 "'new' expression can be simplified"
2022-03-10 08:57:53 -08:00
Alan West a2c1b6e69e
Invoke SDK initialization when using OpenTelemetry.Extensions.Hosting (#2901) 2022-02-16 06:48:45 -08:00
ktmitton 76d944130f
Reworded documentation based on recommendation from another PR (#2689)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-11-30 14:34:01 -08:00
ktmitton 580a2a5cc0
Removed explicit setting of default guest user (#2682)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-11-26 12:15:36 -08:00
ktmitton d7ea809ef6
Updated the target frameworks for the microservice example projects to net6.0 to support the c# 10 requirement during build through the docker pipeline (#2677) 2021-11-24 14:13:45 -08:00
Michael Maxwell baa37f3a05
Remove 'en-us' from URLs (#2329) 2021-09-09 11:24:47 -07:00
Austin Tan c14177996a
Rely on DefaultResource's Service.Name for Exporters (#1768)
* Jaeger and API changes and ProviderMethod

* Changelog

* Fix whitespace

* Not just checking first element of default resource but querying for servicename

* Including the servicename fallback in Zipkin options

* Resolving tests

* we previously removed the adapting of new servicenames from resources in jaeger, this undoes that

* Removing servicename from Jaeger ctor

* re-adding servicename to Ctor for Jaeger

* CodeBlanch 's implementation for preserving servicename

* Scrubbed ServiceName from ZipkinExporterOptions

* Messed up ternary operator order again

* reordering using for SA compliance

* more instances of zipkinoptions.servicename

* removed servicename from API

* Changelog in right places

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-02-02 11:04:54 -08:00
Cijo Thomas fe24768521
fix spec repo links to main branch (#1746) 2021-01-29 12:07:34 -08:00
Cijo Thomas 097925980d
Add GlobalPropagators API and have instrumentation default to it (#1428)
* Add GlobalPropagators API and have instrumentation default to it

* changelog

* fix name

* reset after

* min comment address

* move id format to same place as context propagator

* renam

* renai

* text initialize correctly

* modify microsoervice example to pick global propagator

* make Global propagator get only in api

* Add SetDefaultPropagato to SDK
2020-11-02 20:40:31 -08:00
Cijo Thomas 985c5378c5
Propagate context from either created activity or Current (#1431)
* Propagate context from either created activity or Current

* minor wording change
2020-11-02 09:52:35 -08:00
Cijo Thomas 4975fb062b
Rename of Propagators (#1427)
* Rename IPropagator to ITextMapPropagator, TextMapPropagator to TraceContextPropagator

* changelogs

* build fix

* make ITextMapPropagator an abstract class

* Renaming ITextMapPropagator to TextMapPropagator

* CompositeTextMapPropagator rename

* fix changhelog
2020-10-30 12:18:07 -07:00
Eddy Nakamura c7429d25c9
Normalizing format (#1289)
updating gitattributes

updating

updating

updating

Revert "Normalizing format"

This reverts commit b571e430a73f65605481ef0a622a372bc8a787ae.

Updating everything to utf-8

applying lf

Revert "applying lf"

This reverts commit a54947f24ba69eecc931525f2a69647040fdc28d.

updating config

Co-authored-by: Mikel Blanchard <mblanchard@macrosssoftware.com>
2020-09-19 09:45:29 -07:00
Eddy Nakamura c025116c3e
Replacing Tags for TagObjects and AddTag for SetTag (#1227)
* Replacing Tags for TagObjects and AddTag for SetTag

updating tests

updating tests

* removing toString from stackExchange test method

* tentative to solve redis test failure

* updating tests

* updating test

* checking for only one element

* Trying GetTagValue extension in http-in unit test projects.

* Grpc now uses GetTagValue.

* Updated new unit tests to use SetTag.

* Updated Http instrumentation for GetTagValue.

* Update Sql instrumentation to use GetTagValue.

* Updated Redis instrumentation to use GetTagValue.

* Removed references from test projects that weren't needed.

Co-authored-by: Mikel Blanchard <mblanchard@macrosssoftware.com>
2020-09-06 10:13:08 -07:00
Alan West 255889f84a
Centralize package versions for example projects (#1230)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-09-04 13:25:59 -07:00
Cijo Thomas fd7876416f
Renaming to AddOpenTelemetryTracing (#1209)
* AspNet Instrumentation docs

* Renaming to AddOpenTelemetryTracing

* Revert "AspNet Instrumentation docs"

This reverts commit 4cbb620533.

* typo

* changelog entry
2020-08-31 17:32:29 -07:00
Eddy Nakamura dd4bd3e0bd
Renaming ITextFormat to IPropagator (#1190)
* Renaming ITextFormat to IPropagator

* updating changelog

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-08-28 07:46:14 -07:00
Eddy Nakamura da72b13b72
Renaming TraceContextFormat/B3Format/BaggageFormat (#1175)
* Renaming TraceContextFormat to TextMapPropagator

updating changelog

renaming b3Format to B3PRopagator

renaming BaggageFormat to BaggagePropagator

renaming file

* renaming variables after merge

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-08-27 18:59:09 -07:00
Mikel Blanchard d11dc0a962
CorrelationContext -> Baggage (#1106)
* Baggage context part 1.

* A little cleanup.

* Return strings instead of objects. Removed default value in ctor because it generates a compiler warning about struct default constructor being used.

* Updated SDK & API projects for BaggageContext.

* Fixed broken areas, except for CorrelationContext tests.

* First round of tests and bug fixes.

* FIxed up HTTP instrumentation tests.

* Fixed up shim tests.

* More tests and improvements.

* Fixed broken test.

* Test coverage and bug fixes.

* Deterministic GetHashCode + tests.

* Added a link to the baggage api.

* Updated CHANGELOG.

* Renamed BaggageContext -> Baggage.

* CHANGELOG update.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-08-27 16:03:38 -07:00
Cijo Thomas 6377d6bd87
Renamed extension methods from AddOpenTelemetry to AddOpenTelemetryTracerProvider (#1138)
* Renamed extension methods from AddOpenTelemetry to AddOpenTelemetryTracerProvider

* ununsed removal

* remove unnecessary test

* build fix
2020-08-21 21:28:10 -07:00
Mikel Blanchard dcaea5bd45
Baggage + CorrelationContext improvements by Eddy & Mike (#1048)
* Support W3C Baggage spec.

* Moved baggage propagation to its own ITextFormat. Removed IsInjected.

* updating some tests

* creating nw files

* updating files

* buildable in release

* adding baggage tests

* updating tests

* updating default textformat for http instrumentation

* Removed a few null checks.

* Removed DistributedContext. Drive CorrelationContext off of Activity.Baggage.

* updating issues after merge

* updating based on sanity check

* updating baggage test

* updating tests

* reiley's comments

* move to using

* Updates for http-in and http-out. Updated CHANGELOGs.

* Adding tests.

* updating correlation context

* Added test for TraceContextFormat + BaggageFormat used together.

* Fixed broken tests.

* Code review.

* Test fixup.

* updating order

* updating tests

* updating tests, adding dispose, clearing objects

* updating changelog

* Use "Baggage" instead of "baggage" as the header name.

* Added some basic support for the Baggage limits specified in the spec.

* Fixed and improved ITextFormat log messages.

* Rename TextFormatContext -> PropagationContext.

* Updated ITextFormat implementations so they don't double-extract.

Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-08-18 15:39:59 -07:00
Cijo Thomas 7f2e672dfa
Rename UseExporter to AddExporter (#1066)
* Rename UseZipkinExporter to AddZipkinExporter

* Rename ZPages exporter extension

* rename jaeger to AddJaeger

* Otlp Exporter rename

* rename extension helper class

* Rename zpages

* comments adressed

* markdwonlint ix
2020-08-13 09:27:00 -07:00
Reiley Yang ce0bf3f723
rename AddActivitySource to AddSource, use params (#1036) 2020-08-10 18:02:50 -07:00
Cijo Thomas 6e53eb2056
MicroService example to depend on project reference. (#1031) 2020-08-10 10:36:25 -07:00
Alan West bd738f0447
End-to-end example applications (#936)
* Suite of example applications demonstrating context propagation with RabbitMQ

* Put System.* namespaces first

* Newline at EOF

* Fix markdownlint errors

* markdownlint fix

* Refactor WorkerService separating OpenTelemetry related logic from RabbitMQ boilerplate

* Refactor WebApi separating OpenTelemetry related logic from RabbitMQ boilerplate

* Use environment variables for RabbitMQ user/pass

* Add some comments

* Fix RabbitMQ default user/pass

* Use correct ActivityKind

* Add attributes following messaging specification

* Clearing a path through the RabbitMQ jungle

* Log something when sending/receiving a message

* Code style clean up

* Default logging to info level

* Create queue if it does not exist

* Rename controller to SendMessageController

* Refine the readme

* MarkdownCop

* Fix merge snafu

* Add null checks on activity

* Add a missed null check

* Use AddOpenTelemetry in ConfigureServices

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
Co-authored-by: Mikel Blanchard <mblanchard@macrosssoftware.com>
2020-08-09 23:59:10 -07:00