druid-io / druid-operator

Druid Kubernetes Operator
Other
205 stars 93 forks source link

Introduce State Driven Status, use record pkg for event and clean logging #223

Closed AdheipSingh closed 3 years ago

AdheipSingh commented 3 years ago

Fixes #195 Fixes #190

Description

Changes

  1. Introduce druid cluster status
  2. Use event recorder pkg
  3. clean up unnecessary logging
  4. zero pollution of events as well as logs, reduces the requests on k8s api
  5. state driven status.
  6. EventEmitter interface - can be extended in the future to emit events to various backends ( kafka, statsd just like druid does )

Logging


This PR has:


Key changed/added files in this PR
AdheipSingh commented 3 years ago

@himanshug will need your review here.

himanshug commented 3 years ago

sorry, will review it this week.

AdheipSingh commented 3 years ago

@himanshug :)

himanshug commented 3 years ago

I took a quick look today, LGTM

We can merge this if you have already had this deployed on some clusters and it is functioning as expected.

AdheipSingh commented 3 years ago

We can merge this if you have already had this deployed on some clusters and it is functioning as expected.

yes, we have been running this on prod for couple of months.

@nishantmonu51 waiting on you for this.