Replace NvpPluginV2 with NsxPlugin

The king is dead, long live the king!

Partial-implements blueprint: nicira-plugin-renaming

Change-Id: I9b71479a8d4228d45a6591b169c489c0107fb04c
clean.sh removes all the files

clean.sh was incorrectly looping over the list of files to remove. Because of
this the files were not being removed.

Change-Id: Ie0559e1d396a4d35df6a12dfbceefa7eb261bac5
Closes-Bug:1285924
Move some comments of variables to right place

setup_develop*() in functions has been moved to functions-common. But
some comments about the variables are still left.
This commit moves it to the right place.

Change-Id: Ic360454f1ee72f51c9979d0468dee0913e9b32e4
Source lib/neutron in boot_from_volume.sh

Without lib/neutron, boot_from_volume.sh generates the following
error:

  + _ping_check_neutron private 10.11.12.5 30
  /devstack/functions: line 356: _ping_check_neutron: command not found

Change-Id: Ib72c3f24d614570d69bf5dda35cbaf5847b1d1b9
Devstack install can fail on missing xinetd.d/rsync config

Assuming if the system does not have the xinetd.d/rsync,
the dedicated service is the prefered way.

Change-Id: Ic42651c5c3fb5bf0099786ca81a7bd06ace896a8
nova changes for multinode working under -o errexit

There was a stray inicomment on paste outside of a nova-api block. This
fails under -o errexit because the paste.ini doesn't exist.

Move this to inside the correct block.

Change-Id: Iffbdae6716a1c2a8f650b68edd4faf436434eab1
make ceilometer work if you don't enable ceilometer-api

when doing ceilometer in a multihost devstack, you don't want
ceilometer-api running on the computes. Under -o errexit this
became fatal.

Change-Id: Ie43c8724ba467b810f5a3b075dea45d66dde8648
only do a dbsync if on the database node

ceilometer should only try to reset the database if it's actually
on a node where there is a database.

Change-Id: Ibcfec0556829bff0938e3769c19d34ae6c02b738
Use glance image-show to check for uploaded Docker images

The behavior of the code being replaced was failing
with '-o errexit' should that, as in the common case,
the image has not been uploaded into Glance.

While we could workaround this using a '|| :', the existing
code also happened to overwrite the DOCKER_IMAGE global
which is used elsewhere. It seemed prudent to either change
this variable name or remove it altogether.

Finally, using 'glance image-show' is more deterministic
than grepping the output of 'glance image-list'.

Change-Id: I23188155966ae9db64259b4a9d25a0d98c63c912
Closes-Bug: 1286443
Merge "Devstack install can fail on missing xinetd.d/rsync config"
Merge "clean.sh removes all the files"
Merge "Source lib/neutron in boot_from_volume.sh"
Merge "nova changes for multinode working under -o errexit"
Merge "Move some comments of variables to right place"
Merge "Use glance image-show to check for uploaded Docker images"
Merge "make ceilometer work if you don't enable ceilometer-api"
Merge "Replace NvpPluginV2 with NsxPlugin"
Merge "only do a dbsync if on the database node"
Merge branch 'master' of https://github.com/openstack-dev/devstack