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

NIT: EIP: Healthchecks have moved to ovnkube-controller #3746

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

tssurya
Copy link
Member

@tssurya tssurya commented Jul 5, 2023

These logs are already moved to cluster-manager

- What this PR does and why is it needed

- Special notes for reviewers

- How to verify it

- Description for the changelog

@kyrtapz
Copy link
Contributor

kyrtapz commented Jul 6, 2023

the code moved here:

if config.OVNKubernetesFeature.EgressIPReachabiltyTotalTimeout == 0 {
klog.V(2).Infof("EgressIP node reachability check disabled")
} else if config.OVNKubernetesFeature.EgressIPNodeHealthCheckPort != 0 {
klog.Infof("EgressIP node reachability enabled and using gRPC port %d",
config.OVNKubernetesFeature.EgressIPNodeHealthCheckPort)
}

/lgtm

These logs are already moved to cluster-manager

Signed-off-by: Surya Seetharaman <[email protected]>
@tssurya
Copy link
Member Author

tssurya commented Jul 6, 2023

/cc @jcaamano for approve

@dcbw dcbw merged commit c86a631 into ovn-org:master Jul 7, 2023
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants