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.

Sign up for GitHub

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

Jump to bottom

fix ips CR not found due to etcd error #2472

Merged
merged 1 commit into from Mar 13, 2023
Merged

fix ips CR not found due to etcd error #2472

merged 1 commit into from Mar 13, 2023

Conversation

zhangzujian
Copy link
Member

@zhangzujian zhangzujian commented Mar 13, 2023 •
edited

What type of this PR

  • Bug fixes

Which issue(s) this PR fixes:

If ips CR creation failed, the cni server can not get the CR and reports an error. This results in Pod creation failure:

  Warning  FailedCreatePodSandBox  4m27s (x268 over 62m)  kubelet            (combined from similar events): Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "9ca82d1f5769c01278d15bf5a641453a385f3a8ab1202e2dfe389b630e1c4850": plugin type="kube-ovn" failed (add): request ip return 500 failed to get ip crd for 10.3.0.31, ips.kubeovn.io "deploy-78974c7456-zmzs8.cpaas-system" not found

@zhangzujian zhangzujian added bug Something isn't working need backport labels Mar 13, 2023
@zhangzujian zhangzujian marked this pull request as ready for review March 13, 2023 06:08
@zhangzujian zhangzujian merged commit 0689a72 into kubeovn:master Mar 13, 2023
@zhangzujian zhangzujian deleted the fix-ip-cr branch March 13, 2023 07:28
zhangzujian added a commit that referenced this pull request Mar 13, 2023
@zhangzujian
fix ips CR not found due to etcd error (#2472)
b414585
zhangzujian added a commit that referenced this pull request Mar 13, 2023
@zhangzujian
fix ips CR not found due to etcd error (#2472)
8d08c62
zhangzujian added a commit that referenced this pull request Mar 13, 2023
@zhangzujian
fix ips CR not found due to etcd error (#2472)
bfa779d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hongzhen-ma hongzhen-ma hongzhen-ma approved these changes

Assignees
No one assigned
Labels
bug Something isn't working need backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@zhangzujian @hongzhen-ma