Bug 785304 - Error message for an invalid install location should be more sensible
Summary: Error message for an invalid install location should be more sensible
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: imagefactory
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: rc
Assignee: Ian McLeod
QA Contact: Martin Kočí
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-27 22:21 UTC by Brad P. Crochet
Modified: 2012-08-30 17:12 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:12:29 UTC

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

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.
Note You need to log in before you can comment on or make changes to this bug.