Merge branch 'fix_cron_time' into 'master'

Fix cron time

This is a first attempt at fixing $cron_time. It's currently impossible to use the automatically generated timings. This would fix #4 .

See merge request !4

Merge branch 'master' into 'master'

update README to be more accurate for how the module actually works

See merge request !3

give a default value to $cron_time

This'll make it configure the cron job to a 30mins interval randomly
displaced within the hour.

It's still possible to set $cron_time to a full cron time spec to set
run times manually.

fix default value case for $cron_time

with the current code it's impossible to use the automatically generated
cron time spec since to get there you need to set $cron_time to false,
and this one in turn gets converted to an integer. This throws an error.

To fix this, we'll base the second timing to be 30mins after the first
one.

update README to be more accurate for how the module actually works

Fix storeconfig dependencies on Debian

Fix scope conflict in previous commit

Have passenger class pass along $manage_munin parameter (requires latest passenger module)

Don't include mysql::server in puppet::master:storeconfigs

Fix default value of http_compression parameter in puppet::cron

Add missing storeconfigs parameter to puppet::master class (defaults to false)

Fix /etc/default/puppetmaster on Debian when using mode => passenger

better default value for shorewall_puppetmaster parameter

Add GPLv3 license

Merge remote-tracking branch 'tails/bugfix/8379-lastruncheck-compatibility-with-puppet-3.x'

Merge branch 'merge_immerda' into 'master'

Merge immerda's changes

GENERAL CHANGES
* Now uses parameterized classes
* Deprecates all top-scope "$puppet_*" variables
* File source path changed from "site-puppet" to "site_puppet"

master/puppet.conf
* [main] default vardir changed /srv/puppet to /var/lib/puppet
* [main] plugindest removed
* [agent] logdest removed
* [environments] removed

base.pp
* new $puppet::config, $puppet::config_content

cron/base.pp
* new $puppet::cron::cron_time, $puppet::http_compression, $puppet::cron::stop_service

cron.pp
* new puppet::cron parameterized class

debian.pp
* $puppet_ensure_version now $puppet::ensure_version

init.pp
* new puppet parameteried class
* $use_shorewall deprecated

linux.pp
* $facter_ensure_version now $puppet::facter_ensure_version

master/linux.pp
* $puppetmaster_mode changed to $puppet::master::mode
* does not require apache::base

master
* class puppet:puppetmaster::* renamed puppet::master::*
* $puppet_fileserverconfig changed to $puppet::master::fileserver
* added $puppet::master::config_content
* $puppetmaster_storeconfigs changed to $puppet::master::storeconfigs

master/lastruncheck.pp
* $::puppet_lastruncheck_ignorehosts  changed to $puppet::master::lastruncheck_ignorehosts
* $::puppet_lastruncheck_ignorehosts_str  changed to $puppet::master::lastruncheck_ignorehosts_str
* $::puppet_lastruncheck_timeout_str  changed to $puppet::master::lastruncheck_timeout_str
* does not include cron
* /etc/cron.d/puppetlast command changed

master/cleanupreports
* $puppetmaster_cleanup_reports[_dir] changed to $puppet::master::reports[_dir]
* cron filename changed

master/passenger
* does not install puppetmaster-passenger package on Debian anymore
* restart mechanism by creating /etc/puppet/rack/tmp/restart.txt
* does not manage /usr/share/puppet/rack/puppetmasterd/config.ru anymore, config.ru expected at /etc/puppet/rack (check your Apache configs)

See merge request !1

Merge branch 'master' of git://git.puppet.immerda.ch/module-puppet

Conflicts:
files/master/config.ru
files/master/puppet.conf
manifests/base.pp
manifests/centos.pp
manifests/cron.pp
manifests/cron/base.pp
manifests/cron/linux.pp
manifests/debian.pp
manifests/init.pp
manifests/linux.pp
manifests/master/linux.pp
manifests/master/package/debian.pp
manifests/puppetmaster/base.pp
manifests/puppetmaster/centos.pp
manifests/puppetmaster/checklastrun.pp
manifests/puppetmaster/checklastrun/disable.pp
manifests/puppetmaster/cleanup_reports.pp
manifests/puppetmaster/cleanup_reports/disable.pp
manifests/puppetmaster/debian.pp
manifests/puppetmaster/package.pp
manifests/puppetmaster/package/base.pp
manifests/puppetmaster/passenger.pp

add hiera management file

