Installing Foreman: Class[Foreman::Install::Repos] is already declared

Hi all,

I've executed:
echo include puppet, puppet::server, foreman, foreman_proxy | puppet apply
–modulepath /etc/puppet/modules/common/

in order to install puppet. This fails with an error of:

Duplicate declaration: Class[Foreman::Install::Repos] is already declared
in file /etc/puppet/modules/common/foreman/manifests/install.pp at line 3;
cannot redeclare at
/etc/puppet/modules/common/foreman_proxy/manifests/install.pp:5 on node
foreman.zhaw.ch

Any ideas?

Andy

FYI: From IRC, a patch is progress

··· On Monday, June 25, 2012 1:11:14 PM UTC+2, Andy Edmonds wrote: > > Hi all, > > I've executed: > echo include puppet, puppet::server, foreman, foreman_proxy | puppet apply > --modulepath /etc/puppet/modules/common/ > > in order to install puppet. This fails with an error of: > > Duplicate declaration: Class[Foreman::Install::Repos] is already declared > in file /etc/puppet/modules/common/foreman/manifests/install.pp at line 3; > cannot redeclare at > /etc/puppet/modules/common/foreman_proxy/manifests/install.pp:5 on node > foreman.zhaw.ch > > Any ideas? > > Andy >

Indeed, I have it in a branch. A workaround is to apply the foreman and
foreman proxy classes separately. Hopefully I'll publish tonight though
:wink:

Greg

··· On Mon 25 Jun 2012 12:21:51 BST, Andy Edmonds wrote: > FYI: From IRC, a patch is progress

Ok, I've pushed a patch out, but as I haven't (yet) had time to try it
on a RedHat based OS, I've pushed it to the test branch. You can get
it with:

git clone --recursive --branch test

Let me know, I'll push it to master when it's confirmed good. Oh, it
doesn't have the sudo patch, that'll be tomorrow I hope.

Cheers,
Greg

Confirmed working here.

Andy
andy.edmonds.be

··· On Tue, Jun 26, 2012 at 12:54 AM, Greg Sutcliffe wrote:

Ok, I’ve pushed a patch out, but as I haven’t (yet) had time to try it
on a RedHat based OS, I’ve pushed it to the test branch. You can get
it with:

git clone --recursive --branch test
https://github.com/theforeman/foreman-installer.git

Let me know, I’ll push it to master when it’s confirmed good. Oh, it
doesn’t have the sudo patch, that’ll be tomorrow I hope.

Cheers,
Greg


You received this message because you are subscribed to the Google Groups
"Foreman users" group.
To post to this group, send email to foreman-users@googlegroups.com.
To unsubscribe from this group, send email to
foreman-users+unsubscribe@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/foreman-users?hl=en.

Tested on Debian 6, Centos 6.2, and Ubuntu 12.04, all fine. Pushed to master.

Greg

··· On 26 June 2012 10:22, Andy Edmonds wrote: > Confirmed working here.