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

telemetry: only send cli version #751

Merged
merged 4 commits into from
Jul 25, 2024
Merged

telemetry: only send cli version #751

merged 4 commits into from
Jul 25, 2024

Conversation

miampf
Copy link
Contributor

@miampf miampf commented Jul 19, 2024

Only send the CLI version information in telemetry requests instead of a full version log obtained by contrast --version.

@miampf miampf requested a review from katexochen as a code owner July 19, 2024 13:10
@miampf miampf added the no changelog PRs not listed in the release notes label Jul 19, 2024
@miampf miampf force-pushed the miampf/telemetry/version branch 2 times, most recently from af366e2 to 35b31da Compare July 19, 2024 13:18
cli/main.go Outdated Show resolved Hide resolved
@miampf miampf force-pushed the miampf/telemetry/version branch from 35b31da to cd0a520 Compare July 25, 2024 11:04
@miampf miampf requested a review from katexochen July 25, 2024 12:01
@miampf miampf merged commit 8570926 into main Jul 25, 2024
9 checks passed
@miampf miampf deleted the miampf/telemetry/version branch July 25, 2024 12:15
@katexochen katexochen added bug fix Fixing a user facing bug backport release/v0.8 and removed no changelog PRs not listed in the release notes labels Aug 1, 2024
@katexochen
Copy link
Member

/backport

edgelessci pushed a commit that referenced this pull request Aug 1, 2024
@edgelessci
Copy link
Contributor

Successfully created backport PR for release/v0.8:

katexochen pushed a commit that referenced this pull request Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release/v0.8 bug fix Fixing a user facing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants