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

Handle dual stack pod IPs #1168

Merged
merged 4 commits into from Nov 27, 2019
Merged

Handle dual stack pod IPs #1168

merged 4 commits into from Nov 27, 2019

Conversation

@neiljerram
Copy link
Member

neiljerram commented Nov 22, 2019

This PR:

  • updates the k8s API code to a level that includes the new PodStatus.PodIPs field
  • enhances Pod -> WorkloadEndpoint conversion to take account of PodStatus.PodIPs
  • adds and honours a cni.projectcalico.org/podIPs annotation, analogous to the existing cni.projectcalico.org/podIP
  • adds UT for the Pod -> WorkloadEndpoint conversion.

UT for the generation of the podIPs annotation is in projectcalico/cni-plugin#811.

Handle dual stack pod IPs
Specifically, this was:

    go get k8s.io/api@4ed536977f46736ee9334e6e29097551f17e8211
@neiljerram neiljerram force-pushed the neiljerram:dual-stack branch from f5e4abf to b2015ad Nov 27, 2019
@neiljerram neiljerram changed the title WIP: Handle dual stack pod IPs Handle dual stack pod IPs Nov 27, 2019
Copy link
Member

song-jiang left a comment

LGTM

@neiljerram neiljerram merged commit 0216c08 into projectcalico:master Nov 27, 2019
1 check passed
1 check passed
semaphoreci The build passed on Semaphore.
Details
@neiljerram neiljerram deleted the neiljerram:dual-stack branch Nov 27, 2019
@lmm lmm added this to the Calico v3.11.0 milestone Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.