Add actions to add telemetry to GitHub pipelines#1011
Merged
Conversation
🦋 Changeset detectedLatest commit: b37ed6d The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Krusty93
force-pushed
the
CES-1118-tracciare-package-manager-su-app-insights-via-workflow
branch
from
6d480b4 to
adef77a
Compare
Krusty93
force-pushed
the
CES-1118-tracciare-package-manager-su-app-insights-via-workflow
branch
3 times, most recently
from
27a090d to
7e974fa
Compare
Krusty93
force-pushed
the
CES-1118-tracciare-package-manager-su-app-insights-via-workflow
branch
from
7e974fa to
75d649f
Compare
kin0992
requested changes
Feb 5, 2026
Contributor
|
I left a few minor comments and an issue that you should address first. |
- Updated the setup-telemetry action to use async file operations with error handling. - Introduced Zod validation for required environment variables in both setup and post-execution scripts. - Refactored the telemetry line parsing logic to improve readability and maintainability. - Created dedicated functions for processing event lines and span markers to streamline the telemetry data handling. - Improved logging and error handling throughout the telemetry process. - Ensured proper flushing of telemetry data before process exit.
gunzip
approved these changes
Feb 17, 2026
Contributor
gunzip
left a comment
gunzip
left a comment
There was a problem hiding this comment.
lgtm, remember to switch to main references to CES-1118-tracciare-package-manager-su-app-insights-via-workflow before merging
kin0992
approved these changes
Feb 18, 2026
Krusty93
deleted the
CES-1118-tracciare-package-manager-su-app-insights-via-workflow
branch
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
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.
Resolves CES-1118
This PR introduces a new telemetry stack for GitHub workflows composed of two actions:
Key features:
New GitHub actions:
setup-telemetry
log-telemetry-event
Writes one NDJSON line per invocation.
Outcome
A root span containing one or more child elements including exceptions, custom events, dependencies and nested spans.
Each telemetry item contains attributes for cicd pipelines according to otel standard.