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: delete static route should consider dualstack #2130

Merged
merged 1 commit into from
Dec 6, 2022
Merged

Conversation

oilbeater
Copy link
Collaborator

Fix bug found in qos e2e failure

I1206 04:12:59.519482      13 pod.go:787] update pod kube-system/coredns-6d4b75cb6d-2f4hg
W1206 04:12:59.525818      13 ovn-nbctl-legacy.go:52] ovn-nbctl command error: ovn-nbctl --timeout=60 --no-wait --if-exists lr-route-del ovn-cluster 10.16.0.6,fd00:10:16::6 in 10ms
E1206 04:12:59.526008      13 pod.go:401] error syncing 'kube-system/coredns-6d4b75cb6d-5r6b8': ovn-nbctl: bad prefix argument: 10.16.0.6,fd00:10:16::6
, "exit status 1", requeuing
W1206 04:12:59.530624      13 ovn-nbctl-legacy.go:52] ovn-nbctl command error: ovn-nbctl --timeout=60 --no-wait --if-exists lr-route-del ovn-cluster 10.16.0.11,fd00:10:16::b in 8ms
E1206 04:12:59.530757      13 pod.go:401] error syncing 'qos/update-netem-qos': ovn-nbctl: bad prefix argument: 10.16.0.11,fd00:10:16::b
, "exit status 1", requeuing
W1206 04:12:59.533704      13 ovn-nbctl-legacy.go:52] ovn-nbctl command error: ovn-nbctl --timeout=60 --no-wait --if-exists lr-route-del ovn-cluster 10.16.0.7,fd00:10:16::7 in 9ms
E1206 04:12:59.533842      13 pod.go:401] error syncing 'kube-system/coredns-6d4b75cb6d-2f4hg': ovn-nbctl: bad prefix argument: 10.16.0.7,fd00:10:16::7

What type of this PR

Examples of user facing changes:

  • Bug fixes

Which issue(s) this PR fixes:

Fixes #(issue-number)

@oilbeater oilbeater added bug Something isn't working need backport labels Dec 6, 2022
@oilbeater oilbeater merged commit 27d22b7 into master Dec 6, 2022
@oilbeater oilbeater deleted the fix/dual branch December 6, 2022 05:51
oilbeater added a commit that referenced this pull request Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need backport
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants