Skip to content

Add static routes to the remote driver.#240

Merged
aboch merged 1 commit intomoby:masterfrom
Metaswitch:remote-driver-static-routes
Jun 3, 2015
Merged

Add static routes to the remote driver.#240
aboch merged 1 commit intomoby:masterfrom
Metaswitch:remote-driver-static-routes

Conversation

@tomdee
Copy link
Copy Markdown
Contributor

@tomdee tomdee commented Jun 1, 2015

Signed-off-by: Tom Denham tom.denham@metaswitch.com

@tomdee
Copy link
Copy Markdown
Contributor Author

tomdee commented Jun 3, 2015

ping @mrjana @mavenugo Could I get a review of this PR? Without this fix, static routes aren't exposed over the remote API.

I've discussed it with @squaremo and he's happy with the approach I've taken.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please make this to fit in a single line since there is only one var declared

Signed-off-by: Tom Denham <tom.denham@metaswitch.com>
@tomdee
Copy link
Copy Markdown
Contributor Author

tomdee commented Jun 3, 2015

@mrjana I've pushed changes for two of your comments. I don't think I can implement the other - see #240 (comment)

@mrjana
Copy link
Copy Markdown
Contributor

mrjana commented Jun 3, 2015

LGTM

1 similar comment
@aboch
Copy link
Copy Markdown
Contributor

aboch commented Jun 3, 2015

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants