[PATCH aeolus-image-rubygem] Bug 796783: fixes lingering errors w/ image import

Richard Su rwsu at redhat.com
Fri Feb 24 00:32:05 UTC 2012


On 02/23/2012 03:10 PM, Scott Seago wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=796783
> ---
>   lib/aeolus_image/import.rb |    5 +++--
>   1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/lib/aeolus_image/import.rb b/lib/aeolus_image/import.rb
> index fe0ce22..2e5a9d2 100644
> --- a/lib/aeolus_image/import.rb
> +++ b/lib/aeolus_image/import.rb
> @@ -14,7 +14,7 @@
>
>   module Aeolus
>     module Image
> -    def self.import(provider_name, deltacloud_driver, image_id, account_id, xml=nil)
> +    def self.import(provider_name, deltacloud_driver, image_id, account_id, environment, xml=nil)
>         xml ||= "<image><name>#{image_id}</name></image>"
>         image = Factory::Image.new(
>           :target_name =>  deltacloud_driver,
> @@ -25,7 +25,8 @@ module Aeolus
>         image.save!
>         # Set the provider_account_id on the image
>         iwhd_image = Warehouse::Image.find(image.id)
> -      # Set the account on the provider image
> +      iwhd_image.set_attr("environment", environment)
> +     # Set the account on the provider image
>         # This assumes (as is currently correct) that there will only be one provider image for imported images
>         pimg = iwhd_image.provider_images.first
>         pimg.set_attr('provider_account_identifier', account_id)

Ack and pushed.



More information about the aeolus-devel mailing list