Rename Application to Resource in dashboard OTLP functionality #10868
+1,180
−1,179
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…updated Co-authored-by: JamesNK <[email protected]>
…ge and model classes Co-authored-by: JamesNK <[email protected]>
…el and core updates Co-authored-by: JamesNK <[email protected]>
…ional model and component files Co-authored-by: JamesNK <[email protected]>
Copilot
AI
changed the title
Copilot finished work on behalf of
JamesNK
August 7, 2025 07:22
There was a problem hiding this comment.
Pull Request Overview
This PR comprehensively renames "Application" to "Resource" throughout the dashboard OTLP functionality to align with proper OpenTelemetry terminology. In OpenTelemetry, the correct term for entities that generate telemetry data is "resource" rather than "application".
Key Changes:
- File renames for core OTLP model classes and helper utilities
- Class and type name updates from Application/OtlpApplication to Resource/OtlpResource
- Property and method renames throughout the storage layer, view models, and UI components
Reviewed Changes
Copilot reviewed 25 out of 25 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| TelemetryRepository.cs | Updated storage layer methods and collections from Application to Resource terminology |
| Subscription.cs | Changed ApplicationKey property to ResourceKey |
| ResourceKey.cs | Renamed struct from ApplicationKey to ResourceKey with updated methods |
| GetTracesRequest.cs | Updated ApplicationKey property to ResourceKey |
| GetLogsContext.cs | Updated ApplicationKey property to ResourceKey |
| GetInstrumentRequest.cs | Updated ApplicationKey property to ResourceKey |
| OtlpTrace.cs | Updated method parameter from OtlpApplication to OtlpResource |
| OtlpSpan.cs | Updated Source property and uninstrumented peer from Application to Resource types |
| OtlpResourceView.cs | Renamed class from OtlpApplicationView with updated properties |
| OtlpResourceType.cs | Renamed enum from OtlpApplicationType |
| OtlpResource.cs | Renamed class from OtlpApplication with updated properties and methods |
| OtlpLogEntry.cs | Updated ApplicationView property to ResourceView |
| OtlpHelpers.cs | Renamed GetApplicationKey method to GetResourceKey |
| TracesViewModel.cs | Updated ApplicationKey property to ResourceKey |
| TraceHelpers.cs | Updated OrderedApplication class to OrderedResource with related method updates |
| StructuredLogsViewModel.cs | Updated ApplicationKey property to ResourceKey |
| SpanDetailsViewModel.cs | Updated Applications property to Resources |
| ResourceTypeDetails.cs | Updated GetApplicationKey method to GetResourceKey |
| SpanWaterfallViewModel.cs | Updated method parameters from Application to Resource types |
| ResourcesSelectHelpers.cs | Renamed class from ApplicationsSelectHelpers with updated method names |
| ExemplarsDialogViewModel.cs | Updated Applications property to Resources |
| UnreadLogErrorsBadge.razor.cs | Updated Dictionary key type and variable names |
| StateColumnDisplay.razor.cs | Updated Dictionary key type |
| SignalsActionsDisplay.razor.cs | Updated HandleClearSignal delegate parameter type |
| ClearSignalsButton.razor.cs | Updated HandleClearSignal delegate parameter type and method call |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR renames "Application" to "Resource" throughout the dashboard OTLP functionality to align with proper OpenTelemetry terminology. In OpenTelemetry, the correct term for the entity that generates telemetry data is "resource" rather than "application".
Changes Made
File Renames
OtlpApplication.cs→OtlpResource.csOtlpApplicationType.cs→OtlpResourceType.csOtlpApplicationView.cs→OtlpResourceView.csApplicationKey.cs→ResourceKey.csApplicationsSelectHelpers.cs→ResourcesSelectHelpers.csClass and Type Updates
OtlpApplication→OtlpResourceOtlpApplicationType→OtlpResourceTypeOtlpApplicationView→OtlpResourceViewApplicationKey→ResourceKeyApplicationsSelectHelpers→ResourcesSelectHelpersProperty and Method Updates
ApplicationName→ResourceNameApplicationKey→ResourceKeyGetApplications()→GetResources()GetApplicationByCompositeName()→GetResourceByCompositeName()GetApplicationKey()→GetResourceKey()Affected Areas
The renaming maintains full backward compatibility in terms of functionality while using the correct OpenTelemetry terminology throughout the codebase.
Addressing #10867.
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.