kubernetes/pkg/cloudprovider/providers/openstack
Kubernetes Submit Queue 40b0c5516a
Merge pull request #56258 from databus23/patch-1
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

openstack: remove orphaned routes from terminated instances

**What this PR does / why we need it**:
At the moment the openstack cloudprovider only returns routes where the `NextHop` address points to an existing openstack instance. This is a problem when an instance is terminated before the corresponding node is removed from k8s. The existing route is not returned by the cloudprovider anymore and therefore never considered for deletion by the route controller. When the route's `DestinationCIDR` is reassigned to a new node the router ends up with two routes pointing to a different `NextHop` leading to broken networking.

This PR removes skipping routes pointing to unknown next hops when listing routes. This should cause [this conditional](93dc3763b0/pkg/controller/route/route_controller.go (L208)) in the route controller to succeed and have the route removed if the route controller [feels responsible](93dc3763b0/pkg/controller/route/route_controller.go (L206)).

```release-note
OpenStack cloudprovider: Ensure orphaned routes are removed.
```
2018-01-18 06:53:15 -08:00
..
BUILD use danglingerror 2018-01-07 18:50:42 +02:00
MAINTAINERS.md create cloudprovider "providers" package 2015-08-11 22:36:51 -07:00
metadata.go openstack: fetch volume path from metadata service 2017-11-09 12:36:18 +02:00
metadata_test.go openstack: fetch volume path from metadata service 2017-11-09 12:36:18 +02:00
openstack.go use danglingerror 2018-01-07 18:50:42 +02:00
openstack_client.go Support VolumeV3 for OpenStack cloud Provider 2017-11-22 16:45:38 +08:00
openstack_instances.go Log message at a better level 2018-01-15 15:06:47 -05:00
openstack_loadbalancer.go The lbaas.opts.SubnetId should be set by subnet id. 2018-01-12 18:59:53 +08:00
openstack_metrics.go Recording openstack metrics 2017-05-22 10:47:08 +08:00
openstack_routes.go Merge pull request #56258 from databus23/patch-1 2018-01-18 06:53:15 -08:00
openstack_routes_test.go Find a server name and router id to test TestRoutes 2017-11-27 16:32:10 -05:00
openstack_test.go Merge pull request #56407 from bugosiphionah/cider_volumeresize_test 2017-12-18 05:42:44 -08:00
openstack_volumes.go move detach out of os volumes attach 2018-01-10 19:09:32 +02:00
OWNERS Volunteer to help with sig/openstack 2017-10-13 16:49:59 +08:00