-
Notifications
You must be signed in to change notification settings - Fork 391
Add telemetry for dependency caching #3136
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
base: main
Are you sure you want to change the base?
Conversation
63beaa0
to
249a3cb
Compare
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.
Pull Request Overview
This PR adds telemetry support for dependency caching operations by capturing and reporting metrics about cache restore and upload activities. The changes enable tracking of cache hit rates, download/upload sizes, and operation durations for different programming languages.
- Introduces new types and enums to represent cache operation results and status
- Modifies existing cache functions to return detailed telemetry data instead of simple success indicators
- Updates status reporting to include dependency cache metrics in telemetry payloads
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
src/dependency-caching.ts | Adds telemetry types and modifies cache functions to collect performance metrics |
src/status-report.ts | Updates status report interface to include dependency cache restore results |
src/init-action.ts | Captures dependency cache restore telemetry and passes it to status reporting |
src/analyze-action.ts | Captures dependency cache upload telemetry and includes it in final status reports |
src/status-report.test.ts | Updates test to handle new parameter in createInitWithConfigStatusReport |
lib/init-action.js | Generated JavaScript reflecting TypeScript changes for init action |
lib/analyze-action.js | Generated JavaScript reflecting TypeScript changes for analyze action |
Adds information about the results of restoring and storing dependency caches.
Note: Do not merge until corresponding, internal PRs are done.
Risk assessment
For internal use only. Please select the risk level of this change:
Merge / deployment checklist