Ori.livneh has submitted this change and it was merged.
Change subject: hhvm: declare /tmp/heaps in init.pp
......................................................................
hhvm: declare /tmp/heaps in init.pp
Fix-up for Ibb816474f3
Change-Id: Idac024bd45c5245bfcada1d0f424be2d4fd9289f
---
M modules/hhvm/manifests/debug.pp
M modules/hhvm/manifests/init.pp
2 files changed, 2 insertions(+), 9 deletions(-)
Approvals:
Ori.livneh: Verified; Looks good to me, approved
diff --git a/modules/hhvm/manifests/debug.pp b/modules/hhvm/manifests/debug.pp
index b1da3eb..85ed053 100644
--- a/modules/hhvm/manifests/debug.pp
+++ b/modules/hhvm/manifests/debug.pp
@@ -3,7 +3,7 @@
# Debugging symbols and tools for HHVM and related software.
#
class hhvm::debug {
- require ::hhvm
+
## Debugging symbols
package { [
@@ -127,12 +127,5 @@
owner => 'root',
group => 'root',
mode => '0555',
- }
-
- file { '/tmp/heaps':
- ensure => directory,
- owner => $::hhvm::user,
- group => $::hhvm::group,
- mode => '0775',
}
}
diff --git a/modules/hhvm/manifests/init.pp b/modules/hhvm/manifests/init.pp
index 3cdc1e8..097d435 100644
--- a/modules/hhvm/manifests/init.pp
+++ b/modules/hhvm/manifests/init.pp
@@ -270,7 +270,7 @@
before => Service['hhvm'],
}
- file { [ '/run/hhvm', '/var/cache/hhvm' ]:
+ file { [ '/run/hhvm', '/var/cache/hhvm', '/tmp/heaps' ]:
ensure => directory,
owner => $user,
group => $group,
--
To view, visit https://gerrit.wikimedia.org/r/258499
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idac024bd45c5245bfcada1d0f424be2d4fd9289f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Ori.livneh <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
