Bug 768123

Summary: Configure should not write /root/.aeolus-cli
Product: [Retired] CloudForms Cloud Engine Reporter: Jason Guiditta <jguiditt>
Component: aeolus-configureAssignee: Jason Guiditta <jguiditt>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, deltacloud-maint, slinaber, 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: 2012-08-30 17:16:31 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
err none

Description Jason Guiditta 2011-12-15 20:08:27 UTC 
Description of problem: There are 2 parts to this problem:
1. The current way the example file to copy is located is by searching an absolute path.  This has problems both because it is not platform agnostic, and because it depends on a specific version of aeolus-cli being installed.
2. It makes no sense for the file to be written to this location, as nobody is going to be using /root as their home dir.  In the future, if we want to templatize this to substitute values, that is another matter. For now, this is plain broken/useless


Version-Release number of selected component (if applicable): current testing and master branches.


How reproducible: Run aeolus-configure, see the generated file (as long as your version depends on 0.2 of aeolus-cli, you should not get an error, otherwise, configure will fail).


Steps to Reproduce:
1. yum install aeolus-configure aeolus-all
2. sudo aeolus-configure
3.
  
Actual results: 
1. (Right version of aeolus-cli installed) File written with default values to /root/.aeolus-cli
2. (Newer version of aeolus-cli installed, like 0,3) Configure will report an error saying it couldnt find the file to copy


Expected results:

No file written, no error encountered.

Additional info:
Comment 1 Jason Guiditta 2011-12-15 20:12:17 UTC 
Sent to list for review:

commit de4f655dec7e4d12aaaa09e894141208f017b58b
Author: Jason Guiditta <jguiditt>
Date:   Thu Dec 15 15:09:09 2011 -0500

    BZ# 768123: Configure should not write /root/.aeolus-cli
    
    https://bugzilla.redhat.com/show_bug.cgi?id=768123
    
    This was written incorrectly to use a specific version of aeolus-cli,
    as well as not being needed.  If there is a need in future to have
    configure do something with this file, it should be done in a more
    robust way.
Comment 2 Jason Guiditta 2011-12-15 20:12:48 UTC 
oops, forgot to change status
Comment 3 Jason Guiditta 2011-12-16 19:53:18 UTC 
pushed to master
Comment 4 Shveta 2012-01-10 11:42:22 UTC 
Created attachment 551818 [details]
err
Comment 5 Shveta 2012-01-10 11:44:14 UTC 
we get an error message that :

"Provider was not deleted, because of errors basethere are instances for cloud account 'ssachdev': ec2_1_east/frontend "

The message needs to be further corrected.

===========================================================
rpm -qa|grep aeolus
aeolus-configure-2.6.0-0.20120105171403gitf0043e8.el6.noarch
aeolus-conductor-doc-0.9.0-0.20120105223209git5e068fe.el6.noarch
rubygem-arel-2.0.10-0.aeolus.el6.noarch
rubygem-rack-mount-0.7.1-3.aeolus.el6.noarch
rubygem-ZenTest-4.3.3-2.aeolus.el6.noarch
rubygem-aeolus-image-0.4.0-0.20120105171419git8108b42.el6.noarch
aeolus-conductor-0.9.0-0.20120105223209git5e068fe.el6.noarch
aeolus-all-0.9.0-0.20120105223209git5e068fe.el6.noarch
rubygem-aeolus-cli-0.4.0-0.20120105171410git2c6b457.el6.noarch
aeolus-conductor-daemons-0.9.0-0.20120105223209git5e068fe.el6.noarch
Comment 6 Shveta 2012-01-10 11:47:13 UTC 
Sorry the  comment 4 and 5 was meant for 
https://bugzilla.redhat.com/show_bug.cgi?id=767648

=====================================================

/root/.aeolus-cli file is still seen
Comment 7 Jason Guiditta 2012-01-10 14:33:03 UTC 
the purpose of the patch is not to remove an existing file, merely to no longer write it.  Please test on a lean install or remove the file and then run configure again.  The file should no longer be written
Comment 8 wes hayutin 2012-01-10 14:48:19 UTC 
So after running aeolus-configure.. is the user expected to fill out the $HOME/.aeolus-cli file?

Just wondering what the intent is.. 

Thanks
Comment 9 Jason Guiditta 2012-01-10 15:36:06 UTC 
The file will still be generated by the cli tool the first time you attempt to use it.  For now, it is set to a default that expects a local installation of conductor, and default username password.  As this is a file meant to be edited by a user, if they want something other than the default, they can change it.
Comment 10 John Eckersberg 2012-01-10 19:26:35 UTC 
*** Bug 769165 has been marked as a duplicate of this bug. ***
Comment 11 Steve Linabery 2012-01-10 22:03:11 UTC 
Blindly setting this to ON_QA in the belief that a fix is included in the latest aeolus packages.
Comment 12 Shveta 2012-01-11 06:23:38 UTC 
First time on installation the file is written to /root/.aeolus-cli

Removed it and configured again. The file is not re-written .

Verified in 

rpm -qa|grep aeolus
aeolus-configure-2.6.0-0.20120105171403gitf0043e8.el6.noarch
aeolus-conductor-doc-0.9.0-0.20120105223209git5e068fe.el6.noarch
rubygem-arel-2.0.10-0.aeolus.el6.noarch
rubygem-rack-mount-0.7.1-3.aeolus.el6.noarch
rubygem-ZenTest-4.3.3-2.aeolus.el6.noarch
rubygem-aeolus-image-0.4.0-0.20120105171419git8108b42.el6.noarch
aeolus-configserver-0.4.5-1.el6.noarch
aeolus-conductor-0.9.0-0.20120105223209git5e068fe.el6.noarch
aeolus-all-0.9.0-0.20120105223209git5e068fe.el6.noarch
rubygem-aeolus-cli-0.4.0-0.20120105171410git2c6b457.el6.noarch
aeolus-conductor-daemons-0.9.0-0.20120105223209git5e068fe.el6.noarch