Comment 3 for bug 1250286

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to heat (master)

Reviewed: https://review.openstack.org/55946
Committed: http://github.com/openstack/heat/commit/902154c5b1bd4222e5c0ef484a417885ae1e8991
Submitter: Jenkins
Branch: master

commit 902154c5b1bd4222e5c0ef484a417885ae1e8991
Author: Steve Baker <email address hidden>
Date: Tue Nov 12 11:28:14 2013 +1300

    Rewrite get_secgroup_uuids to avoid resource_by_refid

    get_secgroup_uuids has changed in the following ways:
    * No longer attempts to find a security group name via
      a resource_by_refid lookup. resource_by_refid only exists
      as a workaround for AWS compatibility resource which are
      not backed by an API. Usage of resource_by_refid should be
      avoided in native resources if at all possible.
    * If the security group looks like a UUID, assume it is
      a UUID for a security group that exists
    * Call client.list_security_groups only once, and only if
      there is a non-uuid security group specified
    * Do not attempt to reduce duplicate security groups to a unique
      list. If that is important for resources it can be done in
      validation.
    * Raise PhysicalResourceNotFound if name maps to no group
    * Raise PhysicalResourceNameAmbiguity if name maps to more than one
      group

    Change-Id: I106adc15a7882192884d37c0806947470d139434
    Closes-Bug: #1250286