Skip to content

Comments

Add ZooKeeper connection state alerts and metrics.#14333

Merged
gianm merged 11 commits intoapache:masterfrom
gianm:zk-metrics
Jul 12, 2023
Merged

Add ZooKeeper connection state alerts and metrics.#14333
gianm merged 11 commits intoapache:masterfrom
gianm:zk-metrics

Conversation

@gianm
Copy link
Contributor

@gianm gianm commented May 23, 2023

  • New metric "zk/connected" is an indicator showing 1 when connected, 0 when disconnected.
  • New metric "zk/reconnect/time" measures time spent disconnected prior to a reconnection.
  • New alert when Curator connection state enters LOST or SUSPENDED.

- New metric "zk/connected" is an indicator showing 1 when connected,
  0 when disconnected.
- New metric "zk/disconnected/time" measures time spent disconnected.
- New alert when Curator connection state enters LOST or SUSPENDED.
Copy link
Member

@asdf2014 asdf2014 left a comment

Choose a reason for hiding this comment

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

👍

@gianm gianm merged commit 3ff5148 into apache:master Jul 12, 2023
@gianm gianm deleted the zk-metrics branch July 12, 2023 16:34
sergioferragut pushed a commit to sergioferragut/druid that referenced this pull request Jul 21, 2023
* Add ZooKeeper connection state alerts and metrics.

- New metric "zk/connected" is an indicator showing 1 when connected,
  0 when disconnected.
- New metric "zk/disconnected/time" measures time spent disconnected.
- New alert when Curator connection state enters LOST or SUSPENDED.

* Use right GuardedBy.

* Test fixes, coverage.

* Adjustment.

* Fix tests.

* Fix ITs.

* Improved injection.

* Adjust metric name, add tests.
@LakshSingla LakshSingla added this to the 28.0 milestone Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants