Bug 784865 - configure fails if both ec2-{32,64}bit-util are missing from /etc/imagefactory/imagefactory.conf
Summary: configure fails if both ec2-{32,64}bit-util are missing from /etc/imagefactor...
Keywords:
Status: CLOSED NOTABUG
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-26 14:09 UTC by Brad P. Crochet
Modified: 2012-08-29 14:54 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-01-26 17:29:42 UTC


Attachments (Terms of Use)

Description Brad P. Crochet 2012-01-26 14:09:46 UTC
Description of problem:
If both ec2-{32,64}bit-util entries are missing from /etc/imagefactory/imagefactory.conf are missing, the configure fails. If just one is there, it succeeds. Provider type does not matter.

Version-Release number of selected component (if applicable):
aeolus-configure-2.6.0-0.20120118121620gita996371.fc16.noarch

How reproducible:
Always

Steps to Reproduce:
1. Remove both 'ec2-32bit-util' and 'ec2-64bit-util' entries from /etc/imagefactory/imagefactory.conf. I also change one of the key/secret fields so I know it is/isn't doing something
2. Run 'aeolus-configure -v -d -p mock'
3. You will see the configure fail on imagefactory.conf
  
Actual results:
configure fails with:
notice: /Stage[main]/Aeolus::Image-factory/Service[imagefactory]: Dependency Augeas[imagefactory.conf] has failures: true
warning: /Stage[main]/Aeolus::Image-factory/Service[imagefactory]: Skipping because of failed dependencies


Expected results:
Successful configure

Additional info:

Comment 1 jrd 2012-01-26 15:32:05 UTC
Can we just document this?

Comment 2 Ian McLeod 2012-01-26 15:57:00 UTC
Brad,

I reviewed the code and we absolutely do not depend on either value being present in the config file.  If they are absent, we use sensible defaults.

I have no idea what aeolus-configure is doing to cause a failure in this case, but it is not a factory runtime issue.  Can you either reassign this to the component responsible for aeolus-configure or close as NOTABUG against factory?

Comment 3 Martin Kočí 2012-01-26 16:14:08 UTC
I guess the needinfo is for Brad...moving to Brad then.
Let me know if you need an assistance though.

Comment 4 wes hayutin 2012-01-26 17:23:06 UTC
so IMHO this is not a bug..
I was able to remove the values for 
  "ec2-32bit-util": "",
  "ec2-64bit-util": ""

and run successfully

I was also able to remove the values for
warehouse_key and warehouse_secret 

and aeolus-configure added new values


we can remove the keys of the config file and expect configure to put them back down.

Comment 5 wes hayutin 2012-01-26 17:28:21 UTC
grr.. key board.. should state.. "we cant expect to remove the keys of the config file"


Note You need to log in before you can comment on or make changes to this bug.