Skip to content
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

add teleport version to SubmitEventRequest #49248

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

GavinFrazar
Copy link
Contributor

Depends on cloud PR: https://github.com/gravitational/cloud/pull/11219

Part of our efforts to triage web ui resource enrollment wizard errors is narrowing in on the version of Teleport that was running when an error occurred. This PR adds the teleport cluster (auth server) version to the event request we submit to prehog, which will add it as a property to the posthog event: tp.teleport_version.

proto/prehog/v1alpha/teleport.proto Show resolved Hide resolved
@GavinFrazar GavinFrazar force-pushed the gavinfrazar/add-version-to-error-reporting branch from 8211412 to f98184d Compare November 20, 2024 19:48
@GavinFrazar
Copy link
Contributor Author

i'll wait for the cloud change to be released before merging backports.

@GavinFrazar GavinFrazar added this pull request to the merge queue Nov 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 22, 2024
@GavinFrazar GavinFrazar added this pull request to the merge queue Nov 22, 2024
Merged via the queue into master with commit c62feda Nov 22, 2024
44 checks passed
@GavinFrazar GavinFrazar deleted the gavinfrazar/add-version-to-error-reporting branch November 22, 2024 22:29
@public-teleport-github-review-bot

@GavinFrazar See the table below for backport results.

Branch Result
branch/v15 Failed
branch/v16 Failed
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants