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

fix(smoke-test): dynamically fetch node name for service curl test #811

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aghilish
Copy link

Previously, the smoke test assumed the nginx pod would always be scheduled on node-0, which could lead to failures if the pod was placed on a different node. This update fixes that by dynamically fetching the node where the pod is running.

Changes:
Updated the smoke test to retrieve the correct node name for the curl request instead of assuming node-0.

@elsonrodriguez
Copy link
Contributor

elsonrodriguez commented Nov 16, 2024

This is an interesting one. I believe traffic to nodeports on any machine should still forward to a pod on a different machine, but I also saw this behavior in my resulting cluster after finishing the tutorial.

I scaled the replicas to 2, and then this fixed it. I scaled replicas down to 1, and the cluster is still forwarding correctly from both machines.

It looks like there is a missing route. The main difference I saw is that after scaling both machines have a cni-related route added:

node-0:

...
10.200.0.0/24 dev cni0 proto kernel scope link src 10.200.0.1 linkdown 

node-1

...
10.200.1.0/24 dev cni0 proto kernel scope link src 10.200.1.1 

@vladsf
Copy link

vladsf commented Dec 24, 2024

Yes, I noticed the same behaviour

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