Teardown/setup server enhancements

1. Manager: pass neutron_net_id and neutron_subnet_id in
network_info to setup_server. Pass server_details and
security_services separately to teardown_server

2. Drivers: renamed setup_network/teardown_network methods
to setup_server/teardown_server

3. Generic: pass server backend_details to
set_up_service_instance and delete_service_instance

4 Service_instance: removed passing share_network_id to
setup_server. Return router_id and subnet_id from setup_server.
Added readable naming to service_subnet

Partially-Implements bp setup-teardown-server-enhancements
Change-Id: I9f5ff26fa2635e352043aedf8dde3514deaaba30
Use execute() and trycmd() functions from common code

Update manila's functions 'execute' and 'trycmd' for
using same functions from oslo common code with
Manila's rootwrap.

Partially-implements blueprint use-common-code

Change-Id: I6d6a787d3d091a72dac635289b6647c00fb7df40
Use ssh_execute function from common code

Remove own implementation of 'ssh_execute' function and replace
its usage with 'ssh_execute' function from common code.

Partially-implements blueprint use-common-code

Change-Id: I1c126752dddd420555e2f0e57d9ed5e0b3cde33b
Enabled hacking check H401

* [H401] Docstrings should not start with a space.

Change-Id: Ia128f72f8dd209c9f732e7aa7da4b2d2210d9588
Cleanup manila.utils module (part1)

Removed next functions:
- flatten_dict
- partition_dict
- map_dict_keys
- subset_dict
- check_isinstance
- convert_to_list_dict
- timefunc
- generate_glance_url
- logging_error
- remove_path_on_error
- str_dict_replace
- total_seconds

Usage of 'total_seconds' function was replaced with same
function from common code in timeutils.

Change-Id: I596441a919f9def81a6b60596e74af7a5de22427
py33: use six.iteritems for item iterations (part2)

Second final part of replacement dict.iteritems() with
six.iteritems(dict)

Partially-implements blueprint py3-compatibility

Change-Id: I2c4f23a8ba2defcebb0dacf04cdad9f1dc40c04e
Merge "Use execute() and trycmd() functions from common code"
Merge "Use ssh_execute function from common code"
Merge "Teardown/setup server enhancements"
Merge "py33: use six.iteritems for item iterations (part2)"
Merge "Enabled hacking check H401"
Enabled hacking check H403

* [H403] Multi line docstrings should end on a new line.

Change-Id: Idd2d40c6304a0f75b192dbbe88c1d81b59dfa957
Fix usage of ProcessExecutionError exception

Commit https://github.com/stackforge/manila/commit/2d9a89a065ba7dbed9bb6ae1464a3b9ec99474a7
made changes to Manila, where it uses 'ssh_execute' function
from common code. But exception that is used was not changed.
So, set old exception as link to new one.

Change-Id: If266f07b4c93c50f59f09336b766ce8f8e415509
Closes-Bug: #1340684
Fix E265 block comment should start with '# '

Address the new pep8 warning that block comments should
start with the sequence '# '.

Change-Id: I52ca5d63b1ca9a465ba36dc9254928f858f2a094
Fix E251 unexpected spaces around keyword / parameter equals

Pep8 does not like to have linewraps or spaces around kwarg
assignment, so reindent the lines to fulfill the pep8 rules.

Change-Id: I0bcdcd36400e18fb7afa9d07da3f32ba20efa68b
Address H104  File contains nothing but comments

Those files can just be left empty, no need for a copyright
header.

Change-Id: Ib08633c6e57ffc6126bbd180d32482cd9e846c1b
Fix E131 continuation line unaligned for hanging indent

Just indent them properly to lign up visually.

Change-Id: Id5fdf11b2f0cbd03b4b1f4fcd6e374470ab23e82
Fix E713 test for membership should be 'not in'

Change-Id: I11e4259b3afbb727198ed4eba6cc684c76fb174d
Fix E112 expected an indented block

Comment blocks should be indented alongside the code.

Change-Id: I7cb2207bb2c0dcdc3036c6cf7575133930da8164
Sync with global requirements

Get requirements in sync with global requirements again.
As this updates hacking to a more recent version, add
some blacklisting of hacking warnings to tox.ini to
get the gate passing. Those will be cleaned up
in followup commits.

Change-Id: I18f956d999db21be197ce35044623eaa22ed4185