Upgrade lastruncheck to the latest upstream version (Closes: #8379).

For a while, we've been copying that file from
https://github.com/cafuego/check_puppetmaster.git. Before this commit, we had
the version from upstream commit c121464. Since then, upstream has improved
a few things, in particular they now call puppet in a way that works with
3.x too.

Detailed --no-merges log:

89a3971c Add optional perfdata to output.
4d352938 - Use "-m1" argument to `grep` to only get the first match. - Check if yaml report exists before running grep against it. If the   certificates and reports get out of sync (say, you deleted some   reports) you'll get some "No such file or directory"
09c4c1a7 Use non-deprecated cert syntax (3.x) and make binary path and command configurable.

change char to us-ascii

linting

make tmp dir puppet writeable for cache invalidations

fix cron name globally + linting

do privilege drop properly within cron

get rid off lsb facts

linting

now all puppet version have a status

update the puppetclient puppet.conf to puppetversion > 3.5

update the puppetmaster puppet.conf to puppetversion > 3.5

change char to us-ascii

address puppet 3.x deprecation warnings

Prevent duplicate declarations of 'apt' class

Don't deploy apt_preferences snippet on master if  is undef

Fix site path in passenger class

linting

fix source if we pass config_content

make it possible to pass the content of the config from outside

make it possible to consume an external database

add cleanup job

silence the lastrun output on only successfull runs

Merge remote-tracking branch 'intrigeri/bugfix/have_puppetlast_cronjob_run'

enable a different setting to be set

use ip instead of hostname

this should be passed as a mysql hash here

so far this is only localhost

fix template

correct naming

add a simple dashboard setup

if it runs with passenger there is no such service

fix inheriting

update to latest config.ru

follow new naming convention

checklastrun cronjob: only pass output through in imperfect situations.

That is, silence it (and avoid nagging the administrators with email)
when all hosts are OK, and puppetlast exits with return code 0.

Install puppetlast cronjob under a usable name.

That is, install it as /etc/cron.d/puppetlast,
and cleanup the old buggy /etc/cron.d/puppetlast.cron.

As cron(8) says about /etc/cron.d/: "For example, any file containing dots will
be ignored."

puppet is in bin not sbin

also use new naming convention on linux

Fix typo.

Class hierarchy is off and passenger apt repository is always installed

also, the $puppet_majorversion variable was empty because it was not
fully qualified.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Merge branch 'puppet_cron'

Conflicts:
manifests/cron.pp
manifests/linux.pp

take the changes from puppet_cron

Make all linux distributions use the same path for the cron file

This simplifies the hierarchy and fixes the problem that going from
puppet::cron to puppet under debian doesn't remove the cron file.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

cron service dependancy has some errors and misses the cron class

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

added service dependency for cron

linted cron/linux.pp

removed unnesseccary stuff for absent puppetd.cron

linted linux.pp

linted checklastrun.pp

Merge branch 'master' of ssh://labs.riseup.net/shared-puppet

fix an issue if nothing got compiled

improve puppet munin plugin

correct env var

use correct location to get logs

Merge branch 'master' of ssh://code.leap.se/puppet_puppet

file extension .sh is not recognized by run-parts

modified lastruncheck to run under v2.7.x, see github.com/cafuego/check_puppetmaster

requirements to install puppetmaster with passenger

improve testing for service

implement fixes from #3514

switch to $::operatingsystem as suggested by ng in https://labs.riseup.net/code/issues/4029

Merge branch 'leap' into tmp

switch to $::operatingsystem as suggested by ng in https://labs.riseup.net/code/issues/4029

Merge branch 'master' of ssh://leap.se:4422/puppet_puppet

Conflicts:
manifests/puppetmaster/passenger.pp

Merge remote-tracking branch 'riseup/master'

Conflicts:
manifests/puppetmaster/debian.pp
manifests/puppetmaster/linux.pp
manifests/puppetmaster/package/debian.pp

forgot closing brace

debian specific config.ru location for use with puppetmaster-passenger package

Fix puppet attempting to change the puppet service from 'stopped' to running on each run:

notice: /Stage[main]/Puppet::Base/Service[puppet]/ensure: ensure changed 'stopped' to 'running'

When running in cron mode, you do not want the puppet service enabled, nor do
you want it 'running'. When enabling the cron mode, the first part of
puppet::cron::base does a Service override on the puppet service to set enable
=> false. That is a good thing, but all it does is make the service setup so it
wont be run on boot.

Unfortunately, the puppet service definition also has a 'ensure => running'
which does an /etc/init.d/puppet status and then starts the service if it is not
running. In a cron-only setup, the 'status' command results in a failure,
because the daemon is not running, and then puppet attempts to start it, which
goes nowhere because the /etc/default/puppet is configured not to start.

So, looking further at puppet::cron::base we see there is a case switch, testing
on operatingsystem and if its debian/ubuntu (or openbsd) it falls out of the
case. If it is not one of those, it continues and does a test to see if the
version of puppet is 2.6 and if so then it does an additional puppet Service
override to set ensure=> stopped which keeps the service from being checked to
see if it is running, and if it is not starting it.

So to fix this, we remove debian/ubuntu from the case, so it will continue
through and then we change the $puppet_majorversion test to look for anything
greater than or equal to '2.6', since 2.7 and later are also versions we want
this to work with.

After this change, we no longer get the attempt to restart puppet on every run.

Fix puppet attempting to change the puppet service from 'stopped' to running on each run:

notice: /Stage[main]/Puppet::Base/Service[puppet]/ensure: ensure changed 'stopped' to 'running'

When running in cron mode, you do not want the puppet service enabled, nor do
you want it 'running'. When enabling the cron mode, the first part of
puppet::cron::base does a Service override on the puppet service to set enable
=> false. That is a good thing, but all it does is make the service setup so it
wont be run on boot.

Unfortunately, the puppet service definition also has a 'ensure => running'
which does an /etc/init.d/puppet status and then starts the service if it is not
running. In a cron-only setup, the 'status' command results in a failure,
because the daemon is not running, and then puppet attempts to start it, which
goes nowhere because the /etc/default/puppet is configured not to start.

So, looking further at puppet::cron::base we see there is a case switch, testing
on operatingsystem and if its debian/ubuntu (or openbsd) it falls out of the
case. If it is not one of those, it continues and does a test to see if the
version of puppet is 2.6 and if so then it does an additional puppet Service
override to set ensure=> stopped which keeps the service from being checked to
see if it is running, and if it is not starting it.

So to fix this, we remove debian/ubuntu from the case, so it will continue
through and then we change the $puppet_majorversion test to look for anything
greater than or equal to '2.6', since 2.7 and later are also versions we want
this to work with.

After this change, we no longer get the attempt to restart puppet on every run.

fix puppetmaster service definition for passenger

Because passenger mode doesn't have its own daemon, it was a hack before to have the service definition for 'puppetmaster' be actually looking in the process list for 'apache2'. This sort of worked, but not if you need to notify the service for a restart.

It also didn't actually work, because the hasstatus parameter was set to true, which meant that on every run, puppet did a /etc/init.d/puppetmaster status and found that it was not running and then tried to start it by doing /etc/init.d/puppetmaster start. That doesn't work because its turned off in /etc/default/puppetmaster when puppetmaster_mode='passenger'.

So... this commit removes that hacky service definition and instead just requires the apache::base class, providing the apache service monitoring that is needed when you are running puppetmaster_mode='passenger'. It also has to pull up the Service[puppet] override which was adding the puppetmaster service, which makes no sense because there is no service.

Finally, in order to notify it for changes, we need to use a selector to determine how to reload things based on puppetmaster_mode.

Conflicts:

manifests/puppetmaster/linux.pp

Because passenger mode doesn't have its own daemon, it was a hack before to have the service definition for 'puppetmaster' be actually looking in the process list for 'apache2'. This sort of worked, but not if you need to notify the service for a restart.

It also didn't actually work, because the hasstatus parameter was set to true, which meant that on every run, puppet did a /etc/init.d/puppetmaster status and found that it was not running and then tried to start it by doing /etc/init.d/puppetmaster start. That doesn't work because its turned off in /etc/default/puppetmaster when puppetmaster_mode='passenger'.

So... this commit removes that hacky service definition and instead just requires the apache::base class, providing the apache service monitoring that is needed when you are running puppetmaster_mode='passenger'. It also has to pull up the Service[puppet] override which was adding the puppetmaster service, which makes no sense because there is no service.

Finally, in order to notify it for changes, we need to use a selector to determine how to reload things based on puppetmaster_mode.

Temporarily disabled Package['puppetmaster']

otherwise you now get following error:
err: Could not retrieve catalog from remote server: Error 400 on SERVER:
Could not find resource(s) Package[puppetmaster] for overriding on node
flea.leap.se

see https://leap.se/code/issues/198

update README to specify that the $puppetmaster_ensure_version only works for debian

remove no longer needed inheritance of puppet::puppetmaster::package as we are not doing any overrides, instead we can just include the classes that we also want to load

remove the complicated test case for determining if 'puppetmaster-common' is needed or not, this is handled by the apt::preferences_snippet and debian's package dependency resolution now

stop inheriting puppet::master::package::base from puppet::puppetmaster::package::debian, it is no longer needed, because we are no longer overriding anything

remove unnecessary inheritance of puppet::puppetmaster::package to puppet::puppetmaster::linux

provide a warning message if you have tried to specify $puppetmaster_ensure_version on an operatingsystem that is not supported

update config.ru to newer version as supplied by puppetlabs for passenger applications. This is likely not to work with older versions of passenger

add apt::preferences::snippet to enable specifying the puppetmaster version.

The preferences snippet requires that we set the package parameter to 'puppet*' to pull in the correct dependencies.

We set the priority to 2000 because according to apt_preferences(5):

   P > 1000 causes a version to be installed even if this constitutes a downgrade of the package

which is the desired behavior.

This should resolve issue #2928

we must fully qualify paths to execs

update puppet.conf default file to have the newer section headings, required to eliminate deprecation warnings

minor README language fix

add information to README about specifying versions

added puppetmaster service def. for passenger