Bug 784865

Summary: configure fails if both ec2-{32,64}bit-util are missing from /etc/imagefactory/imagefactory.conf
Product: [Retired] CloudForms Cloud Engine Reporter: Brad P. Crochet <brad>
Component: imagefactoryAssignee: Ian McLeod <imcleod>
Status: CLOSED NOTABUG QA Contact: Martin Kočí <mkoci>
Severity: low Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, brad, dajohnso, deltacloud-maint, dgao, 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-01-26 17:29:42 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-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"