Skip to content

add telemetry to ResourceStateManager and CcapiService#150

Merged
satyakigh merged 3 commits intomainfrom
resource-telemetry
Oct 29, 2025
Merged

add telemetry to ResourceStateManager and CcapiService#150
satyakigh merged 3 commits intomainfrom
resource-telemetry

Conversation

@Zee2413
Copy link
Contributor

@Zee2413 Zee2413 commented Oct 28, 2025

Metrics Added (27 total)

CcapiService (CloudControl API - 6 metrics)

• CcapiService.getResource - count, duration, fault
• CcapiService.listResources - count, duration, fault

ResourceStateManager Methods (12 metrics)

• getResource - count, duration, fault
• listResources - count, duration, fault
• searchResourceByIdentifier - count, duration, fault
• refreshResourceList - count, duration, fault

State Performance (3 metrics)

• state.hit - Resource found in cache
• state.miss - Resource fetched from API
• state.fault - ResourceNotFoundException

Refresh Operations (1 metric)

• refresh.fault - Refresh failures

State Lifecycle (2 metrics)

• state.invalidated - Profile/region change cleared resourceStateMap
• list.invalidated - Profile/region change cleared resourceListMap

State Gauges (3 metrics)

• state.types - Number of resource types in resourceStateMap
• state.count - Total individual resources in resourceStateMap
• list.types - Number of resource types in resourceListMap

@Zee2413 Zee2413 requested a review from a team as a code owner October 28, 2025 22:09
@Zee2413 Zee2413 force-pushed the resource-telemetry branch from 95580cd to f2ac450 Compare October 29, 2025 03:36
@satyakigh satyakigh merged commit 6de5dcb into main Oct 29, 2025
8 checks passed
@satyakigh satyakigh deleted the resource-telemetry branch October 29, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants