EgressIP coverage#382
Conversation
Signed-off-by: Jose Castillo Lema <josecastillolema@gmail.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Signed-off-by: Jose Castillo Lema <josecastillolema@gmail.com>
|
/lgtm |
venkataanil
left a comment
There was a problem hiding this comment.
eipStartupLatencyTotal is 0, that needs be investigated
| direction: 1 | ||
| threshold: 10 | ||
|
|
||
| - name: startupEipNodeIPReqCount |
There was a problem hiding this comment.
Do you mean direction/threshold or the startupEipNodeIPReqCount metric?
Since you are familiar with the implementation, can you please take a look? |
Egress IP coverage for the weekly testing.
@venkataanil can you please review the eip related values to see if they make sense:
eipStartupLatencyTotal_value: 0startupEipNodeIPReqCount_avg: 452.108eipRecoveryLatencyTotal_value: NaN