Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kubevirt, e2e: Fill in the http server pods slice #4224

Merged

Conversation

qinqon
Copy link
Contributor

@qinqon qinqon commented Mar 14, 2024

- What this PR does and why is it needed
At kubevirt e2e tests the pods create at each nodes where not being added to the pods slice, this means that they were not created and the test for east/west communication was not run.

This change fill in again that slice so the test is run again.

Copy link

netlify bot commented Mar 14, 2024

Deploy Preview for subtle-torrone-bb0c84 ready!

Name Link
🔨 Latest commit a180787
🔍 Latest deploy log https://app.netlify.com/sites/subtle-torrone-bb0c84/deploys/65f871d42adf7e00088ebc0d
😎 Deploy Preview https://deploy-preview-4224--subtle-torrone-bb0c84.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@qinqon qinqon force-pushed the kubevirt-e2e-fill-in-httpsevertestpods branch from f23d0b1 to 394bdf5 Compare March 14, 2024 11:36
@coveralls
Copy link

coveralls commented Mar 14, 2024

Coverage Status

coverage: 52.472% (-0.03%) from 52.497%
when pulling 0f5a20a on qinqon:kubevirt-e2e-fill-in-httpsevertestpods
into 7d164bd on ovn-org:master.

@qinqon qinqon force-pushed the kubevirt-e2e-fill-in-httpsevertestpods branch from 394bdf5 to 78913ec Compare March 14, 2024 12:08
@tssurya tssurya added area/e2e-testing feature/kubevirt-live-migration All issues related to kubevirt live migration labels Mar 14, 2024
@qinqon qinqon force-pushed the kubevirt-e2e-fill-in-httpsevertestpods branch 4 times, most recently from 9d4bff0 to 0f5a20a Compare March 18, 2024 06:46
At kubevirt e2e tests the pods create at each nodes where not being
added to the pods slice, this means that they were not created and the
test for east/west communication was not run.

This change fill in again that slice so the test is run again.

Signed-off-by: Enrique Llorente <ellorent@redhat.com>
@qinqon qinqon force-pushed the kubevirt-e2e-fill-in-httpsevertestpods branch from 0f5a20a to a180787 Compare March 18, 2024 16:54
@jcaamano jcaamano merged commit 5359e7d into ovn-org:master Mar 19, 2024
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/e2e-testing feature/kubevirt-live-migration All issues related to kubevirt live migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants