SDN-5217: Add EgressIP route advertisement test #29617
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
openshift-ci
bot
added
the
do-not-merge/work-in-progress
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: pliurh The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: c2bc2ff
New tests seen in this PR at sha: c2bc2ff
|
openshift-ci
bot
removed
the
do-not-merge/work-in-progress
openshift-ci-robot
added
the
jira/valid-reference
|
@pliurh: This pull request references SDN-5217 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
openshift-merge-robot
added
the
needs-rebase
|
Job Failure Risk Analysis for sha: 448c951
|
|
/test e2e-metal-ipi-ovn-dualstack-bgp-techpreview |
openshift-merge-robot
removed
the
needs-rebase
pliurh
force-pushed
the
bgp_eip
branch
2 times, most recently
from
10172f6 to
fb8f630
Compare
|
Job Failure Risk Analysis for sha: fb8f630
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: fb8f630
New tests seen in this PR at sha: fb8f630
|
pliurh
force-pushed
the
bgp_eip
branch
2 times, most recently
from
ba89fe4 to
dbf2eca
Compare
|
Job Failure Risk Analysis for sha: dbf2eca
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: dbf2eca
New tests seen in this PR at sha: dbf2eca
|
pliurh
force-pushed
the
bgp_eip
branch
2 times, most recently
from
e790d2c to
d62f961
Compare
|
/test e2e-metal-ipi-ovn-dualstack-bgp-techpreview |
|
Job Failure Risk Analysis for sha: d62f961
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: d62f961
New tests seen in this PR at sha: d62f961
|
pliurh
force-pushed
the
bgp_eip
branch
4 times, most recently
from
df649d7 to
ab59e98
Compare
|
/retest |
|
/test e2e-metal-ipi-ovn-dualstack-bgp-techpreview |
|
Job Failure Risk Analysis for sha: 1038721
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: 1038721
New tests seen in this PR at sha: 1038721
|
|
Job Failure Risk Analysis for sha: f7905e2
|
|
Job Failure Risk Analysis for sha: f7905e2
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: f7905e2
New tests seen in this PR at sha: f7905e2
|
|
Job Failure Risk Analysis for sha: f7905e2
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: f7905e2
New tests seen in this PR at sha: f7905e2
|
|
/retest |
|
Job Failure Risk Analysis for sha: d320fe2
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: d320fe2
New tests seen in this PR at sha: d320fe2
|
|
Job Failure Risk Analysis for sha: d320fe2
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: d320fe2
New tests seen in this PR at sha: d320fe2
|
|
/test ci/prow/e2e-metal-ipi-serial @pliurh we have some EIP test failing in the serial jobs, not sure if this PR might be related. |
|
@jcaamano: The specified target(s) for The following commands are available to trigger optional jobs: Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/test e2e-metal-ipi-serial |
|
Job Failure Risk Analysis for sha: d320fe2
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: d320fe2
New tests seen in this PR at sha: d320fe2
|
|
Job Failure Risk Analysis for sha: d320fe2
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: d320fe2
New tests seen in this PR at sha: d320fe2
|
The tcpdump output format varies between cloud platforms and bare metal, resulting in the v.s. |
|
@pliurh EGressIP tests still failing in the e2e-gcp-fips-serial and e2e-openstack-serial jobs. I am starting to think that parsing tcpdump output is not working that well for us. |
|
Job Failure Risk Analysis for sha: 3c99105
Risk analysis has seen new tests most likely introduced by this PR. New tests seen in this PR at sha: 3c99105
|
|
Job Failure Risk Analysis for sha: 3c99105
Risk analysis has seen new tests most likely introduced by this PR. New tests seen in this PR at sha: 3c99105
|
1. Tests for both IPv4 and IPv6 EgressIP 2. Tests for advertising EgressIP for the default network and UDN to the default VRF.
Parse tcpdump output based on regex instead of position. Send probe requests with a sequence number for debgging. Use ClusterIP instead of route as the prober target. Signed-off-by: Peng Liu <pliu@redhat.com>
Signed-off-by: Peng Liu <pliu@redhat.com>
|
Job Failure Risk Analysis for sha: 41263c3
Risk analysis has seen new tests most likely introduced by this PR. New tests seen in this PR at sha: 41263c3
|
|
/retest |
|
@pliurh: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
Job Failure Risk Analysis for sha: 41263c3
Risk analysis has seen new tests most likely introduced by this PR. New tests seen in this PR at sha: 41263c3
|
|
Job Failure Risk Analysis for sha: 41263c3
Risk analysis has seen new tests most likely introduced by this PR. New tests seen in this PR at sha: 41263c3
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
to the default VRF.
based on #29419