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

AvailableIPRange E2E failed #2494

Closed
oilbeater opened this issue Mar 16, 2023 · 1 comment · Fixed by #2505
Closed

AvailableIPRange E2E failed #2494

oilbeater opened this issue Mar 16, 2023 · 1 comment · Fixed by #2505
Assignees
@changluyi
Labels
test automation tests

Comments

@oilbeater
Copy link
Collaborator

https://github.com/kubeovn/kube-ovn/actions/runs/4428379583/jobs/7767667188
@oilbeater oilbeater added the test automation tests label Mar 16, 2023
@changluyi
Copy link
Collaborator

changluyi commented Mar 17, 2023 •
edited
Loading

貌似没法用updatestatus去解决，
比如pod创建发现subnet更新触发retry,由于handleAddOrUpdateSubnet会有多次patch ,resourceversion会变多次，如果碰到冲突去get最新的subnet，重新updatestatus,但是这一次retry也没法保证handleAddOrUpdateSubnet的处理逻辑已经走完。

时序图：
handleAddOrUpdateSubnet(patch第一次) =》 handleAddOrUpdateSubnet(patch第二次) =》。。。。

如果pod触发的更新subnet字段发生在第一次和第二次之间，resourceversion没有变化。但实际上handleAddOrUpdateSubnet还没走完。

@changluyi changluyi mentioned this issue Mar 17, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@changluyi changluyi

Labels
test automation tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix subnet iprange not correct kubeovn/kube-ovn
2 participants
@oilbeater @changluyi