Commit Graph

98 Commits

Author SHA1 Message Date
Cijo Thomas 34a95c7fb2
Fix metric OTLP exporter examples (#2291) 2021-08-26 14:23:37 -07:00
Mikel Blanchard d9ad092dd3
Merge to main: New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API (#2270)
* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API. (#2249)

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 2 (#2254)

* Use a single context.Items key for state management to make things more efficient.

* Added a comment for clarity.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 3 (#2256)

* Update ASP.NET instrumentation to use the new TelemetryHttpModule.

* Fixed TelemetryHttpModule not starting its Activity objects. Added an example of request suppression.

* Tweaks an logging improvements.

* Sealed AspNetInstrumentationEventSource.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 4 (#2258)

* Fixed up TelemetryHttpModule unit tests.

* Added tests for the new HasStarted helper and added checks for StartedButNotSampledObj when not sampled.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 5 (#2261)

* Updated ASP.NET instrumentation tests for new TelemetryHttpModule.

* Added a test for the new RecordException option.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 6 (#2264)

* CHANGELOG & README updates.

* Apply suggestions from code review

Co-authored-by: Reiley Yang <reyang@microsoft.com>

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Lint + sanity checks.

* Lint attempt 2.

* Restored CHANGELOG changes lost in merge.

Co-authored-by: Reiley Yang <reyang@microsoft.com>
2021-08-25 11:41:10 -07:00
Cijo Thomas f1e745d5e8
Add histogram support for Prometheus exporter (#2282) 2021-08-24 20:27:04 -07:00
Cijo Thomas 899777eead
Add Gauge export support in Prometheus (#2281) 2021-08-24 13:46:18 -07:00
Mikel Blanchard dc28d6b9a7 AspNet.TelemetryHttpModule public api + test fixes + renames (#2240) 2021-08-10 20:50:27 -07:00
Mikel Blanchard ec118d4fde Rename Microsoft.AspNet.TelemetryCorrelation -> OpenTelemetry.Instrumentation.AspNet.TelemetryHttpModule. Added project to the solution + updated references & namespaces. (#2238) 2021-08-10 20:50:27 -07:00
Utkarsh Umesan Pillai 54d46d80a0
Remove unncessary usings (#2213) 2021-07-29 17:27:28 -07:00
Utkarsh Umesan Pillai 58dc7eae5e
Rename MetricExportIntervalMilliSeconds to MetricExportIntervalMilliseconds (#2194)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-07-26 17:28:27 -07:00
Utkarsh Umesan Pillai d99db276cd
Rename MetricExportInterval to MtricExportIntervalMilliSeconds (#2193) 2021-07-26 15:56:17 -07:00
Cijo Thomas 7c611c8537
Metrics to Main (#2174) 2021-07-22 07:35:38 -07:00
Vladimir Kloz 56357fcd04
Fix the Zipkin exporter registration (#2135)
There was a missing line to set the service name similar to other places
2021-07-14 13:21:34 -07:00
Utkarsh Umesan Pillai 1b1da753d4
Remove unnecessary usings (#2101) 2021-06-23 18:19:11 -07:00
Tom Tan c69694c5ec
Change type of http.status_code in example to int to follow semmantic (#2096)
convension
2021-06-22 11:11:19 -07:00
Mikel Blanchard 6cdfcea67c
Logs: Scope improvements (#2026) 2021-05-06 10:12:43 -07:00
Cijo Thomas f65b650805
AspNetCore instrumentation options to be configurable from DI (#1997) 2021-04-19 22:31:28 -07:00
Mikel Blanchard 0208fd69f5
Dependency injection support (#1889) 2021-03-26 11:36:18 -07:00
Austin Tan caab6d63f4
Update Jaeger README and Example (#1922) 2021-03-18 14:59:45 -07:00
Utkarsh Umesan Pillai 904b7dd1d3
Add OtlpExporter as an option to ASP.NET example application (#1794)
* Add OtlpExporter as an option to ASP.NET example application

* Fixed indentation

* Added an extra line before EOF

* Removing set ApContext switch for unencrypted channel support

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-02-05 11:29:58 -08:00
Utkarsh Umesan Pillai 7687edc350
Updated the endpoint to include http: (#1793) 2021-02-04 17:55:32 -08:00
Cijo Thomas 19cf0c97bf
Remove grpc class options from OTLPExporter. Make Endpoint Uri for all frameworks (#1789)
* Remove grpc class options from OTLPExporter. Make Endpoint Uri

* readme changes

* more comment.
2021-02-04 16:25:49 -08:00
Kyle Tully fbfdb7030d
Added examples for remaining Jaeger Exporter options (#1669)
* Added examples for remaining Jaeger Exporter options

* updated process tags to use index initializer syntax

* removed process tags example

* removing process tags, checking format

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-02-03 10:31:29 -08:00
Cijo Thomas a4562881a0
OTLP exporter to use default Resource to get servicename (#1776)
* OTLP exporter to use default Resource to get servicename

* markdownlint

* fix unitest

* fix test for windows and linux
2021-02-02 15:15:30 -08:00
Cijo Thomas f77280a2b8
Changed default port for OTLP Exporter from 55680 to 4317 (#1775) 2021-02-02 12:07:59 -08: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 d724634d64
Modify console example to show activityprocessor (#1755)
* Modify console example to show activityprocessor

* revert change
2021-01-29 16:20:57 -08:00
Cijo Thomas fe24768521
fix spec repo links to main branch (#1746) 2021-01-29 12:07:34 -08:00
Cijo Thomas 29995ec245
Remove metrics from 1.0.0 release (#1743)
* Remove metrics from 1.0.0 release

* remove docs for metrics

* public api change
2021-01-29 10:05:50 -08:00
Utkarsh Umesan Pillai 17dd4a7d56
Added instructions to setup Jaegar locally using Docker (#1722)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-01-26 13:04:52 -08:00
Alan West 4e4f95bb53
OTLP exporter: use Grpc.Net.Client for netstandard2.1 (#1662)
* Use Grpc.Net.Client for netstandard2.1

* Fix AspNetCore example app

* Better comment for Http2UnencryptedSupport switch

* Update PublicAPI

* Update changelog

* Fix line endings

* Suppress instrumentation on export

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2021-01-04 11:45:21 -08:00
Utkarsh Umesan Pillai c0e27a83fd
Add example for InMemory Exporter (#1639)
* Added example for InMemory exporter

* Update examples/Console/Examples.Console.csproj

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Addressed the PR comments

* Fix comments

* Fix markdown

Co-authored-by: Reiley Yang <reyang@microsoft.com>
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-12-01 15:15:34 -08:00
Alan West 9e5995535f
Clean up documentation of OpenTracing shim (#1637) 2020-12-01 12:05:42 -08:00
Eddy Nakamura 2785ffbbaa
Metrics are obsolete (#1611)
* Metrics are obsolete

* updating changelog

* updating changelog
2020-11-24 09:45:36 -08:00
Mikel Blanchard 932c2587d2
Status Improvements (#1579)
* Send StatusCode as a string. Send error flag in Zipkin & Jaeger when StatusCode == Error.

* Missed one spot.

* Removed code setting otel.status_description to http status description.

* Unit test fixup.

* Update CHANGELOG.

* Updated http instrumentation CHANGELOG.

* Code review.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-17 23:39:28 -08:00
Alan West fd812eea04
Remove ServiceName from JaegerExporterOptions (#1572)
* Remove ServiceName from JaegerExporterOptions

* Update changelog

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-17 14:55:11 -08:00
Alan West 0e3081731a
Remove ServiceName from OtlpExporterOptions (#1557)
* Remove ServiceName from OtlpExporterOptions

* Update PublicAPI

* Update changelog

* Add back default service name when resource does not have service.name

* Fix test

* Fix examples post merge
2020-11-17 14:08:51 -08:00
Mikel Blanchard d24a042964
ResourceBuilder API (#1533)
* Attempting to improve the Resource API.

* XML comment change.

* Updated Resource API based on feedback.

* CHANGELOG update.

* Tweaked variable name.

* CHANGELOG updates.

* Removed OtelEnvResourceDetector from default Resource. Fixed some XML comments.

* Past tense fix for CHANGELOG.

* CHANGELOG formatting.

* Removed the ability to change the environment variable name for OtelEnvResourceDetector.

* Updates for recent refactors.

* Code review.

* Example updates.

* Cleanup.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-17 06:55:09 -08:00
Austin Tan 92ec254342
Extracting package versions to common.props (#1538)
* Most of the common props stuff

* remove coverlet from Zpages Test

* Clean up other mention of coverlet in comment

* Moving appropriate package versions to nonprod

* Common.props removal of packagevers

* reverting some changes to a grpc package

* Fixing tabs and adding one more packagae referecne

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
2020-11-16 14:30:45 -08:00
Cijo Thomas 51806b723a
Mark Activity extension methods not required by spec as internal (#1545)
* Mark activity extensions as internal

* format

* test proj adjustments

* more cleanups

* temporarily remove benchmark. this should be brought back after splitting the project into multiple projects

* fix w3ctracecontext tes

* fix grpctest refs

* Update src/OpenTelemetry.Api/CHANGELOG.md

Co-authored-by: Reiley Yang <reyang@microsoft.com>

Co-authored-by: Reiley Yang <reyang@microsoft.com>
2020-11-15 16:21:00 -08:00
Reiley Yang 6cc8e44d4d
remove unused using statements (#1534) 2020-11-13 06:33:02 -08:00
Alan West 6c2c19a2d8
Better instructions for running gRPC examples (#1526)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-12 16:35:32 -08:00
Alan West 48ea77fd1b
Update documentation, example, and comments (#1527)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-12 14:44:03 -08:00
Cijo Thomas 4385164d85
Added instructions to run console examples (#1525) 2020-11-12 12:19:07 -08:00
Cijo Thomas fa1e17f4ce
OpenTelemetry Instrumentation API doc improvements (#1499)
* OpenTelemetry Instrumentation API doc improvements

* fix links

* metric warning

* propagator fix

* comparison link

* fix markdown

* fix link

* edits from review
2020-11-10 11:50:50 -08:00
Eddy Nakamura 356c4095fa
SpanAttributeConstants is internal (#1457)
* SpanAttributeConstants is internal

* updating changelog

fixing tests

Revert "fixing tests"

This reverts commit 66167c899025f2e56aca957bbcbe6d658a866d17.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-04 13:13:31 -08:00
Eddy Nakamura 7c9bee3f6d
Removing Propagator from Options (#1448)
* Removing Propagator from Options

updating changelog

fixing tests

fixing tests

* updating changelog

* moving to dispose

* Revert "moving to dispose"

This reverts commit 01d41406b3.

* fixing build

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-04 01:57:55 -08:00
Mikel Blanchard 7be994a4be
Added a targets option to the console exporter. (#1449) 2020-11-03 16:10:31 -08:00
Utkarsh Umesan Pillai d82ab98320
Extend ConsoleExporter to add support for Logs (#1438)
* Added support for Logs; Removed disaplayAsJson option from ConsoleExporter; Added net451 as a traget framework for OpenTelemetry.Exporter.Console

* Print LogRecord exception on console

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-02 23:02:13 -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
Mikel Blanchard ddac284e53
Updated Jaeger & Zipkin exporters to check for default activity.ParentSpanId (#1433)
* Updated Jaeger & Zipkin exporters to check for default activity.ParentSpanId.

* Tabs are less filling, but spaces taste great.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-02 10:07:41 -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