Skip to content

Collect events for analytics#77

Merged
PrasadG193 merged 3 commits intomainfrom
collect-events
May 31, 2021
Merged

Collect events for analytics#77
PrasadG193 merged 3 commits intomainfrom
collect-events

Conversation

@PrasadG193
Copy link
Copy Markdown
Member

  • Add new command kbrew analytics to check status and turn on/off the analytics collection
  • Collect following events: install-success, install-fail, install-timeout and k8s-events
  • Collect and report k8s-events in case of install-fail and install-timeout.
  • k8s-events are the Warning level events for involved Pods

Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>
@PrasadG193 PrasadG193 requested review from sahil-lakhwani and vishal-biyani and removed request for sahil-lakhwani and vishal-biyani May 27, 2021 06:02
Copy link
Copy Markdown
Contributor

@sahil-lakhwani sahil-lakhwani left a comment

Choose a reason for hiding this comment

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

I think there are now many kube clients being created. We should follow up with a PR to have a global client if possible.

Comment thread pkg/kube/kube.go
Comment thread pkg/events/events.go
Comment thread pkg/config/config.go
Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>
Signed-off-by: Prasad Ghangal <prasad.ghangal@gmail.com>
@PrasadG193 PrasadG193 requested a review from sahil-lakhwani May 30, 2021 11:47
@PrasadG193 PrasadG193 merged commit 7edf4a8 into main May 31, 2021
@PrasadG193 PrasadG193 deleted the collect-events branch May 31, 2021 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants