Bug 723934

Summary: aeolus-image needs a human readable error instead of 404 resource not found
Product: [Retired] CloudForms Cloud Engine Reporter: wes hayutin <whayutin>
Component: rubygem-aeolus-imageAssignee: Martyn Taylor <mtaylor>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.3.1CC: akarol, dajohnso, ssachdev
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description wes hayutin 2011-07-21 15:09:29 UTC 
Description of problem:

You can see from the workflow below.. when no images have been created aeolus-image is trying to tell the user 404 .. no images found.  However a more clear message for our users would be an improvement.



[root@hp-ml150g6-01 ~]# aeolus-image list --images
/usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/abstract_response.rb:48:in `return!': 404 Resource Not Found (RestClient::ResourceNotFound)
	from /usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/request.rb:220:in `process_result'
	from /usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/request.rb:169:in `transmit'
	from /usr/lib/ruby/1.8/net/http.rb:543:in `start'
	from /usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/request.rb:166:in `transmit'
	from /usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/request.rb:60:in `execute'
	from /usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/request.rb:31:in `execute'
	from /usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/lib/restclient/resource.rb:54:in `get'
	from /usr/lib/ruby/gems/1.8/gems/aeolus-image-0.0.1/lib/list_command.rb:10:in `images'
	from /usr/lib/ruby/gems/1.8/gems/aeolus-image-0.0.1/lib/config_parser.rb:188:in `send'
	from /usr/lib/ruby/gems/1.8/gems/aeolus-image-0.0.1/lib/config_parser.rb:188:in `list'
	from /usr/lib/ruby/gems/1.8/gems/aeolus-image-0.0.1/lib/config_parser.rb:30:in `send'
	from /usr/lib/ruby/gems/1.8/gems/aeolus-image-0.0.1/lib/config_parser.rb:30:in `process'
	from /usr/lib/ruby/gems/1.8/gems/aeolus-image-0.0.1/bin/aeolus-image:6
	from /usr/bin/aeolus-image:19:in `load'
	from /usr/bin/aeolus-image:19
[root@hp-ml150g6-01 ~]# aeolus-image import --provider ec2-us-east-1 --target ec2 --id  ami-5e837b37

Target Image: 7ea6d181-68bf-45c3-913b-d83e9bc3b17a
Image: a049a01d-7279-4f70-8037-9d3f4b5ef09e
Build: b096daa6-82da-4b7c-a5f8-6b4be2fd87f6
Provider Image: 5107e675-8791-4848-8cd9-0b1ad32758e2
[root@hp-ml150g6-01 ~]# aeolus-image list --images
IMAGE ID                                 LASTEST PUSHED BUILD                     NAME             TARGET     OS     OS VERSION     ARCH     DESCRIPTION     
a049a01d-7279-4f70-8037-9d3f4b5ef09e     b096daa6-82da-4b7c-a5f8-6b4be2fd87f6     ami-5e837b37     ec2
Comment 1 Martyn Taylor 2011-07-21 15:33:09 UTC 
So this happens when the relevant bucket is not yet created in iwhd.  Rather than throw an error message Im just going to return an empty list of images.  Since its not really an error.

Cheers

Martyn
Comment 2 Martyn Taylor 2011-07-21 16:56:54 UTC 
commit f475955efa9217ca48d098eaf0155f734432a87f
Author: Martyn Taylor <mtaylor>
Date:   Thu Jul 21 16:23:47 2011 +0100

    BZ#723934 Check bucket exists in iwhd
Comment 3 Aziza Karol 2011-09-22 09:37:38 UTC 
If there are no images then aeolus-image returns an empty list of images.

[root@dell-pesc440-01 nodes]# aeolus-image list --images
IMAGE ID     LASTEST PUSHED BUILD     NAME     OS     OS VERSION     ARCH     DESCRIPTION  

verified on:
[root@dell-pesc440-01 nodes]# rpm -qa | grep aeolus
aeolus-conductor-doc-0.4.0-0.20110921152740gitac47a38.fc15.noarch
aeolus-conductor-daemons-0.4.0-0.20110921152740gitac47a38.fc15.noarch
rubygem-aeolus-image-0.1.0-3.20110919115936gitd1d24b4.fc15.noarch
aeolus-configure-2.0.2-4.20110921102958gitcaa9608.fc15.noarch
aeolus-conductor-0.4.0-0.20110921152740gitac47a38.fc15.noarch
aeolus-all-0.4.0-0.20110921152740gitac47a38.fc15.noarch

This issue can be marked as verified.
Comment 5 wes hayutin 2011-12-08 13:55:07 UTC 
perm close
Comment 6 wes hayutin 2011-12-08 13:58:02 UTC 
closing out old bugs