Skip to content

Add OpenTelemetry Collector extension/component #603

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.

Sign up for GitHub

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

Merged
aaronpowell merged 22 commits into from
Sep 5, 2025
Merged

Add OpenTelemetry Collector extension/component #603

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
ef60fdf
Added OpenTelemetry Collector component
martinjt Aug 7, 2025
5401d2d
Fixed DevCertHostingExtensions
martinjt Aug 7, 2025
68d4102
Update src/Shared/DevCertHostingExtensions.cs
martinjt Aug 7, 2025
81ffb8d
Apply suggestions from code review
martinjt Aug 7, 2025
790c37a
Renames
martinjt Aug 11, 2025
84a0ecf
Switch the image syntax
martinjt Aug 11, 2025
bd8de8a
Move to OnBeforeStart for DevCerts
martinjt Aug 11, 2025
0240f5c
Updated Devcerts code
martinjt Aug 22, 2025
8b54668
Updates to devcerts to check for duplicate mounts.
martinjt Aug 29, 2025
19be5ba
Added an example of the collector usage
martinjt Aug 29, 2025
5f9f442
Moved to eventing and made all other resources wait for this collector
martinjt Aug 29, 2025
f908dda
made the extension methods internal
martinjt Aug 31, 2025
26c32d4
Added a basic test
martinjt Aug 31, 2025
11d3af2
Reworked Extensions and added README with examples
martinjt Sep 1, 2025
267b291
Added tests to workflow
martinjt Sep 1, 2025
2b48b7a
Added more tests
martinjt Sep 1, 2025
abd19d4
object has a null guard earlier so we don't have to force it
aaronpowell Sep 4, 2025
446c6a8
Fixing a CA1853 - Dictionary.Remove does its own guard check
aaronpowell Sep 4, 2025
53e2270
Tightening up resource access
aaronpowell Sep 4, 2025
760a610
Expanding the unit tests
aaronpowell Sep 4, 2025
0c69e58
More tests
aaronpowell Sep 4, 2025
52f1159
Adding martinjt as codeowner
aaronpowell Sep 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed DevCertHostingExtensions
  • Loading branch information
@martinjt
martinjt committed Aug 29, 2025
commit 5401d2d93212b4cbe52cef8850b3dae4e5abda8c
5 changes: 5 additions & 0 deletions ...ting.OpenTelemetryCollector/CommunityToolkit.Aspire.Hosting.OpenTelemetryCollector.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,9 @@
<ItemGroup>
<PackageReference Include="Aspire.Hosting" />
</ItemGroup>

<ItemGroup>
<Compile Include="$(SharedDir)\DevCertHostingExtensions.cs" Link="Utils\DevCertHostingExtensions.cs" />
</ItemGroup>

</Project>
11 changes: 9 additions & 2 deletions src/Shared/DevCertHostingExtensions.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,13 @@
namespace Aspire.Hosting;
using Aspire.Hosting;
using Aspire.Hosting.ApplicationModel;
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.Hosting;
using Microsoft.Extensions.Logging;
using System.Diagnostics;

public static class DevCertHostingExtensions
namespace Aspire;

internal static class DevCertHostingExtensions
{
/// <summary>
/// Injects the ASP.NET Core HTTPS developer certificate into the resource via the specified environment variables when
Expand Down