HOT str_replace default to empty string for param

If a str_replace param cannot be resolved or is None,
substitute an empty string in order to prevent raising
an exception during validate.

Closes Bug: #1234390
Change-Id: Ica6b84cfc2157b4e1f561f737e9aa67b9f6018ca
Add some docs on how to setup Ceilometer to use Alarms

We need to use Ceilometer's mongo backend if we want to use
alarms as the sql backend is missing a feature.

Works around bug #1223710

Change-Id: Ib569cfd650ce14480496530e7bbf1b2f71a24c02
Enabled request_id in RequestContext

As a means to enable logging on request id's, request_id in the RequestContext
can be set.

blueprint log-request-id

Change-Id: I13cd494d5af3255c03ec30e1040ef01c856c960d
resource_data saved unencrypted

Sqlalchemy is commiting unencrypted
data when a session commit happens after
doing a resource_data_get.

Closes-Bug #1233337
Change-Id: I9d20bf1faaf12cc1427976bf1223d4dfc9392c5f
Handle NetworkNotFoundClient and PortNotFoundClient

neutronclient has changed is behavior (recently?) in that it now
raises PortNotFoundClient for missing ports and NetworkNotFoundClient
for missing nets. These exceptions extend NeutronClientException but
do not set the status_code to 404 so the existing resource 404
checking has stopped working.

This change handles both 404 exceptions and the new exceptions
so that it continues to work with different neutron releases.

Change-Id: Id06e04d4b98e62b8062b119e959634fd61c8beb5
Closes-Bug: #1234924
Add contrib to tox pep8 check

This implements Clint's comment in
https://review.openstack.org/#/c/48690/1 adding # noqa
to the relative imports and adding contrib to the pep8 check.

run_tests.sh -p now returns no warnings.

Change-Id: I05085151258a6d0f3cc348211f64533fd4cbded9
Closes-Bug: #1234938
Merge "HOT str_replace default to empty string for param"
Merge "Add some docs on how to setup Ceilometer to use Alarms"
Merge "Add contrib to tox pep8 check"
Don't obtain resources through stack.resources

A particular logical resource name can be looked up by subscripting the
stack object directly.

Change-Id: I621280318cfb104ab9bfe1ae7c31fe2b5e55cfc6
Unit tests: don't use status strings from class

We should always obtain the status strings from the object we're actually
using them with. In this test, the parser.Stack class is actually stubbed
out, which can cause problems for accessing its attributes.

Change-Id: I8e060c6c937542dd11e96f243af26ee4dfa348dc
Make parser.Stack a proper Mapping

We no longer iterate over things in dependency order, so get rid of the
iterator methods that return values instead of keys. Instead, make
parser.Stack conform to the standard collections.Mapping interface.

Change-Id: Id66b2da4ba8e4335c72f1b538abfc2e97eab1e20
Merge "resource_data saved unencrypted"
It is necessary to update pool_id for Neutron LB

Closes-Bug: 1235903
Signed-off-by: Stephen Gran <stephen.gran@guardian.co.uk>
Change-Id: Ie48312efd7f04c425436b8dae9c8bccf21dc768e
allow "members" property to be absent

Closes-Bug: #1235904
Change-Id: If18f6d386d07ee65ce54f8372ad646864b486303
Signed-off-by: Stephen Gran <stephen.gran@guardian.co.uk>
Merge "It is necessary to update pool_id for Neutron LB"
Merge "Don't obtain resources through stack.resources"
Merge "Unit tests: don't use status strings from class"
Merge "Handle NetworkNotFoundClient and PortNotFoundClient"
Merge "Enabled request_id in RequestContext"