Bug 768123 - Configure should not write /root/.aeolus-cli
Summary: Configure should not write /root/.aeolus-cli
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-configure
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Jason Guiditta
QA Contact: wes hayutin
URL:
Whiteboard:
: 769165 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-15 20:08 UTC by Jason Guiditta
Modified: 2012-08-30 17:16 UTC (History)
4 users (show)

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


Attachments (Terms of Use)
err (232.33 KB, image/png)
2012-01-10 11:42 UTC, Shveta
no flags Details

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


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