Install objgraph package

The objgraph library is used to grab object counts for memory leak
detection. We already have this installed on zuul01, but that appears
to have been done manually. This cements the install.

Since puppet-nodepool requires the same packages (yappi and objgraph),
wrap both of the installs in a conditional check to prevent puppet
errors about being defined in multiple places.

Change-Id: I63ab5a21f7b98341bb203883f40c2df5404687ba
This commit is contained in:
David Shrewsbury 2019-04-22 14:16:35 -04:00
parent eba82d5a99
commit 8d7153dbb3
2 changed files with 16 additions and 4 deletions

View File

@ -132,10 +132,20 @@ class zuul (
}
}
package { 'yappi':
ensure => present,
provider => $pip_provider,
require => Class['pip'],
if ! defined(Package['yappi']) {
package { 'yappi':
ensure => present,
provider => $pip_provider,
require => Class['pip'],
}
}
if ! defined(Package['objgraph']) {
package { 'objgraph':
ensure => present,
provider => $pip_provider,
require => Class['pip'],
}
}
if ! $zuulv3 {
@ -204,6 +214,7 @@ class zuul (
Package['build-essential'],
Package['libffi-dev'],
Package['libssl-dev'],
Package['objgraph'],
Package['yappi'],
],
}

View File

@ -77,6 +77,7 @@ describe 'puppet-zuul module', :if => ['debian', 'ubuntu'].include?(os[:family])
describe 'required pip packages' do
packages = [
package('objgraph'),
package('yappi'),
package('zuul')
]