Commit Graph

914 Commits

Author SHA1 Message Date
Cijo Thomas 4385164d85
Added instructions to run console examples (#1525) 2020-11-12 12:19:07 -08:00
xiang17 93f614ba58
[SelfDiagnostics] Update config refresh period to 10 seconds (#1523)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-12 12:07:54 -08:00
Eddy Nakamura 2823db731f
Moving <Compile> to Includes folder (#1519)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-12 11:11:38 -08:00
Eddy Nakamura eee0036b2e
Fixing ActivitySource from DiagnosticSource (#1515)
* Fixing ActivitySource from DiagnosticSource

* updating changelog

* fixing changelog, updating instrumentation aspnetcore benchmark

* undoing

* updating assembly naming

* updating changelog

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-12 10:46:57 -08:00
Eddy Nakamura d1c174df17
Adding service.namespace as tags for zipkin (#1521)
updating changelog

updating changelog

updating changelog

Revert "Adding service.namespace as tags for zipkin"

This reverts commit 1868e71b964fa030bc33ece441047b19fa221688.

updating file and changelog
2020-11-12 10:27:34 -08:00
Prashant Srivastava 4ab6da0812
Adding support for default resource creation with environment variable (#1436)
* Adding default and envVar resource. Initial commit

* Adding Create method for resource creation with default resource and relevant unit tests.

* Adding isNullOrEmpty check to env var value to remove redundant condition.

* TracerProviderBuilder now calls Resource.Create to get the default resource with EnvVar attributes as well

* The default reource now includes the attributes from env var as well. Refactoring unit tests for the same.

* Adding ResourceDetector interface and public methods to add attributes from default and env var resources

* Missed out new resource files. Adding

* adding some more tests

* Adding missing import

* Updating public api list

* Refactoring method name for new methods

* Making IResourceDetector and corresponding impl internal until spec is official about detectors

* Refactoring return

* Was using wrong Sdk in test. Fixed.

* Adding error handling to OtelEnvResourceDetector

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-12 10:16:05 -08:00
Cijo Thomas cb7f4217ef
Remove ununsed constants from Resource (#1516)
* Remove ununsed constants from Resource

* fix public API
2020-11-11 19:04:42 -08:00
Eddy Nakamura be72b17570
Updating Jaeger tags based on spec (#1513)
* Updating Jaeger tags based on spec

* updating changelog

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-11 16:54:48 -08:00
Cijo Thomas 6157364796
Zipkin tags used for InstrumentationLibrary fixes (#1512)
* Fix zipkin exporter to match spec about instrumentationlibrary

* changelog

* fix unwanted space
2020-11-11 16:38:45 -08:00
Austin Tan 67f77ab227
Update to ZipkinExporterOptions (#1510)
* Update to ZipkinExporterOptions

* Mistake on pre-compiler message for 4.5.2, versions OTHER than .NET 4.5.2, not below

* Length of line for markdownlint
2020-11-11 11:21:32 -08:00
Cijo Thomas 1a308a6bd2
Net5.0 cleanup (#1506)
* Net5.0 cleanup

* more
2020-11-11 06:52:03 -08:00
Cijo Thomas 7c55d087c3
Do not throw when TracerProvider AddProcessor is called on API (#1507)
* Do not throw when TracerProvider AddProcessor is called on API

* better code
2020-11-11 06:25:58 -08:00
Cijo Thomas 18cb0a3e98
Modify GHActions to use new .net5 (#1503) 2020-11-10 22:08:57 -08:00
Cijo Thomas baa2972e49
Instrumentation Library doc (#1467)
* Draft of instrumentation doc

* more details

* markdowns

* minor

* link

* fix link

* doc fix

* minor

* link
2020-11-10 17:59:18 -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
Cijo Thomas a29e031bf5
Use GA version of DiagnosticSource (#1500)
* Use GA version of DiagnosticSource

* fix space

* Update bcl backages to be utpo 6.0 version
2020-11-10 11:17:56 -08:00
Eddy Nakamura 7063ea055a
Updating grpc status code (#1488)
* Updating grpc status code

* validating rpc.grpc.status_code tag

* updating assert

* updating method

* updating tests

* renaming to tryGet, returning false if not valid

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-09 21:14:17 -08:00
Alan West 30f6b70f3e
Wait for processor invocations (#1497)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-09 17:32:37 -08:00
Cijo Thomas 8775969819
Remove RentrantExportProcessor as it is not required by spec (#1496)
* Remove RentrantExportProcessor as it is not required by spec

* clean up api doc

* fix doc

* remove deadlinks

* remove test

* more removals
2020-11-09 16:31:23 -08:00
xiang17 32a500c586
[SelfDiagnosticsModule] Add user guide for troubleshooting (#1477)
* Add troubleshooting doc to enable self diagnostics module

* Add explanation on EventLevel and parsing failure behavior.

* Change the configuration file name to OTEL_DIAGNOSTICS.json

* Add more explanation

* Rename EventLevel to LogLevel

* Add link to EventLevel enum

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-09 14:57:25 -08:00
Mikel Blanchard b956a5f7e9
Resource allocation improvement (#1463)
* Introduces the concept of IProviderContainer which allows injection of a TracerProvider into lower levels (Processor, Exporter, etc.).

* Removed IProviderContainer and fixed the provider types.

* Removed log provider example.

* Working on getting everything working with the new API.

* Attempting to get all the tests passing.

* Added concept of BaseProvider. Updated public API.

* Project fixup.

* Removed OnParentProviderSet.

* Updated changelog.

* Test fixup.

* Test fixup.

* Cleanup.
2020-11-09 13:39:25 -08:00
Eddy Nakamura 052ecc44b2
Enable .net5.0 (#1486)
* Enable .net5.0

* fixing build issues

* updating sanity

* fixing windows pipelines

* adding aspnetcore5.0

* adding net5 workaround

* skipping Flackytest

* updating to latest stable version

* updating code coverage to latest preview
2020-11-09 12:34:03 -08:00
Eddy Nakamura bebc80f307
Enable publicApi for OT (#806)
updating projects, adding options parameter

Applying suggestions

Apply suggestions from code review

Co-authored-by: Paulo Janotti <pjanotti@splunk.com>

code review - cijo

removing aspnetcore "metapackage"

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>

updating public apis

updating public api, adding tool to migrate

Apply suggestions from code review

Co-authored-by: Sergey Kanzhelev <S.Kanzhelev@live.com>

Adding tool to migrate unshipped to shipped

updating contributing guidelines

updating lint

adding publicapi to shims

updating publicapi files

removing publicapi that works only for debug

updating publicapi

updating publicapi

removing extra file (merge problem)

updating publicapi

updating to latest version

updating spacing

updating contribution guideline

markdownlint

markdownlint

updating public api

updating publicapi

updating

updating

updating apis

Enable publicApi for OT

updating projects, adding options parameter

Applying suggestions

Apply suggestions from code review

Co-authored-by: Paulo Janotti <pjanotti@splunk.com>

code review - cijo

removing aspnetcore "metapackage"

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>

updating public apis

updating public api, adding tool to migrate

Apply suggestions from code review

Co-authored-by: Sergey Kanzhelev <S.Kanzhelev@live.com>

Adding tool to migrate unshipped to shipped

updating contributing guidelines

updating lint

adding publicapi to shims

updating publicapi files

removing publicapi that works only for debug

updating publicapi

updating publicapi

removing extra file (merge problem)

updating publicapi

updating to latest version

updating spacing

updating contribution guideline

markdownlint

markdownlint

updating public api

updating publicapi

updating

solving merge issue

updating - 01

updating - 02

updating - 03

updating - 04

updating - 05

updating - 06

updating - 07

updating - 08

updating - 09

updating - 09

updating - 10

updating - 11

updating - 12

updating - 12

updating encoding

updating

updating - 13

updating - 14

updating - 15

updating - 16

fixing spacing

updating - 17

fixing spacing

updating - 18

adding suppression

simplifying changes

fixing merge issues

updates after merge

updating public api files

fixing build after merge

fixing build issues

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-06 09:28:01 -08:00
Cijo Thomas 8d91622ec3
Add Alan West as approver (#1478)
Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
2020-11-06 09:04:44 -08:00
Cijo Thomas a155082abb
Minor wording change for Grpc readme (#1479)
* Minor wording change for Grpc readme

* tpos
2020-11-05 22:12:20 -08:00
Alan West 6939dd5b4f
Grpc.Net.Client context propagation (#1464)
* Move HttpRequestMessage context getter/setter methods to static class

* Share HttpRequestMessageContextPropagation with Grpc.Net.Client instrumentation

* Propagate context from Grpc.Net.Client instrumentation when suppressing downstream instrumentation

* Use Baggage.Current

* Document instrumentaiton options

* Add code example

* AddHttpClientInstrumentation

* Remove ASP.NET section

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-05 20:37:43 -08:00
xiang17 ad35765145
[SelfDiagnosticsModule] Reads configuration file and creates log file (#1468)
* Add ConfigRefresher

* Refactor parser related code into ConfigParser

* Add comments

* Remove ThreadLocal in ConfigParser and fix initialization order in ConfigRefresher constructor

* Use compiled regualr expressions

* Add comment for a race condition during configuration file refresh when the file changed

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-05 14:44:07 -08:00
Vishwesh Bankwar 0cf26d3eb6
vibankwa/aspnet enrich documentation (#1465)
* Doc update - adding Enrich e.g. in AspNet Readme

* rmv special topic - enrich

* rmv old way of enrich in aspnet

* adding processor option to enrich in aspnet readme

Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-05 14:32:06 -08:00
Alan West 6278780511
Instrumentation docs Part #2 - HTTP options (#1244)
* Document HTTP instrumentation options

* Clarify HttpClient vs. HttpWebRequest instrumentation

* Remove section about adding HttpClient vs. HttpWebRequest instrumentation

* Update documentation on Enrich option

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-05 14:14:55 -08:00
Cijo Thomas f8a9b7002b
Small changes to release process (#1476)
* Releasing instruction minor update

* minor
2020-11-05 12:58:21 -08:00
Cijo Thomas 2bc7924b06
Minor test refactor (#1474) 2020-11-05 12:13:51 -08:00
Cijo Thomas 4ef3091413
Changelog update for beta8 (#1475) 2020-11-05 11:25:35 -08:00
Cijo Thomas 11f08c324a
Rename Instrumentation to DiagnosticSourceInstrumentation to avoid confusion (#1471)
* Rename Instrumentation to DiagnosticSourceInstrumentation to avoid confusion

* add back
2020-11-05 08:29:43 -08:00
Cijo Thomas 8d87ebc14c
Mark DiagnosticSource subscription helpers as internal (#1466)
* Mark DiagnosticSource subscription helpers as internal

* remove space
2020-11-04 21:52:21 -08:00
Cijo Thomas 55cac800f9
AddInstrumentation and AddDiagnosticSourceInstrumentation APIs. (#1454)
* Separate AddInstrumentation and AddDSInstrumentation

* Mark AddDiagnosticSourceInstrumentation as internal

* changelog

* order

Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
2020-11-04 17:05:27 -08:00
humphrieslk 1e92aeda04
Add start timestamp to OpenTelemetry metric aggregators (#1375)
* Add start timestamp to OpenTelemetry metric aggregators

* Update doc and move protected members to the bottom

* Fix test header documentation

* Run dotnet-format on AggregatorTest

* Update documentation, use DateTimeOffset

* Remove cruft

* Use ticks instead of ms

Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
Co-authored-by: Sergey Kanzhelev <S.Kanzhelev@live.com>
2020-11-04 13:25:05 -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
xiang17 0890a3bf0c
[SelfDiagnosticsModule] Write event to log file in a circular way in … (#1418)
* [SelfDiagnosticsModule] Write event to log file in a circular way in EventListener.OnEventWritten

* fix typo

* accomodate feedbacks

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
2020-11-04 12:30:07 -08:00
Cijo Thomas 38e058f92c
Removed AddOpenTelemetryTracing method which takes Func returning TracerProvider (#1456)
Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
2020-11-04 12:17:36 -08:00
Cijo Thomas b1dcc14ee1
Fix HttpWebRequest test issue (#1460) 2020-11-04 12:05:36 -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
Austin Tan 7a039aa948
All files containing 'Throw new ArgumentOutOfRangeException' updated to use three-param ctor (#1455)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-04 01:30:30 -08:00
Utkarsh Umesan Pillai fcde600f22
Add extension method to add ConsoleExporter for logs (#1452)
* Added an extension method to add ConsoleExporter for logs; Updated the logs getting-started docs to add ConsoleExporter

* Updated CHANGELOG.md

* Addressing PR comments

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-03 20:10:48 -08:00
Eddy Nakamura c55145b186
Fixing activityContext propagation (#1446)
* Fixing activityContext propagation

* Adding tests

* updating tests

* adding comments

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-11-03 17:21:06 -08:00
Mikel Blanchard 7be994a4be
Added a targets option to the console exporter. (#1449) 2020-11-03 16:10:31 -08:00
Eddy Nakamura 46094add40
Fixing OutOfMemory CodeQL (#1443) 2020-11-03 06:58:57 -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