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

feat: Add telemetry to track chectl command usage #1052

Merged
merged 1 commit into from
Jan 5, 2021
Merged

feat: Add telemetry to track chectl command usage #1052

merged 1 commit into from
Jan 5, 2021

Conversation

flacatus
Copy link
Collaborator

@flacatus flacatus commented Jan 5, 2021

Signed-off-by: Flavius Lacatusu flacatus@redhat.com

What does this PR do?

This PR introduce to chectl the ability to collect and send to segment commands usage and flags:

  • I create hooks which are executed in every command and emit the analytic hook
  • Created a segment adapter to send the usage data to segment
  • Cli Prompt to ask user if allows chectl to collect anonymous data and send to segment
  • Store the user confirmation of usage data into chectl config directory. Is usefull to not ask user in every command if allow chectl to collect data

Refference issue:eclipse-che/che#18069

Screenshot/screencast of this PR

What issues does this PR fix or reference?

How to test this PR?

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Signed-off-by: Flavius Lacatusu <flacatus@redhat.com>
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: flacatus, tolusha
To complete the pull request process, please assign after the PR has been reviewed.
You can assign the PR to them by writing /assign in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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.

3 participants