Commit Graph

510 Commits

Author SHA1 Message Date
Alan West ea107d3b69
Rename contants in SemanticConventions class to conform to recommended C# naming conventions (#892) 2020-07-22 23:55:17 -07:00
Eddy Nakamura 3aca129967
Moving benchmark to test folder (#885)
* Moving benchmark to test folder

updating namespaces

updating path

Reiley comments

* updating paranthesis
2020-07-22 17:09:28 -07:00
Cijo Thomas 4b4f0e95af
More refactorings, renamings, namespace shortenings. (#886)
* Rename OpenTelemetrySdk to TracerProviderSdk. Rename OpenTelemetryBuilder to TracerProviderBuilder

* renaming

* Shorter namespace for TracerProvider

* simpler namespace.

* revert unwanted change brought by git merge

* doc changes

* remove dup
2020-07-22 16:46:45 -07:00
Reiley Yang e243394211
Clean up instrumentation library doc (#883)
* clean up instrumentation doc

* clean up

* fix nits

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-22 15:03:31 -07:00
Mikel Blanchard cafc62ee7c
Batching Activity Processor fixes by Eddy & Mike (#853)
* Attempting to fix the threading in BatchingActivityProcessor.

* catching all exceptions in the same way

* adding cancellationtokensource

* Added back in export timeout.

* Don't allocated linked CTS if it isn't needed.

* Unit test fixes.

* Remove the possibility of many timer threads waiting to enter the flush lock.

* Reverted timer change to BatchingActivityProcessor.

* Respect cancellation in jaeger & zipkin exporters.

* Moving lock up to the batch in Jaeger Exporter.

* Added an event for SpanExporter timeout. Updated BroadcastActivityProcessor to call shutdown on dispose.

* Unit test fixes.

* Let OperationCanceledException bubble up in JaegerUdpBatcher.

* Flaky test.

* Flaky test, attempt 2.

* Trying to get more reliable tests.

* Added exception docs for OperationCanceledException.

Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-22 12:37:36 -07:00
Mikel Blanchard 32e886ff1f
Switched AttemptToActivateActiveSpan event to Informational level. (#881)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-22 11:42:59 -07:00
Reiley Yang bffc739265
Revert the CI task merge + add filters (#878)
* revert the ci task merge

* add filters

* fix typo
2020-07-22 09:38:59 -07:00
Eddy Nakamura c4ff8ab640
renaming from Samples to Examples (#880)
* renaming from Samples to Examples

* updating changelog

* renaming from OT.Examples.name to Examples.name
2020-07-22 09:18:56 -07:00
Eddy Nakamura a2b1b45c94
Enable unique EventSource for each project (#879)
* Enable unique EventSource for each project

* updating using place from Hosting project

* InstrumentationEventSource is now internal, adding EventSource to Hosting project

* removing unused methods, updating messages
2020-07-22 07:34:28 -07:00
Cijo Thomas 1ed4a28927
Basic eventsource logging for ZipkinExporter and minor renamings (#877)
* Rename to OpenTelemetryProtocolExporterEventSource

* Add basic eventsource logging int ZipkinExporter

* clean readme
2020-07-22 06:53:39 -07:00
Cijo Thomas 13dd5d717e
More cleanups. (#875)
* Rename samplers

* Cleanups

* typos

* remove internal as it has nothing now

* rename more
2020-07-22 06:41:24 -07:00
Reiley Yang b91b45ae44
Clean up outdated docs (#874)
* clean up outdated stuff

* ul
2020-07-22 00:17:59 -07:00
Cijo Thomas d796db07b4
Replace Span with Shim Span. (#850)
* Remove tracerfactory and tracerbuilder

* Remove tracer extensions

* Add Span and Tracer shims. Remove all span code.

* more wrapper apis

* Wrap ActivityEvent

* wrap links

* change tracestate back to keyvalue pair in contxt

* ore tracer fix

* OpenTracing shim

* example change

* remove spancontext

* remove more of the new stuffs

* benchmark fixes

* remove Event class

* TraceState consistent name

* casing fix

* equality fix

* fix test

* test todo

* typo

* rename after accepting shim chanegs from Raj

* remove new from the end of many names.

* benchmark fix

* TraceStateUtils naming

* TraceStateUtilsNew rename to fix build issue at CI.

* renamings
2020-07-21 23:11:13 -07:00
Reiley Yang f432f4b96c
clean up contributing.md (#871) 2020-07-21 23:00:05 -07:00
Reiley Yang c9c39555d3
Add get started document (#870)
* add get started document

* fix nit

* remove unused stuff

* fix broken link
2020-07-21 20:12:53 -07:00
Peter Wiese c1dedf0589
Don't throw when checking if an outbound HttpRequestMessage request is injected with the traceparent header (#869)
* If an outbound HttpRequestMessage does not include the traceparent header then don't throw.

Made the header values getters and setter static to avoid the Func and Action allocations per invocation.

* Missed a getter usage in HttpInListener

* Extending an existing unit test to validate the new HttpRequestMessage header value getter.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-21 17:13:33 -07:00
Rajkumar Rangaraj 609c57c31f
Update OpenTracingShim to use OpenTelemetry shim (#859)
* Update otlp test

* Converted Span to Activity

* Modified tests

* Changed dispose in tests and added exporters sample

* Renamed all activity to span

* Modified activity to Activity

* Modified existing tests, no need to use Activity based shim

* Modified sample.

* Incorporating Cijo's feedback
2020-07-21 17:03:26 -07:00
Mikel Blanchard 4c77bc94b3
ParentOrElse Sampler (#852)
* Added ParentOrElseActivitySampler. Updated AlwaysOn, AlwaysOff, & Probability ActivitySamplers for spec changes.

* Updated documentation comments.

* Added missing braces in ParentOrElse description.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
Co-authored-by: Eddy Nakamura <ednakamu@microsoft.com>
2020-07-21 14:46:41 -07:00
Cijo Thomas df2d44be07
Small tweaks to Docs. (#868)
* Minor improvements to API readme.md

* SDK docs minor with sampling added

* line lentg

* intallation step at top

* lint
2020-07-21 14:11:15 -07:00
Timothy Mothra a3c0b80840
fix Jaeger EventSource id (#865)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-21 12:59:36 -07:00
Eddy Nakamura 29df2bb840
Disable master merge workflow for linux/win (#867)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-21 12:49:54 -07:00
Reiley Yang cc141cda0f
Avoid the "Win" trick (#862)
* remove Win hack

* remove Win from folder name
2020-07-21 12:13:55 -07:00
Cijo Thomas f62b24ed48
Upgrade to DiagnosticSource version from nuget. This ensure we can release beta to nuget and users dont have to add any additional nuget source. (#866) 2020-07-21 11:27:29 -07:00
Eddy Nakamura 68d3c791e7
Updating samples folder path (#857)
* Updating samples folder path

* Updating from Web to AspNetCore

* updating paths and namespaces

* merging docfx, markdownlint and spellcheck to docs.yml
merging integration-redis and integration-sql to integration.yml

* updating files to work on vscode

* creating extensions.json to simplify usage

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-21 10:18:51 -07:00
Cijo Thomas a2c4f9c630
Temporarily disable code coverage (#861) 2020-07-21 00:31:45 -07:00
Reiley Yang 6e66179a9b
Minor improvements over consistency (#860)
* make package description consistent

* remove nits

* better description

* avoid the .NET vs .NET Core confusion
2020-07-21 00:17:45 -07:00
Reiley Yang 234399c093
Move SDK doc, minor clean up (#858)
* move SDK doc, minor clean up

* update link

* fix typo

* fix typo

* remove exporter from SDK doc

* remove broken link

* update badge

* fix typo

* fix code snippet indentation

* fix code snippet indentation

* fix nits

* fix nits

* fix nits
2020-07-20 21:19:54 -07:00
Reiley Yang a4284aca24
Minor doc cleanup (#856)
* minor doc cleanup

* revert to CRLF to minimize diff

* improve wording

* make docfx happy
2020-07-20 17:20:36 -07:00
Eddy Nakamura 47689cea36
Updating exporter docs (#855)
* Updating exporter docs

* updating based on MDlint

* Adding prerequisite section
2020-07-20 17:03:03 -07:00
Reiley Yang 2892bdd072
Clean up doc structure (#851)
* more docs

* lint

* simplify command
2020-07-20 15:09:12 -07:00
Reiley Yang 5ad99f5110
clean up Jaeger doc structure (#849) 2020-07-20 12:49:10 -07:00
Cijo Thomas 00ee600fc7
Basic docs (#847)
Basic API usage doc, SDK usage doc.
2020-07-20 11:49:47 -07:00
Mikel Blanchard b5a9c3a5d6
Cleaning up options in dependency instrumentation. (#846)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-20 10:20:39 -07:00
Cijo Thomas af172a8943
Basic OTel shim around Activity and associated classes. (#836)
* Basic OTel shim around Activity and associated classes.

* review comments

* remove Event public class. Make fluent syntax easier

* cooment

* cleanup sample
2020-07-20 10:01:38 -07:00
Reiley Yang 157495803c
Add markdownlint to CI (#848)
* markdownlint

* rename

* rename

* fix lint issue discovered by markdownlint
2020-07-20 07:51:37 -07:00
Mikel Blanchard c294cd7f76
Added options to HttpWebRequest instrumentation. (#741)
* Added options to HttpWebRequest instrumentation.

* Cleaned up some usings.

* Added IsInjected to ITextFormatActivity.

* Switched the Http instrumentation filter back to internal so we can move ahead with the other stuff while the API is reviewed.

* Updated HttpClient instrumentation to check IsInjected instead of traceparent header directly.

* FilterFunc is back to public. Removed ctors from http options.

* Removed ctor that isn't needed.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-19 18:51:14 -07:00
Eddy Nakamura dfd965a374
Feature/updating customization doc (#845)
* Updating customization doc

* updating text
2020-07-19 08:14:53 -07:00
Reiley Yang f689163096
Add misspell to CI (#841) 2020-07-18 09:48:08 -07:00
Eddy Nakamura c3324f3e1b
Improving solution (#840)
* t # This is a combination of 2 commits.

Improving solution

removing white space

updating method to create activity

* Update benchmarks/Benchmarks.csproj

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

Co-authored-by: Mikel Blanchard <mblanchard@macrosssoftware.com>
Co-authored-by: Reiley Yang <reyang@microsoft.com>
2020-07-18 09:22:20 -07:00
Eddy Nakamura 5c20cf5b34
Fixing culture in PrometheusMetricBuilder (#839)
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-18 08:57:55 -07:00
Mikel Blanchard e13c33f596
Status cleanup (#838)
* Cleaned up status usage.

* Unit test fixup.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-18 07:56:38 -07:00
Reiley Yang eb42d6b6d1
Add markdown sanity check CI (#837)
* markdown sanity check

* minor adjustment to the folder structure

* clean up Jaeger README.md
2020-07-18 07:31:01 -07:00
Eddy Nakamura 51ed7e30db
Adding Kind Setter Extension for Activity (#835)
* Adding Kind Setter Extension for Activity

updating kind

* CodeBlanch suggestions

* adding readonly property

* changing casing

* cijos suggestions

* cijo suggestion

* removing duplicated test
2020-07-17 16:28:02 -07:00
Eddy Nakamura 595f8c2379
Cleaning Jaeger Exporter (#832)
Updating tests

removing JaegerConversionExtensions

renaming methods

Fixed up Jaeger unit tests and status code helper.

removing unused references
2020-07-17 08:07:29 -07:00
Cijo Thomas b91b2bc9f5
Replace Span with Activity for JaegerBenchmark (#834)
* Replace Span with Activity for JaegerBenchmark

* min
2020-07-17 07:43:42 -07:00
Mikel Blanchard b57f3cf320
.NET Core SqlClient Integration Tests + Fixes (#826)
* .NET Core SqlClient instrumentation integration tests, bug fixes, and cleanup.

* Dependency instrumentation docker artifacts and integration GitHub workflow.

* Code review feedback.

* Added docker-compose-wait to dockerfile so tests don't start until sql is listening.

* Code review.

* Namespaces for OTel ActivitySources.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-16 10:52:29 -07:00
Cijo Thomas 6dbc69f76b
Add Mikel as maintainer (#805)
Co-authored-by: Mike Goldsmith <goldsmith.mike@gmail.com>
2020-07-16 14:25:10 +01:00
Eddy Nakamura 5b34e3b3f8
Adding metapackages (#810)
Adding IsPackable to test.props, updating test nuget versions

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>
2020-07-15 23:31:57 -07:00
Rajkumar Rangaraj 9467014b72
Changed dipsose pattern in test (#833) 2020-07-15 22:17:07 -07:00
Peter Wiese b319c0bed6
Replace references to SpanAttributeConstants with constants from SemanticConventions (#828)
* Replace references to SpanAttributeConstants with constants from SemanticConventions

* Missed a few references to the "component" attribute in some tests.

Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
2020-07-15 17:27:21 -07:00