Skip to content

Commit

Permalink
[8982026] fix metrics e2e test (Azure#4125)
Browse files Browse the repository at this point in the history
* added metrics for current number of connected clients, individual device connect/disconnected to/from iot edge

* fixed metrics description

* fixed UT ConnectionManagerTest by adding missing mock which impacted by metrics change

* Update edge-hub/src/Microsoft.Azure.Devices.Edge.Hub.Core/DeviceConnectionMetrics.cs

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* Update edge-hub/src/Microsoft.Azure.Devices.Edge.Hub.Core/DeviceConnectionMetrics.cs

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* Update edge-hub/src/Microsoft.Azure.Devices.Edge.Hub.Core/DeviceConnectionMetrics.cs

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* Update doc/BuiltInMetrics.md

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* Update doc/BuiltInMetrics.md

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* Update doc/BuiltInMetrics.md

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* Update doc/BuiltInMetrics.md

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>

* modified to use Array.Empty

* modified failed connection metrics description to align with doc and added ignore disconnect in e2e test

* fixed typo in md file

Co-authored-by: Venkat Yalla <veyalla@microsoft.com>
  • Loading branch information
davilu and veyalla committed Dec 18, 2020
1 parent 19a86aa commit feeb588
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion doc/BuiltInMetrics.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ instance_number | A Guid representing the current runtime. On restart, all metri
| `edgehub_operation_retry_total` | `id` (Module ID)<br>`operation` (Operation name) | Total number of times edgeHub operations were retried | counter |
| `edgehub_client_connect_failed_total` | `id` (Device/Module ID)<br> `reason` (not authenticated)<br> | Total number of times each client failed to connect to edgeHub | counter |
| `edgehub_connected_clients` | | Current number of clients connected to edgeHub | gauge |
| `edgehub_client_connect_success_total` | `id` (Device/Module ID)<br> | Total number of times each client successfully connected to edgeHub | counter | | counter |
| `edgehub_client_connect_success_total` | `id` (Device/Module ID)<br> | Total number of times each client successfully connected to edgeHub | counter |
| `edgehub_client_disconnect_total` | `id` (Device/Module ID)<br> | Total number of times each client disconnected from edgeHub | counter |


Expand Down

0 comments on commit feeb588

Please sign in to comment.