Skip to content

EgressIP coverage#382

Open
josecastillolema wants to merge 2 commits into
cloud-bulldozer:mainfrom
josecastillolema:eip
Open

EgressIP coverage#382
josecastillolema wants to merge 2 commits into
cloud-bulldozer:mainfrom
josecastillolema:eip

Conversation

@josecastillolema

@josecastillolema josecastillolema commented May 4, 2026

Copy link
Copy Markdown
Member

Egress IP coverage for the weekly testing.

$ VERSION=4.22 uv run orion --config examples/metal-perfscale-cpt-egressip.yaml --lookback 5d  --hunter-analyze --es-server=https://server.com --metadata-index=metadata --benchmark-index=benchmark
...
metal-perfscale-cpt-node-density
================================
time                 uuid                                  ocpVersion                            podReadyLatency_P99    containersStartedLatency_P99    ovsCPU-Workers_avg    eipStartupLatencyTotal_value    startupEipNodeIPReqCount_avg
-------------------  ------------------------------------  ----------------------------------  ---------------------  ------------------------------  --------------------  ------------------------------  ------------------------------
2026-05-02 19:16:08  189bd0b3-2aff-4aad-a070-2b4163bf832f  4.22.0-0.nightly-2026-05-02-045017                   2000                            1849               2254.34                               0                         452.108

time                 uuid                                  ocpVersion                            eipRecoveryLatencyTotal_value  buildUrl
-------------------  ------------------------------------  ----------------------------------  -------------------------------  ------------------------------------------------------------------------------------------------------------------------------------------------------------------
2026-05-02 19:16:08  189bd0b3-2aff-4aad-a070-2b4163bf832f  4.22.0-0.nightly-2026-05-02-045017                              nan  https://prow.ci.openshift.org/view/gs/origin-ci-test/logs/periodic-ci-openshift-eng-ocp-qe-perfscale-ci-main-metal-4.22-nightly-x86-weekly-eip/2050621496811851776
No regressions found

@venkataanil can you please review the eip related values to see if they make sense:

  • eipStartupLatencyTotal_value: 0
  • startupEipNodeIPReqCount_avg: 452.108
  • eipRecoveryLatencyTotal_value: NaN

Signed-off-by: Jose Castillo Lema <josecastillolema@gmail.com>
@openshift-ci

openshift-ci Bot commented May 4, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mcornea for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: Jose Castillo Lema <josecastillolema@gmail.com>
@josecastillolema
josecastillolema requested review from chentex, mohit-sheth and venkataanil and removed request for avasilevskii and vishnuchalla May 4, 2026 09:26
@chentex

chentex commented May 4, 2026

Copy link
Copy Markdown
Member

/lgtm

@venkataanil venkataanil left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

eipStartupLatencyTotal is 0, that needs be investigated

direction: 1
threshold: 10

- name: startupEipNodeIPReqCount

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is not required

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Do you mean direction/threshold or the startupEipNodeIPReqCount metric?

@josecastillolema

Copy link
Copy Markdown
Member Author

eipStartupLatencyTotal is 0, that needs be investigated

Since you are familiar with the implementation, can you please take a look?

@chentex
chentex requested a review from venkataanil June 3, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants