Skip to content

Commit

Permalink
Add functionality to ensure the control host-only interface (vboxnet0…
Browse files Browse the repository at this point in the history
…) with a configured DHCP server exists before proceeding to deploy a virtualbox. Create the interface and the DHCP servers as needed. Added unittests for this functionality
  • Loading branch information
goodwillcoding committed Jun 21, 2014
1 parent 3d994ff commit d64dbb6
Show file tree
Hide file tree
Showing 4 changed files with 819 additions and 18 deletions.
1 change: 1 addition & 0 deletions nixops/backends/tests/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
# -*- coding: utf-8 -*-
1 change: 1 addition & 0 deletions nixops/backends/tests/virtualbox/__init__.py
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
# -*- coding: utf-8 -*-
Loading

2 comments on commit d64dbb6

@aszlig
Copy link

@aszlig aszlig commented on d64dbb6 Jun 21, 2014

Choose a reason for hiding this comment

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

Although I'm personally not a big fan of the # ~* # comments, why not move the tests to where the other tests reside?

@goodwillcoding
Copy link
Owner Author

Choose a reason for hiding this comment

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

The reason I put these tests under backends/virtualbox is because nixops.backend is a provided namespace which tells me other backends can be installed in other packages and would have tests in their respective backend folders.

Please sign in to comment.