A direct reference to a swift, s3 resource can be resolved to 'None'

[geoffroy]

If you build a custom resource 'A' that have a list of references pointing to some 'AWS::S3::Bucket', 'OS::Swift::Container' with a classic '{Ref: "foo"}' pointer, then all the references will be resolved to 'None'.

Basically, the original code of 'resource.Resource' got a protection if the function 'fnGetRefId' is called before storing the resource:
> def FnGetRefId(self):
> if self.resource_id is not None:
> return unicode(self.resource_id)
> else:
> return unicode(self.name)

But 'resources.s3.S3Bucket' and the 'resources.swift.SwiftContainer' class override this function to just
> def FnGetRefId(self):
> return unicode(self.resource_id)

Is there is a particular reason that reference on this 2 types of resources should not be directly made ?

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/41397

[Zane Bitter (zaneb)]

References only resolve to None during validation. After the resource is created, they will resolve correctly.

Arguably it's actually better to resolve to None than to the Wrong Thing during validation. Neither is a particularly good solution.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/41431

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/41397
Committed: http://github.com/openstack/heat/commit/bfddf669cc70958bbe2b4d1b5495151ac116e5e9
Submitter: Jenkins
Branch: master

commit bfddf669cc70958bbe2b4d1b5495151ac116e5e9
Author: Maksym Iarmak <email address hidden>
Date: Wed Aug 14 18:20:12 2013 +0300

    Tests for s3 and swift resources were extended

    According to the description of the bug, the problem
    is in the mechanism of dependencies. That is an internal
    function Ref is not working properly. To repeat it, check
    was added to the test templates (s3_test, swift_test). Thus,
    the bug has not been reproduced.

    Also, the method FnGetRefId in s3 and swift resources,
    despite its name, returns the resource name in the format
    'stack_name-resource_name-resource_id'. This is achieved
    by the method physical_resource_name, that implemented in
    superclass resource.Resource.

    Bug 1202987

    Change-Id: I0f85c39e10e8b455f93fdd768cd5e6ee14497277