Cisco n1kv plugin missing subnet check on network delete

Fix proposed to branch: master
Review: https://review.openstack.org/111349

Reviewed: https://review.openstack.org/111349
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=89fb8b633b1ae34d0b56d73ee115462f4acc9540
Submitter: Jenkins
Branch: master

commit 89fb8b633b1ae34d0b56d73ee115462f4acc9540
Author: Marga Millet <email address hidden>
Date: Fri Aug 1 10:41:45 2014 -0700

    Raise exception for network delete with subnets presents

    N1kv plugin should raise an exception during network delete
    if there is a subnet that is tied to that network.

    Changed the order of the Assert arguments since now it's encoraged to have <expected_value, actual_value> now.

    Change-Id: I92f177af0e2a2fbc460e0d668989ce590b7d1cb2
    Closes-Bug: 1336107

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/114429

Reviewed: https://review.openstack.org/114429
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=534bfbc05e28edfb5907a53dc5265086e1c609a4
Submitter: Jenkins
Branch: stable/icehouse

commit 534bfbc05e28edfb5907a53dc5265086e1c609a4
Author: Marga Millet <email address hidden>
Date: Fri Aug 1 10:41:45 2014 -0700

    Raise exception for network delete with subnets presents

    N1kv plugin should raise an exception during network delete
    if there is a subnet that is tied to that network.

    Also changed the order of the assert arguments since now
    it's encoraged to have <expected_value, actual_value>.

    PLEASE NOTE:
    This is a port to icehouse branch. It looks like there are extra test
    cases not relevant to the fix. However, they are need it because:
    - the code I've commited in delete_network() changes the behavior and
      it'll fail when we try to delete a network if there are subnets on it.
    - hence all the test cases which tries to delete a network and don't
      delete the subnets first, will fail.
    - Hence ALL the test cases in test_db_plugin.py file which does that
      (deleting the network without deleting the subnet first), needs to
      be overriding and this is why you see all these test cases which
      seems to not be related with the diffs in n1kv_neutron_plugin.py
      file but they are actually very related.

    Conflicts:
     neutron/tests/unit/cisco/n1kv/test_n1kv_plugin.py

    Closes-Bug: 1336107
    (cherry picked from commit 89fb8b633b1ae34d0b56d73ee115462f4acc9540)
    Change-Id: I92f177af0e2a2fbc460e0d668989ce590b7d1cb2