1. 20 Nov, 2020 5 commits
  2. 02 Jul, 2020 1 commit
  3. 20 Jan, 2016 5 commits
  4. 10 Nov, 2015 3 commits
  5. 09 Oct, 2015 2 commits
    • micah's avatar
      Merge branch 'UpdateApacheConf' into 'master' · f215de3d
      micah authored
      update the apache2.conf file
      
      updating to the debian 3.5.1.dfsg-2+b1 version of the file. mostly
      whitespace changes, one 2.2 vs 2.4 change and a change for docs.
      
      See merge request !14
      f215de3d
    • micah's avatar
      Merge branch 'check_http_sni_support' into 'master' · 71f8db0a
      micah authored
      Add --sni to check_http --ssl commands
      
      Makes sure correct SSL certificate is selected on hosts which use SNI for SSL, and also avoids some sites returning a `HTTP CRITICAL - Error on receive` error. Does not seem to affect non-SNI hosts.
      
      See merge request !13
      71f8db0a
  6. 08 Oct, 2015 1 commit
    • taggart's avatar
      update the apache2.conf file · 7cb5f062
      taggart authored
      updating to the debian 3.5.1.dfsg-2+b1 version of the file. mostly
      whitespace changes, one 2.2 vs 2.4 change and a change for docs.
      7cb5f062
  7. 07 Oct, 2015 1 commit
  8. 06 Oct, 2015 1 commit
    • taggart's avatar
      Merge remote-tracking branch 'shared/master' into OctMerge · 6eedb1bc
      taggart authored
      merging with shared
      
      Conflicts:
      	README
      	README.pnp4nagios
      	files/pnp4nagios/apache.conf
      	manifests/defaults/commands.pp
      	manifests/init.pp
      	manifests/pnp4nagios.pp
      	manifests/pnp4nagios/popup.pp
      	manifests/service.pp
      	templates/nrpe/nrpe_command.erb
      6eedb1bc
  9. 05 Oct, 2015 1 commit
    • micah's avatar
      Merge branch 'fix_apache' into 'master' · 2cd1b529
      micah authored
      Fix apache
      
      the main class is currently unusable with apache, which is the default case.
      
      we need to move management of files necessary for nagios+apache+debian into the "nagios" class to fix this.
      
      This partly responds to #4
      
      See merge request !12
      2cd1b529
  10. 02 Oct, 2015 4 commits
    • LeLutin's avatar
      lint init.pp · 93a00004
      LeLutin authored
      one line is too long, break the code block into multiple lines to fix
      this issue. it also creates more breathing space and makes the code
      easier to read.
      93a00004
    • LeLutin's avatar
      files managed for apache on debian break usage through main class · 2ee46c9a
      LeLutin authored
      Currently the nagios::apache class manages some files for debian. This
      means that when using the main class for setting up nagios with apache
      (which is the default!) lacks those files and the setup doesn't work.
      
      Split out management of those files into another class to avoid making
      the code in the main class too encumbered and make the main class
      include that (instead of nagios::apache). nagios::apache will include
      the main class anyway.
      
      Adjust README file so that all cases of http management are done through
      the "nagios" class.
      2ee46c9a
    • taggart's avatar
      Merge branch 'non_ascii_char' into 'master' · 63b80c9c
      taggart authored
      remove non-ascii char in comment
      
      puppet 3.x refuses to run when there are non-ASCII chars in comments
      (wth!). so replace that char with an ASCII representation of it.
      
      See merge request !11
      63b80c9c
    • LeLutin's avatar
      remove non-ascii char in comment · 395c17f2
      LeLutin authored
      puppet 3.x refuses to run when there are non-ASCII chars in comments
      (wth!). so replace that char with an ASCII representation of it.
      395c17f2
  11. 07 May, 2015 7 commits
    • micah's avatar
      Merge branch 'fix_ordering' into 'master' · 1fd73c0f
      micah authored
      require package before config files
      
      In the current state of the module, almost 100% of the time nagios
      doesn't install correctly since ordering of the resources is not
      enforced.
      
      This is a major issue since the module is currently not usable in its current state because of the problem it creates.
      
      See merge request !5
      1fd73c0f
    • LeLutin's avatar
      require package before config files · 53c19e4a
      LeLutin authored
      In the current state of the module, almost 100% of the time nagios
      doesn't install correctly since ordering of the resources is not
      enforced.
      53c19e4a
    • micah's avatar
      Merge branch 'no_alias_for_host' into 'master' · 18e63a63
      micah authored
      stop defining an alias by default for hosts
      
      Since the "alias" attribute is special for puppet in that it needs to be
      a string that's unique throughout all of the resources on a given host's
      catalogue, we shouldn't always define an alias unless users really want
      them.
      
      This alias might (and did for some ppl) create a conflict when using a
      class that is named the same as $::hostname.
      
      We've just hit hit limitation on a Koumbit node. The alias attribute is legitimate in terms of nagios host attributes, but the name of the attribute clashes with the special-purpose attribute for puppet. So one might argue that the native puppet types are flawed, and they would be correct! but getting those types fixed would be a pain in the ass since they haven't been maintained for quite some time.
      
      See merge request !10
      18e63a63
    • micah's avatar
      Merge branch 'nrpe_command_template' into 'master' · 40f247d6
      micah authored
      added templates/nrpe/nrpe_command.erb
      
      commit:7fb80fbe uses nagios::nrpe::command { "check_load": ...}, but the
      template for it is missing.
      
      amended by LeLutin: use @ prefix on variables in template
      
      This is a followup of: https://labs.riseup.net/code/issues/4100
      
      See merge request !8
      40f247d6
    • micah's avatar
      Merge branch 'plugin_path' into 'master' · 9b8f961d
      micah authored
      Plugin path
      
      Avoid using lib64 in the path for x86_64 architecture when using debian. this path is not available anymore starting with debian wheezy.
      Then lint the file to make it easier to read / maintain.
      
      This is a followup of https://labs.riseup.net/code/issues/4237
      
      See merge request !7
      9b8f961d
    • micah's avatar
      Merge branch 'ordering_in_example' into 'master' · 12a297b7
      micah authored
      Ordering in example
      
      The usage example in the readme has a couple of small and bigger problems. This branch makes the example more accurate.
      
      See merge request !6
      12a297b7
    • micah's avatar
      Merge branch 'manage_resource.cfg' into 'master' · 2bbe7aae
      micah authored
      Only use the private directory on CentOS
      
      Debian and ubuntu have their resource.cfg file placed in /etc/nagios3/
      and when it's absent it can cause issues with the package (because the
      package is badly done).
      
      This is a followup to issue https://labs.riseup.net/code/issues/4240
      
      See merge request !4
      2bbe7aae
  12. 23 Apr, 2015 1 commit
    • LeLutin's avatar
      stop defining an alias by default for hosts · 29c61a62
      LeLutin authored
      Since the "alias" attribute is special for puppet in that it needs to be
      a string that's unique throughout all of the resources on a given host's
      catalogue, we shouldn't always define an alias unless users really want
      them.
      
      This alias might (and did for some ppl) create a conflict when using a
      class that is named the same as $::hostname.
      29c61a62
  13. 19 Apr, 2015 8 commits