SDN-5217: Add EgressIP route advertisement test #29617
Closed
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
Contributor
|
[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. DetailsNeeds 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. DetailsIn 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
|
Contributor
Author
|
/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
Contributor
Author
|
/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
Contributor
|
/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
|
Contributor
Author
The tcpdump output format varies between cloud platforms and bare metal, resulting in the v.s. |
Contributor
|
@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 <[email protected]>
Signed-off-by: Peng Liu <[email protected]>
|
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
|
Contributor
|
/retest |
|
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
|
Contributor
Author
|
/retest |
Contributor
Author
|
/test e2e-aws-ovn-serial |
Contributor
|
@pliurh: The specified target(s) for The following commands are available to trigger optional jobs: Use DetailsIn 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. |
|
Job Failure Risk Analysis for sha: 41263c3
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: 41263c3
New tests seen in this PR at sha: 41263c3
|
Contributor
Author
|
/test e2e-metal-ipi-ovn-dualstack-bgp-techpreview |
1 similar comment
Contributor
Author
|
/test e2e-metal-ipi-ovn-dualstack-bgp-techpreview |
Contributor
|
@pliurh: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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 Test Risks for sha: 41263c3
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 Test Risks for sha: 41263c3
New tests seen in this PR at sha: 41263c3
|
Contributor
Author
|
/test e2e-metal-ipi-ovn-dualstack-bgp-techpreview |
|
Job Failure Risk Analysis for sha: 41263c3
Risk analysis has seen new tests most likely introduced by this PR. New Test Risks for sha: 41263c3
New tests seen in this PR at sha: 41263c3
|
Contributor
|
/close in favor of #29727 |
Contributor
|
@jcaamano: Closed this PR. DetailsIn 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. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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