Bug 798548

Summary: environment param is not passed when image import fails
Product: [Retired] CloudForms Cloud Engine Reporter: Jan Provaznik <jprovazn>
Component: aeolus-conductorAssignee: Jan Provaznik <jprovazn>
Status: CLOSED ERRATA QA Contact: pushpesh sharma <psharma>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, bbandari, deltacloud-maint, morazi, psharma, ssachdev, sseago
Target Milestone: beta4   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-05-15 22:47:53 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
verified_screenshot none

Description Jan Provaznik 2012-02-29 08:52:53 UTC 
1. go to image import page
2. fill in some name and image id which doesn't exist
3. press "continue"

a user is redirected to images index page with flash message "The record you tried to access does not exist, it may have been deleted"

expected:
a user is redirected back to image import page with message " The requested image was not found on the provider."

(Environment param should be passed as a param to redirect_to in rescue block of import method.)
Comment 1 Scott Seago 2012-02-29 18:50:30 UTC 
This has been fixed already -- I can't recall the bz number though, but it's in beta3. Can you re-confirm this with aeolus-image-rubygem and conductor on master?
Comment 2 Jan Provaznik 2012-03-02 17:55:27 UTC 
Yes, bug is still present, a patch sent: https://fedorahosted.org/pipermail/aeolus-devel/2012-March/009362.html
Comment 3 Jan Provaznik 2012-03-06 15:26:01 UTC 
pushed, commit d3eda5e7ecc24eba75115724a357ee9a6cb5fffa
Comment 5 pushpesh sharma 2012-04-17 07:28:44 UTC 
Marking this as verified.while attempting to import a non-existent ami no from ec2 account fails and displays a proper error message. PFA for detailed screenshot.    


[root@dhcp201-169 ~]# rpm -qa|grep aeolus
aeolus-configure-2.5.2-1.el6.noarch
aeolus-conductor-0.8.7-1.el6.noarch
rubygem-aeolus-cli-0.3.1-1.el6.noarch
aeolus-conductor-doc-0.8.7-1.el6.noarch
aeolus-all-0.8.7-1.el6.noarch
aeolus-conductor-daemons-0.8.7-1.el6.noarch
rubygem-aeolus-image-0.3.0-12.el6.noarch
Comment 6 pushpesh sharma 2012-04-17 07:29:34 UTC 
Created attachment 577927 [details]
verified_screenshot
Comment 7 errata-xmlrpc 2012-05-15 22:47:53 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-0583.html