Skip to content

Conversation

@Grim-lock
Copy link
Contributor

octavia API has some difference from neutron lbaas API, so create a separate directory, and I will add L7 API later.

@auhlig
Copy link
Member

auhlig commented Aug 29, 2017

That was a lot of work. Many thanks @Grim-lock!
Although this looks very promising could you add tests?

@Grim-lock
Copy link
Contributor Author

@auhlig finished the test case

@auhlig
Copy link
Member

auhlig commented Sep 6, 2017

Perfect @Grim-lock! Many thank. Good to merge from my side.
What do you think @vinodborole ?

@auhlig auhlig added this to the 3.1.0 Release milestone Sep 6, 2017
@auhlig auhlig mentioned this pull request Sep 6, 2017
@vinodborole
Copy link
Contributor

@Grim-lock many thanks. LGTM

@auhlig auhlig merged commit 719db04 into ContainX:master Sep 7, 2017
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