Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(telemetry): Collect project specific tags #228

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Conversation

andreiborza
Copy link
Member

@andreiborza andreiborza commented Jan 13, 2025

Collects

  • project: string if single project is specified
  • projects: string[] if multiple projects are specified, comma-separated
  • set-commits: 'auto' | 'skip'
  • sourcemaps: boolean if sourcemaps are set by the user
  • sourcemaps-uploaded: true iff sourcemaps have been uploaded, not set otherwise
  • finalized: true iff release has been finalized
  • organization: string the user org

It also sets the user.id to the user org.

ref: #217

Some of the tags listed in the reference issue have not been collected:

  • project id, not available -> only project slug is collected
  • action release version -> already collected
  • checkout action version -> we do not have access to this
  • platform -> we do not have access to this in the action
  • runtime -> we do not have access to this in the action
  • os -> we do not have access to this in the action
  • debugId -> will be added with Add optional option to inject DebugIds #220
  • self-hosted or Saas -> we do not enable Sentry when the project is self-hosted so this tag would always be false/'self-hosted'
  • parameters: picked the ones listed above, but we can discuss adding more
  • measurements: to be looked into, might require extra api calls.

@andreiborza andreiborza requested review from Lms24 and smeubank January 13, 2025 13:08
Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one Q, rest looks good!

src/main.ts Show resolved Hide resolved
@andreiborza andreiborza merged commit 6dca92a into master Jan 13, 2025
7 checks passed
@andreiborza andreiborza deleted the ab/collect-tags branch January 13, 2025 14:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants