-
Notifications
You must be signed in to change notification settings - Fork 160
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.
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
Metrics 3 versions #842
Metrics 3 versions #842
Conversation
Co-authored-by: Tim Hess <thess@vmware.com>
…nto WavefrontExporter
0be1291
to
5b87573
Compare
It looks like there is now a GA release of the core components |
@@ -253,6 +254,23 @@ public void AllActuators_AreAutowired() | |||
Assert.IsType<AllActuatorsStartupFilter>(filter); | |||
} | |||
|
|||
[Fact] | |||
public void Wavefront_IsAutowired() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be two separate tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if anyone would want to enable metrics but not traces or vice versa, at the moment thats not an option. Thats mostly why its the same test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What would happen if you have a ref for EndpointCore (and Wavefront config) but not TracingCore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works as expected, adding two separate tests.
src/Management/src/OpenTelemetryBase/Exporters/Wavefront/WavefrontTraceExporter.cs
Show resolved
Hide resolved
/// </summary> | ||
/// <param name="hostBuilder">Your hostBuilder</param> | ||
/// <returns>The configured hostBuilder</returns> | ||
public static IHostBuilder AddDistributedTracing(this IHostBuilder hostBuilder) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Action<TracerProviderBuilder> action
could be exposed here too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
/azp run Steeltoe.All |
Azure Pipelines successfully started running 1 pipeline(s). |
src/Bootstrap/test/Autoconfig.Test/WebHostBuilderExtensionsTest.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Azure Pipelines successfully started running 1 pipeline(s). |
SonarCloud Quality Gate failed. |
Update to GA of OpenTelemetry Core packages & latest RC of non core
Add support for Proxy (without needing telegraf) and compatible with boot experience
Add support for wavefront with AddSteeltoe autoconfiguration