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

Enable 'direct-pod-ip' mode LB svc #2

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

Conversation

chestack
Copy link

@chestack chestack commented Jun 20, 2022

  • add new flag 'direct-pod-ip'(default is true) to identify if service controller create LB svc in directPodIP mode or nodePort mode
  • add goroutine to sync endpoints changes of svc, update service LB if endpoints changed(pods not ready or replicas scaled out/in)
  • do sync endpoints change if directPodIP mode, otherwise sync nodes change if nodePort mode

Related-Task: #https://easystack.atlassian.net/browse/EAS-105343

@chestack
Copy link
Author

note: cloud-provider-openstack depends on a release from tag v0.24.0+this change

@chestack chestack requested review from xieyunyue and yylt June 20, 2022 03:25
@chestack chestack force-pushed the direct-pod-ip branch 2 times, most recently from 04425e5 to 13c7c6b Compare June 20, 2022 10:18
@chestack chestack changed the title [WIP] Enable 'direct-pod-ip' mode LB svc Enable 'direct-pod-ip' mode LB svc Jun 20, 2022
@chestack chestack force-pushed the direct-pod-ip branch 3 times, most recently from 5ca6fb2 to d4234f9 Compare June 21, 2022 04:00
1.add new flag 'direct-pod-ip'(default is true) to identify
  if service controller create LB svc in directPodIP mode or nodePort mode
2.add goroutine to sync endpoints changes of svc,
  update svc LB if endpoints changed(pods not ready or replicas scaled)
3.do sync endpoints change if directPodIP mode,
  otherwise sync nodes change if nodePort mode

Closes-Task: #EAS-105343
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.

1 participant