[Puppet Users] Re: Puppet class dependency

2011-10-31 Thread jcbollinger
On Oct 29, 2:50 am, Jean Baptiste FAVRE jean.baptiste.fa...@gmail.com wrote: Hello Andrew, Thanks for your answer. Infact, pkgrepo *is* already like you suggest: class pkgrepo {     Nginx::Vhost[website.domain.tld] - Class['Pkgrepo']     include pkgrepo::install,pkgrepo::config } Just

[Puppet Users] Re: Puppet class dependency

2011-10-31 Thread devon
And for classes definition, I got: class pkgrepo {          Nginx::Vhost[packages.iscoolapp.com] - Class['Pkgrepo']          include pkgrepo::install,pkgrepo::config } The way I understand it, all included classes are basically included at the same time before catalog compilation. And then

Re: [Puppet Users] Re: Puppet class dependency

2011-10-31 Thread Jean Baptiste FAVRE
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hello Devon, You're tight and that's the point I was looking for. I discovered that point when doing some tests after Andrew's contribution. So, now it works and I'll be able to deal with class ordering. Thanks for your answer, JB On 31/10/2011

Re: [Puppet Users] Re: Puppet class dependency

2011-10-31 Thread Jean Baptiste FAVRE
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hello John, Answers inline On 31/10/2011 15:40, jcbollinger wrote: Your setup is a bit strange in using arrow syntax within a class to declare a dependency of that class itself, though I'm not sure that shouldn't work. Instead, the key

[Puppet Users] Re: Puppet class dependency

2011-10-29 Thread Andrew Hendry
Hi Jean, Is Nginx::Vhost[packages.iscoolapp.com] getting set up somewhere else in your code? If not should your pkgrepo be like this? class pkgrepo { Nginx::Vhost[website.domain.tld] - Class['Pkgrepo'] include pkgrepo::install,pkgrepo::config } Or pkgrepo::repository {

Re: [Puppet Users] Re: Puppet class dependency

2011-10-29 Thread Jean Baptiste FAVRE
Hello Andrew, Thanks for your answer. Infact, pkgrepo *is* already like you suggest: class pkgrepo { Nginx::Vhost[website.domain.tld] - Class['Pkgrepo'] include pkgrepo::install,pkgrepo::config } Just forget about Nginx::Vhost[packages.iscoolapp.com] and replace it wherever you see it