Rename instance_actions v3 to server_actions

* Renames the os-instance_actions v3 plugin to os-server-actions
* Also modifies api specific tests and expected user input JSON
* DocImpact only for v3 Nova api documentation for this plugin.

Closes-Bug: #1206032

DocImpact

Change-Id: Id5049e10f5ef540d0d8e3871d800e9d9cb33da73
VMWare - Check for compute node before triggering destroy

While booting an instance, if Claim on that compute node is not successful
compute manager triggers a call to destroy instance. Destroy fails since the
compute node (Cluster/ESX host) is set only after claim is successful.
Hence the check for node is made before destroy, to ensure that exception
does not get thrown. This ensures that the instance gets rescheduled
Closes-Bug: #1307408

Change-Id: Iaaf931e9c1e6cf046497e2c64952f92e802ad4be
Make instance_exists() take an instance, not instance_name

Some virt drivers may not support/need to use instance.name and have
a different way to find out if an instance exists. NOTE: this still
uses dict access on instance, as the _run_instance() path is not
converted to Instance objects yet.

NOTE: This also removes a duplicate call to self._lookup_by_name()
in libvirt's resume_state_on_host_boot(). It was calling
self.instance_exists() and then self._lookup_by_name(), but instance_exists()
calls _lookup_by_name() itself.

Change-Id: I701cb5416ec47262268d1a088018b6be2774d1c5
Correct the log in conductor unshelve_instance

The log in conductor unshelve_instance is not correct, this patch
adjust it.

Change-Id: I9706862fa39d3c9cb5af0d412368546b504bc4a5
libvirt: fix comment for get_num_instances

The comment had an invalid function name.

Change-Id: Iea10186a6cd649a42dadc631bef5f0e32fe2f479
Nova should pass device_id='' instead of None to neutron.update_port()

In the case that neutron raises an error and nova needs to undo the api
calls it's done against neutron it should pass device_id='' instead of
None otherwise neutron raises an exception.

Note: This should have a unit test but it's near impossible
to add one right without duplicating all of _stub_allocate_for_instance
due to the fact that it's using some crazy logic in the tests to make
decisions based on an index that's passed it. X.x

Change-Id: I91334c07401ff379f71fdef1d5bd080cc68a72ab
Closes-bug: #1285886
Merge "Rename instance_actions v3 to server_actions"
Merge "Correct the log in conductor unshelve_instance"
Merge "Nova should pass device_id='' instead of None to neutron.update_port()"
Merge "libvirt: fix comment for get_num_instances"
Merge "Make instance_exists() take an instance, not instance_name"
Merge "VMWare - Check for compute node before triggering destroy"
Deprecate instance_get_all_by_filters() from conductor

This deprecates the instance_get_all_by_filters() method from
conductor, which is no longer used.

This also removes a test that was mocking the conductor method,
but that really tests nothing at all. Perhaps a new test to ensure
that the period=0 case is handled properly is in order, but this
test definitely wasn't doing that.

Related to blueprint compute-manager-objects-juno

Change-Id: I1e7bc19296d1773bbc2f5f9e77a1faf0e6ac1fe6
Deprecate instance_get_active_by_window_joined() from conductor

This deprecates the instance_get_active_by_window_joined() method
from conductor which is no longer used.

Related to blueprint compute-manager-objects-juno

Change-Id: Ia9ce0cf49180806da64cc034ad12ddf2b3f1c424