Update mongo.pp

[leopoiroux]

By default, Mongo extension is not installed in "conf.d" with PHP 5.4.
The "mongo.ini" is only in "mods-availaible"

[jippi]

Have you tested this against master without the patch ? There should be a fix for the default config directory detection there

[leopoiroux]

I tested the latest version (0.9.0).

With "PHP 5.4" all extensions are installed in "mods-available" and a symlink is made ​​in "conf.d". Only extensions in "conf.d" are loaded. It seems that "pecl install mongo" did not create this symlink :(

[leopoiroux]

For information I also found that "php::dev" should be set "require" by "php::extension::mongo"

[leopoiroux]

Have you tested my patch ?

[jippi]

If this is a bug with mongo, then chances are that other pecl extensions have the same problem - I wonder if it should be solved in a more generic way.

What i've done in my own setup is to declare something like this

define php_support::ensure_module($ensure = 'present') {

  if $ensure == 'present' {
    exec { "php5enmod -s cli $name":
      creates => "/etc/php5/cli/conf.d/20-$name.ini",
      notify  => Service['supervisor']
    }

    if defined(Service['php5-fpm']) {
      exec { "php5enmod -s fpm $name":
        creates => "/etc/php5/fpm/conf.d/20-$name.ini",
        notify  => Service['php5-fpm']
      }
    }
  }

  if $ensure == 'absent' {
    exec { "php5dismod -s cli $name":
      onlyif  => "test -e '/etc/php5/cli/conf.d/20-$name.ini'",
      notify  => Service['supervisor']
    }

    if defined(Service['php5-fpm']) {
      exec { "php5dismod -s fpm $name":
        onlyif  => "test -e '/etc/php5/fpm/conf.d/20-$name.ini'",
        notify  => Service['php5-fpm']
      }
    }
  }

}

Wonder if it can be made more generic so it can solve the mongo issue, and other pecl installs

[leopoiroux]

Oh yes !
php5enmod is more better than my solution.
Thks

[leopoiroux]

When can you push your feature ?
This exactly what I need

[Peekmo]

Hello,

I need it too for the same extension (mongo)
Thank you

[jippi]

@leopoiroux I'm super swamped with a huge project at work, so not sure when I will have time to dig into it :(

Also I'm not sure if php5dismod and friends exist on RHEL and other platform either :)

[schkovich]

None of the above is going to work case priority other than 20 is needed until bug 1273225 in php5enmod is fixed.

For example Solr extension is dependant on Json extension and has to be loaded after Json e.g. priority has to be set to 30.

[jippi]

I ended up hacking a file { content => "; amazing inline stuff" } } to work around it - which is terrible but functional

[igalic]

anyone looked at #116 recently :P

[schkovich]

Here is a proposal on how to tackle the problem in a bit more generic way. It's mixture of @jippi's idea given in this thread, @igalic's solution given in #116 and a workaround the hiera and php5(dis)enable incomparability problem described in bug #783246.

[jippi]

@schkovich can you PR it for review? :)

[schkovich]

Perhaps I might @jippi ;)

I was reluctant to open PR since it's a draft: I am not sure about naming, support for PHP < 5.4 is not there, I made a lot of commits and @igalic will ask me to rebase ;), etc. Anyway we can ditch PR anytime. :)

[schkovich]

This should be fixed in #162
refs #148

[jippi]

is this still true for 7.x ?

[schkovich]

@jippi most likely yes since those two enable/disable commands are debian specific. anyway, i will check it and revert.

[jippi]

seems like it's stalled :)