[PATCH aeolus-configure] BZ# 768123: Configure should not write /root/.aeolus-cli

John Eckersberg jeckersb at redhat.com
Fri Dec 16 19:07:16 UTC 2011


On Thu, 15 Dec 2011 15:11:50 -0500, Jason Guiditta <jguiditt at redhat.com> wrote:
> 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.
> ---
>  recipes/aeolus/manifests/image-factory.pp |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
> 
> diff --git a/recipes/aeolus/manifests/image-factory.pp b/recipes/aeolus/manifests/image-factory.pp
> index d6e37ec..79cfd1b 100644
> --- a/recipes/aeolus/manifests/image-factory.pp
> +++ b/recipes/aeolus/manifests/image-factory.pp
> @@ -19,9 +19,6 @@ class aeolus::image-factory inherits aeolus {
>    # image factory client
>    package { 'rubygem-aeolus-image': ensure => 'installed' }
>    package { 'rubygem-aeolus-cli': ensure => 'installed' }
> -  file{"/root/.aeolus-cli":
> -    source => "/usr/lib/ruby/gems/1.8/gems/aeolus-cli-0.2.0/examples/aeolus-cli",
> -    require => Package['rubygem-aeolus-image', 'rubygem-aeolus-cli'] }
>  
>    # image factory services
>    package { 'libvirt':
> -- 
> 1.7.7.4
> 

ACK



More information about the aeolus-devel mailing list