system-config/modules/openstack_project/manifests/tmpcleanup.pp

# == Class: openstack_project::tmpcleanup
#
class openstack_project::tmpcleanup (
) {

  if $::osfamily == 'Debian' {
    include tmpreaper
  }

  # FIXME need to implement an something on RHEL to fine tune the
  # temp directory cleanup.

}
Split out tmp clean and auto upgrades. Create abstractions for tmpcleanup and automatic_upgrades which can be used to install/include distro specific modules modules for these features. On Ubuntu this will still install unattended_upgrades and tmpreaper. On RHEL we need to implement similar functionality and modules. Change-Id: I9209610582e23c86dccb5e70691bb598dad36950 Reviewed-on: https://review.openstack.org/23197 Reviewed-by: Jeremy Stanley <fungi@yuggoth.org> Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins 2013-02-28 14:31:14 -05:00			`# == Class: openstack_project::tmpcleanup`
			`#`
			`class openstack_project::tmpcleanup (`
			`) {`

Use facter's osfamily instead of operatingsystem. The osfamily fact is more flexible when we want to support more GNU/Linux distributions while avoiding unnecessary code duplication. Change-Id: Iea4c73c19e7f94df8daccf25c764b6a4539a1ea0 Reviewed-on: https://review.openstack.org/27400 Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: Jeremy Stanley <fungi@yuggoth.org> Reviewed-by: Jeremy Stanley <fungi@yuggoth.org> Tested-by: Jenkins 2013-04-24 03:32:06 +00:00			`if $::osfamily == 'Debian' {`
Split out tmp clean and auto upgrades. Create abstractions for tmpcleanup and automatic_upgrades which can be used to install/include distro specific modules modules for these features. On Ubuntu this will still install unattended_upgrades and tmpreaper. On RHEL we need to implement similar functionality and modules. Change-Id: I9209610582e23c86dccb5e70691bb598dad36950 Reviewed-on: https://review.openstack.org/23197 Reviewed-by: Jeremy Stanley <fungi@yuggoth.org> Reviewed-by: Clark Boylan <clark.boylan@gmail.com> Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins 2013-02-28 14:31:14 -05:00			`include tmpreaper`
			`}`

			`# FIXME need to implement an something on RHEL to fine tune the`
			`# temp directory cleanup.`

			`}`