Skip to content

Use replace when adding host route#367

Merged
mogren merged 1 commit intoaws:masterfrom
mogren:route-replace
Mar 30, 2019
Merged

Use replace when adding host route#367
mogren merged 1 commit intoaws:masterfrom
mogren:route-replace

Conversation

@mogren
Copy link
Copy Markdown
Contributor

@mogren mogren commented Mar 29, 2019

Issue #, if available:
Related to #344 and #357

Description of changes:

  • Add RouteReplace to the netlink wrapper
  • Use RouteReplace if RouteAdd fails in driver.go
  • More spelling cleanups
  • Removed unused mockgen binary

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

* Add RouteReplace to the netlink wrapper
* Use RouteReplace if RouteAdd fails in driver.go
* More spelling cleanups
* Removed unused mockgen binary
Copy link
Copy Markdown
Contributor

@tiffanyfay tiffanyfay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants