Skip to content
This repository was archived by the owner on Jul 25, 2019. It is now read-only.

Fix network pod teardown #277

Merged
merged 1 commit into from
Apr 4, 2016
Merged

Fix network pod teardown #277

merged 1 commit into from
Apr 4, 2016

Conversation

pravisankar
Copy link

del_ovs_flows() uses ovs port to fetch qos record,
so delete ovs port in the end.

Bug fix for https://bugzilla.redhat.com/show_bug.cgi?id=1322077

Fix network pod teardown
8cc7397 
del_ovs_flows() uses ovs port to fetch qos record,
so delete ovs port in the end.
@pravisankar
Copy link
Author

This was the issue:

  • del_ovs_port
  • ovs-vsctl --if-exists del-port vethd07ecd9
  • del_ovs_flows
  • ovs-ofctl -O OpenFlow13 del-flows br0 ip,nw_dst=10.128.0.2
  • ovs-ofctl -O OpenFlow13 del-flows br0 ip,nw_src=10.128.0.2
  • ovs-ofctl -O OpenFlow13 del-flows br0 arp,nw_dst=10.128.0.2
  • ovs-ofctl -O OpenFlow13 del-flows br0 arp,nw_src=10.128.0.2
    ++ ovs-vsctl get port vethd07ecd9 qos
    ovs-vsctl: no row "vethd07ecd9" in table Port
  • qos=
    , exit status 1

@pravisankar
Copy link
Author

@openshift/networking PTAL

@danwinship
Copy link
Contributor

LGTM

@dcbw dcbw merged commit 18a0250 into openshift:master Apr 4, 2016
@dcbw
Copy link
Contributor

dcbw commented Apr 4, 2016

LGTM

@pravisankar pravisankar deleted the fix-pod-teardown branch April 5, 2016 21:37
@pravisankar pravisankar mentioned this pull request Apr 11, 2016
Merged
@dragon9783
Copy link

LGTM

@pweil- pweil- mentioned this pull request Apr 15, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pravisankar @danwinship @dcbw @dragon9783