Bug 785304

Summary: Error message for an invalid install location should be more sensible
Product: [Retired] CloudForms Cloud Engine Reporter: Brad P. Crochet <brad>
Component: imagefactoryAssignee: Ian McLeod <imcleod>
Status: CLOSED CURRENTRELEASE QA Contact: Martin Kočí <mkoci>
Severity: low Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, brad, dajohnso, deltacloud-maint, dgao, hbrock, ssachdev, whayutin
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: 2012-08-30 17:12:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Brad P. Crochet 2012-01-27 22:21:08 UTC
There needs to be a better error message in the logs when then install location from a TDL is invalid.

Comment 1 wes hayutin 2012-01-28 14:45:17 UTC
Please add rpm versions and more details for this bug.
Thanks!

Comment 2 Brad P. Crochet 2012-01-30 13:59:25 UTC
RPM version is: imagefactory-1.0.0rc3_5_g1041f75-1.fc16.noarch

This was put in at the request of imcleod.

Currently, when the install location is invalid, there is a stack trace, and a generic 404 error in the imagefactory.log, but no details as to what exactly the 404 is referring to.

Comment 3 Ian McLeod 2012-02-01 19:01:44 UTC
This should not be a 1.0.0 blocker.

Comment 4 Brad P. Crochet 2012-02-02 15:45:53 UTC
Definitely should not be a 1.0.0 blocker

Comment 5 Hugh Brock 2012-02-08 16:14:33 UTC
Ian -- fine, it's not a 1.0 blocker, but should we try to fix it for 1.0 at all? We have a window to do it. Please dev-ack+ if we should try to fix it, or move to 1.1.0 if we should not.

Comment 6 wes hayutin 2012-02-22 23:45:51 UTC
moving version to 1.0.0 .  version = found in version

Comment 7 Ian McLeod 2012-03-12 16:11:13 UTC
Improved logging update is here:

https://github.com/aeolusproject/oz/commit/afbd2fc4eee4242e79768a149495da220f6ccaa3

brewed on Feb 20th as 0.8.0-5

Comment 8 Martin Kočí 2012-03-14 16:19:06 UTC
VERIFIED with packages:

#rpm -qa | grep 'aeolus\|imagefactory-\|oz-\|iwhd'
rubygem-aeolus-cli-0.3.0-14.el6.noarch
imagefactory-1.0.0rc9-1.el6.noarch
rubygem-aeolus-image-0.3.0-12.el6.noarch
aeolus-conductor-doc-0.8.0-43.el6.noarch
aeolus-conductor-0.8.0-43.el6.noarch
aeolus-configure-2.5.0-18.el6.noarch
imagefactory-jeosconf-ec2-fedora-1.0.0rc9-1.el6.noarch
iwhd-1.5-2.el6.x86_64
oz-0.8.0-5.el6.noarch
aeolus-all-0.8.0-43.el6.noarch
aeolus-conductor-daemons-0.8.0-43.el6.noarch
imagefactory-jeosconf-ec2-rhel-1.0.0rc9-1.el6.noarch

Older packages reproduced worse error message.