Skip to content

Conversation

gabhijit-izel
Copy link
Contributor

@gabhijit-izel gabhijit-izel commented Nov 3, 2016

Description

Intermediate pull request for #15. Improvements identified basic working code, good enough to merge into master.

Breaking

YES

Breaking changes

One should update the Openstack-Shell driver when this pull request is merged into develop. Will not work with old driver.


This change is Reviewable

Basic implementation of Connectivity PoC. Very basic, not tested yet
Updating the version to test from pypi. So that new dependencies are all downloaded
Basic connectivity working - now. Apps can connect using VLAN service.
Subnets are not hard-coded anymore - they are taken from an exclusion list of "reserverd networks". Fixed issues where if a network already existed to use it
Upped the version to 0.0.15
Upped the version to 0.0.15
Upped the version to 0.0.15
Fixed Docstrings.
@coveralls
Copy link

Coverage Status

Coverage decreased (-12.3%) to 82.696% when pulling f34b7ad on feature/gabhijit-izel_15_poc_connectivity into 70c5301 on develop.

@alexazarh
Copy link
Contributor

Reviewed 20 of 20 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from Reviewable

@alexazarh alexazarh merged commit 0a91dda into develop Nov 3, 2016
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