some tests assume existance of some agents

[YAMAMOTO Takashi]

following tests seem to have an assumption some of neutron agents are registered.

test_add_remove_network_from_dhcp_agent
test_list_networks_hosted_by_one_dhcp
AgentManagementTestJSON

unfortunately the assumption is false for the latest version of midonet,
where agent/dhcp_agent_scheduler/lbaas_agent_scheduler extensions are
enabled but in typical setup no neutron agents are run.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[Armando Migliaccio (armando-migliaccio)]

I am a bit baffled by this bug report. You can't implement an extension and expect to honor it the way you see fit, that's a kick in the teeth of portability.

[Armando Migliaccio (armando-migliaccio)]

"some tests assume existence of some agents" ==> Yes, that's how it's supposed to work. It's not an oversight or a design flaw.

[YAMAMOTO Takashi (yamamoto)]

Armando,

i'm not sure if i understand.
to me, it seems wrong to have this level of implementation knowledge in api tests.

with the reference implementation, surely "no agents" is likely due to some regression.
but my understanding is that it's the job of scenario tests to ensure the functionalities and
detect such a regression. not api tests.

in case of midonet, these agents are optional.
ie. it can be configured to work with or without these agents.
either ways it's fully functional.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Ken'ichi Ohmichi (<email address hidden>) on branch: master
Review: https://review.openstack.org/239328
Reason: As http://lists.openstack.org/pipermail/openstack-dev/2016-May/096204.html , we start abandoning patches have gotten old without any updating after negative feedback. Please restore if necessary to restart this again.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/239329