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 Azure IPAM regression #13397

Merged
merged 2 commits into from
Oct 5, 2020
Merged

Fix Azure IPAM regression #13397

merged 2 commits into from
Oct 5, 2020

Conversation

tgraf
Copy link
Member

@tgraf tgraf commented Oct 4, 2020

No description provided.

The commit dd99958 introduced support for multiple interfaces and
added a new requirement for the gateway IP to be known. This is not
always the case. Make it optional.

This resulted in errors like these:
```
Warning  FailedCreatePodSandBox  3m46s (x264 over 8m38s)  kubelet, aks-nodepool1-10706209-vmss000001  (combined from similar events): Failed to create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "600afaf8013aac81bdd001e12f4ee115ef07a9c1520dc72b7c9c449a8f3f3b86" network for pod "coredns-869cb84759-jrxdd": networkPlugin cni failed to set up pod "coredns-869cb84759-jrxdd_kube-system" network: unable to setup interface datapath: unable to parse routing info: invalid ip:
```

Fixes: #13154
Fixes: dd99958 ("azure: support multiple pods subnets")
Signed-off-by: Thomas Graf <thomas@cilium.io>
@tgraf tgraf added kind/bug This is a bug in the Cilium logic. release-note/bug This PR fixes an issue in a previous release of Cilium. kind/regression This functionality worked fine before, but was broken in a newer release of Cilium. needs-backport/1.8 labels Oct 4, 2020
@tgraf tgraf requested a review from a team as a code owner October 4, 2020 19:50
@tgraf tgraf requested a review from a team October 4, 2020 19:50
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.5 Oct 4, 2020
When following the AKS installation instructions, there will be existing
pods which are not managed by Cilium. Ask the user to restart them.

Fixes: #13154

Signed-off-by: Thomas Graf <thomas@cilium.io>
@tgraf
Copy link
Member Author

tgraf commented Oct 4, 2020

test-me-please

@tgraf
Copy link
Member Author

tgraf commented Oct 5, 2020

The GKE test failed because the FQDN proxy can't bind to the port:

2020-10-04T21:30:16.42229825Z level=fatal msg="Error while creating daemon" error="listen tcp :40457: bind: address already in use" subsys=daemon

Filed: #13400

@tgraf
Copy link
Member Author

tgraf commented Oct 5, 2020

test-gke

@tgraf
Copy link
Member Author

tgraf commented Oct 5, 2020

Only 1.8 backport is needed, the bug is not present in 1.7.

@tgraf tgraf merged commit 0c5758d into master Oct 5, 2020
@tgraf tgraf deleted the pr/tgraf/fix-azure-ipam branch October 5, 2020 11:13
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.5 Oct 6, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.5 Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This is a bug in the Cilium logic. kind/regression This functionality worked fine before, but was broken in a newer release of Cilium. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
No open projects
1.8.5
Backport done to v1.8
Development

Successfully merging this pull request may close these issues.

None yet

3 participants