Use a more sensible network topology for some router tests

This is achieved by creating subnets on separate logical
networks. A negative tests is also added to verify that
interface attachment fails when done with overlapping
subnets.

Closes-bug: #1316742

Change-Id: Idb1965e32d30acd962e1a3fa755b6c8b25c1bc31
Sahara: add API tests for job binary internals

* create/list/get/delete API tests for job binary internals were added
* "get_job_binary_internal_data" method to Sahara API client was added
* "test_job_binary_internal_get_data" test was added

Partially implements: blueprint savanna-api-tests

Change-Id: Ieaa765a1e941e5c1ac1b866099dbf966d64399b2
VPNaas IPSec policies tests

testcases (create, update, delete, show, and list) for IPSec policies.

Partially implements: blueprint vpnaas-api-test-cases

Change-Id: I39f5f55ae8ef23ebdef29cefcde7d6e2c4f2640c
Convert get_console_output length=None to -1 for Nova V3 API calls

Nova commit 8e5577f52f660960738f1fc0384dff77fa30fc8a makes the nova v3
get_console_output extension handle unlimited length with -1.

In the nova v2 API the length parameter is optional but the nova v3 API
uses jsonschema validation so it expects a value to check against.

Closes-Bug: #1328582

Change-Id: Ib1ae85e644e08c24aaf0ba2baaa8812286ef0ebd
Cleanup for local variable in test_attach_volume

This patch cleanup the test_attach_volume for unnecessary and
unused local variable.

During https://review.openstack.org/#/c/87191/7/, it came out
that we should clean up this file for unnecessary local variable.
So for logical breakdown of changes, this patch do the cleanup
part separately.

Change-Id: I4afc88a09f43f8b068b16368cbddb4bff21059c6
Heat Overlapping ip issue

The hard coded cidr used by heat neutron scenario,
was overlapping with the host systems cidr.
Because the neutron scenario does not used his own router,
it also caused issue for the cfn init scenario.

* The neutron scenario will use the first configured cidr
* The neutron scenario will use his own router
* The neutron scenario will retry singaling the heat-cfn-api,
  because if neutron is overload and the vm boot is fast and we
  do not wait even for the meta data api service connectivity (enabled
  config drive), the signal could be sent in a wrong time when
  the L3 connectivity is not ready, but the L2 is.
* Let neutron decide the allocation_pools (simplification)
* Using comma_delimited_list for passing not hard coded dns servers
  to the stack

Change-Id: I06bd197b0f6c012a1416016a40f29ddd080b21b9
Partial-Bug: #1297560
Merge "Heat Overlapping ip issue"
Merge "Sahara: add API tests for job binary internals"
Merge "VPNaas IPSec policies tests"
Merge "Cleanup for local variable in test_attach_volume"
Merge "Convert get_console_output length=None to -1 for Nova V3 API calls"
Merge "Use a more sensible network topology for some router tests"
Merge branch 'master' of https://github.com/openstack/tempest