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

base: remove ovn patch for skipping ct #3141

Merged
merged 3 commits into from
Aug 17, 2023
Merged

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Aug 15, 2023

What type of this PR

  • Bug fixes

Which issue(s) this PR fixes:

In redhat/rocky 8.6, pods cannot coommunicate with each other via pod ips:

tcp      6 117 SYN_SENT src=10.16.0.6 dst=10.16.0.4 sport=37890 dport=9153 [UNREPLIED] src=10.16.0.4 dst=10.16.0.6 sport=9153 dport=37890 mark=0 zone=9 use=1
tcp      6 298 ESTABLISHED src=10.16.0.4 dst=10.16.0.6 sport=9153 dport=37890 [UNREPLIED] src=10.16.0.6 dst=10.16.0.4 sport=37890 dport=51655 mark=0 zone=8 use=1

WHAT

🤖 Generated by Copilot at 621b685

Increased the timeout-minutes for two GitHub workflow jobs that run network policy tests. This is to prevent the jobs from failing due to timeout errors.

🤖 Generated by Copilot at 621b685

timeout-minutes grows
netpol tests are slow and flaky
autumn leaves fall fast

HOW

🤖 Generated by Copilot at 621b685

  • Increase the timeout-minutes for two jobs that run time-consuming and flaky tests (link, link)

@zhangzujian zhangzujian marked this pull request as ready for review August 15, 2023 09:18
@zhangzujian zhangzujian merged commit 46cee03 into kubeovn:master Aug 17, 2023
65 checks passed
@zhangzujian zhangzujian deleted the fix-ct branch August 17, 2023 07:18
zhangzujian added a commit that referenced this pull request Aug 17, 2023
* base: remove ovn patch for skipping ct
* add gdb to base image
* ci: run k8s network policy e2e in series

---------

Signed-off-by: 张祖建 <[email protected]>
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.

2 participants