Removed `#rabbit_server` method

This was a mistake on my part adding this method.  The rabbit-host
directive takes a host, not a host and port.  We are going to lock
0.3.0 version of this library Monday, and not add any minor features.
Wanted to delete this confusing method, before we lock this version
down.

Change-Id: Ibc48fc267f2b0a1ee83963f49818edbd67adaca2
Add common messaging attributes

Similar to the way in which we handle database attributes, provide a
mechanism that allows for centralized messaging configuration. Right
now this supports rabbit, but there is no reason other message queues
could not be added in the default['openstack']['mq'] namespace.

Change-Id: Id9d34fa7c256920dd575eee57c7b3820dacf8068
Merge "Add common messaging attributes"
Correct faulty Chef search query

The search returns a Hash, not a list, so we need
to remove the reference to [0] in the memcached_servers
library routine call otherwise you get:

undefined method `[]' for nil:NilClass

in chef-client runs.

Change-Id: I479aca833a2eb80fa2baf3e54809b2edaa713b89
Merge "Correct faulty Chef search query"
fix quantum endpoint path

quantumclient appends the API version to the url, which would result in
faulty urls like https://localhost:9696/v2/v2.0/...

Change-Id: I6b0b97db7c02d54b517bff106472c0a8a3fa22c6
Incorrectly mocked search results

Replaced `::Chef::Search::Query` with `search`.  Search only returns
an array of nodes.

Change-Id: I9280a066eb182f1d50bf4f641f9516fd47f7cabf
Should always return an array

I thought search would return an empty array.  Apparently it it
is not, and not a bad thing to make sure that it always does.
  http://paste.openstack.org/show/40125/

Change-Id: Ia39a092ffae69f94619ac9c640c1ffcc4d70510f
Revert "Correct faulty Chef search query"

This reverts commit 8311869e5b99fecefd567ce3f1ad1cbdf8d5c5c6.

When a role is present on the current node either the node attributes or a
subset are returned depending if section is passed.  Conversely if role is
not present on the current node search_for is called which always returns an
array.  This is inconsistant.  Since it appears the roles should contain the
attributes it doesn't matter which one is return, so the first should suffice.
Update spec tests stub to properly return an array from search_for

Change-Id: I180fe8c6a4d0e31e877b91f8cbad54018ef81049
Remove config_by_role in favor of search_for

config_by_role is only used in once place:
openstack-compute::nova-common. And, in this single place it happens
to return inconsistent results. Now that we have search_for, which
does return consistent results, move this one last config_by_role
call to use that. In the interest of good clean living, I am removing
this code.

Bumping to version 0.4.0

Change-Id: Ib755a5d568008599ee7d663f43716e7fe47251f9
endpoint scheme fix to match default nonssl keystone.conf

Change-Id: I913bfd4826d0b1b4fe9ccb035e525a6f5553bf29
Remove hardcoded localhost for mysql host param

Currently we have hardcoded the mysql database host to be localhost.
This should not be necessary if mysql is configured correctly. As
decisions about database configuration are outside the scope of this
cookbook, removing the hardcoded value.

Change-Id: I13a4ed271229793e634858fc2a41502e11f2fac5
Add quantum support in logger

Change-Id: I8339758cdc4d716e92c015fdd5e5fcb44dd74494
Merge "Remove hardcoded localhost for mysql host param"