Add latency metric for notification service

This MR is to create custom metric for event notification latency. The latency is the time between the message is sent into message bus until the first successful attempt to deliver the message to subscriber. The response time from the subscriber is not included.

In the customDimension of the metric, we included correlation id, data partition id and topic name the message belongs to.

Issue: #126

changed milestone to %M25 - Release 0.28

added Azure label

changed the description

added MREnhancement label

mentioned in issue #126

marked this merge request as ready

changed title from Draft: Vl/latency metric to Add latency metric for notification service

changed the description

requested review from @danielscholl, @Rustam_Lotsmanenko, @ajoshi19, @ylesnikau, @dhudsons, and @ashley_beitz

assigned to @vleskiv

approved this merge request

I'm interested to see if the pipeline passes for non-azure providers, as the metric service impl is only added for Azure. I'm inclined to keep the boolean parameter csp specific, and not always set to true.

Update: made this MR as DRAFT until I make the said changes (which should be tomorrow, hopefully)

unapproved this merge request

changed title from Add latency metric for notification service to DRAFT - Add latency metric for notification service

added 3 commits

• a7b51df9...d0ce9b88 - 2 commits from branch master
• fef8a42c - Merge branch 'master' of...

Compare with previous version

changed title from DRAFT - Add latency metric for notification service to Add latency metric for notification service

added Common Code MRLogging labels and removed MREnhancement label

resolved all threads

added 1 commit

• 7be00a41 - add other provider impls

Compare with previous version

added 1 commit

• 81a3abe9 - Updating NOTICE

Compare with previous version

added 3 commits

• 81a3abe9...85475e78 - 2 commits from branch master
• a5630d6a - Merge branch 'master' of...

Compare with previous version

Looks mostly good, but it seems like quite a few of the files (and all of the new files) lack a license header.

requested changes

added 1 commit

• ea109efe - addressed comments

Compare with previous version

resolved all threads

approved this merge request

approved this merge request

mentioned in commit 43b308a5

merged