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

ovn db: add support for listening on pod ip #2235

Merged
merged 8 commits into from
Jan 19, 2023

Conversation

zhangzujian
Copy link
Member

What type of this PR

  • Security

Which issue(s) this PR fixes:

Fixes #2200

@zhangzujian zhangzujian force-pushed the ovn-listen-pod-ip branch 7 times, most recently from 7cfe0cc to 475e7a8 Compare January 12, 2023 01:39
@zhangzujian zhangzujian marked this pull request as ready for review January 12, 2023 02:43
@zhangzujian zhangzujian requested a review from changluyi January 12, 2023 02:43
@zhangzujian zhangzujian force-pushed the ovn-listen-pod-ip branch 3 times, most recently from c61970f to 08a9901 Compare January 12, 2023 06:53
@zhangzujian zhangzujian removed the request for review from changluyi January 12, 2023 06:56
@zhangzujian zhangzujian marked this pull request as draft January 12, 2023 06:56
@zhangzujian zhangzujian force-pushed the ovn-listen-pod-ip branch 5 times, most recently from ae2bad7 to 92d862f Compare January 12, 2023 12:05
@zhangzujian zhangzujian marked this pull request as ready for review January 12, 2023 12:36
@zhangzujian zhangzujian requested a review from changluyi January 12, 2023 12:36
addr = fmt.Sprintf("[%s]", podIps[0])
}
}
addr = os.Getenv("POD_IP")
Copy link
Collaborator

Choose a reason for hiding this comment

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

the name "addr" is changed to "ip" in kube-ovn-controller/cmd

e2enode "k8s.io/kubernetes/test/e2e/framework/node"

"github.com/onsi/ginkgo/v2"

Copy link
Collaborator

Choose a reason for hiding this comment

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

no need white line

@zhangzujian zhangzujian merged commit 625a685 into kubeovn:master Jan 19, 2023
@zhangzujian zhangzujian deleted the ovn-listen-pod-ip branch January 19, 2023 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants