Nevermind... under control - thanks for the fish
Craig
On Jun 24, 2011, at 9:26 AM, Craig White wrote:
>
> On Jun 23, 2011, at 6:17 PM, Nigel Kersten wrote:
>
>> On Thu, Jun 23, 2011 at 4:07 PM, Craig White <craig.wh...@ttiltd.com> wrote:
>>> class nginx::install {
>>> $prerequisites = [ "build-essential", "libcurl4-openssl-dev", "libssl-dev",
>>> "zlib1g-dev" ]
>>> case $operatingsystem {
>>> centos, redhat: {
>>> }
>>> debian, ubuntu: {
>>> package { $prerequisites : ensure => "installed" :
>>> ensure => present {
>>> exec { "Installing nginx via passenger":
>>> path => "/bin:/usr/bin",
>>> environment => "HOME=/root",
>>> command => "passenger-install-nginx-module --auto-download
>>> --auto",
>>> user => "root",
>>> group => "root",
>>> unless => "ls -l /opt | grep nginx",
>>> logoutput => on_failure,
>>> }
>>> }
>>> file {"/etc/init.d/nginx":
>>> source => "puppet:///modules/nginx/nginx-initd",
>>> owner => root,
>>> group => root,
>>> mode => 755,
>>> require => Class["nginx::install"],
>>> notify => Class["nginx::service"],
>>> }
>>> }
>>> }
>>> }
>>> }
>>>
>>> The above has a syntax error on line 7 and the catalog won't build but
>>> essentially I want to 'ensure' that the pre-requsite packages are installed
>>> prior to attempting to execute the command (because if the pre-requisites
>>> aren't there, the command will fail).
>>
>> That definitely won't compile :)
>>
>> You should be able to just declare the exec and package(s) resources
>> separately and define a relationship, like:
>>
>> exec { "Installing nginx via passenger":
>> path => "/bin:/usr/bin",
>> environment => "HOME=/root",
>> command => "passenger-install-nginx-module --auto-download --auto",
>> user => "root",
>> group => "root",
>> unless => "ls -l /opt | grep nginx",
>> logoutput => on_failure,
>> require => Package[$prerequisites],
>> }
>>
>> That last line should work for you. Otherwise you could define it the
>> opposite way around:
>>
>> package { $prerequisites:
>> ensure => installed,
>> before => Exec["Installing nginx via passenger"],
>> }
>>
>> Both those methods should work, but I have a vague memory of the first
>> method failing in a point release or two.
>>
>> What version of Puppet are you running? You may have a much clearer
>> way of expressing the conditional you're using than the somewhat messy
>> case statement you're using at the moment.
> ----
> puppet 2.6.8
>
> Your first example would necessitate creating installation class for those
> pre-requisite packages which I was hoping to just slide by without doing. In
> fact, my way that I did it sort of worked with a little tweaking but it
> seemed to try to install nginx before the pre-requisite packages so the first
> tine through, it would time out on the command but load the pre-requisite
> packages afterwards and then the second time through, it would install via
> the command which was messy.
>
> this is where I am at at the moment...
>
> class nginx::install {
> case $operatingsystem {
> centos, redhat: {
> }
> debian, ubuntu: {
> exec { "Installing nginx via passenger":
> path => "/bin:/usr/bin",
> environment => "HOME=/root",
> command => "passenger-install-nginx-module --auto-download
> --auto",
> user => "root",
> group => "root",
> unless => "ls -l /opt | grep nginx",
> logoutput => on_failure,
> require => Class["gems::passenger", "nginx::prerequisites"]
> }
> }
> }
> }
>
> and still the 'require' doesn't seem to be executed prior to evaluating the
> 'command' to exec - which still gives me issues.
>
> Craig
>
--
You received this message because you are subscribed to the Google Groups
"Puppet Users" group.
To post to this group, send email to puppet-users@googlegroups.com.
To unsubscribe from this group, send email to
puppet-users+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/puppet-users?hl=en.
