[Backport 2.x] Adds latency metrics for ClusterState Appliers and Listeners

[opensearch-trigger-bot]

Backport a254aa9 from #12333.

[github-actions]

❕ Gradle check result for 6d23c1d: UNSTABLE

• TEST FAILURES:

      1 org.opensearch.remotestore.SegmentReplicationUsingRemoteStoreIT.testCancellation

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

[mch2]

Detect breaking changes workflow is correctly catching an issue here - This breaks the ctor for ClusterService which is marked as api - ex. https://github.com/opensearch-project/asynchronous-search/pull/576/files

[shwetathareja]

Detect breaking changes workflow is correctly catching an issue here - This breaks the ctor for ClusterService which is marked as api - ex. https://github.com/opensearch-project/asynchronous-search/pull/576/files

@mch2 ClusterService, MasterService, AllocationService, for all these classes singleton objects are created in Node.java and then injected all throughout. How do you suggest we can handle constructor changes here?

[opensearch-trigger-bot]

This PR is stalled because it has been open for 30 days with no activity.

[mch2]

Detect breaking changes workflow is correctly catching an issue here - This breaks the ctor for ClusterService which is marked as api - ex. https://github.com/opensearch-project/asynchronous-search/pull/576/files

@mch2 ClusterService, MasterService, AllocationService, for all these classes singleton objects are created in Node.java and then injected all throughout. How do you suggest we can handle constructor changes here?

apologies @shwetathareja for the late reply - seems this has been resolved with overloading the ctors in #13926. I think we can merge this backport with a rebase?

[gargharsh3134]

Another backport PR was merged to address this (#14118).
This PR is no longer required and can be closed.