Eevans has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/404705 )
Change subject: [WIP] cassandra: create parent data directories with exec ...................................................................... [WIP] cassandra: create parent data directories with exec Bug: T175284 Change-Id: I9136f1410d07efc80f40c2a438621e43d18b8c6c --- M modules/cassandra/manifests/instance.pp 1 file changed, 7 insertions(+), 8 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/operations/puppet refs/changes/05/404705/1 diff --git a/modules/cassandra/manifests/instance.pp b/modules/cassandra/manifests/instance.pp index 0a31322..ef0a5c4 100644 --- a/modules/cassandra/manifests/instance.pp +++ b/modules/cassandra/manifests/instance.pp @@ -356,14 +356,13 @@ require => Package['cassandra'], } - file { [$instance_data_file_directories, - $commitlog_directory, - $saved_caches_directory]: - ensure => directory, - owner => 'cassandra', - group => 'cassandra', - mode => '0750', - require => File[$data_directory_base], + # (instance_)data_file_directories is an array of arbitrary, fully-qualified + # paths. Since we cannot guarantee a common base path, ensure will not work. + [$instance_data_file_directories, $commitlog_directory, $saved_caches_directory].each | $data_dir | { + exec { 'install-data-directory': + command => 'install -o cassandra -g cassandra -m 750 -d ${data_dir}', + path => '/usr/bin/:/bin/' + } } file { "${config_directory}/cassandra-env.sh": -- To view, visit https://gerrit.wikimedia.org/r/404705 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I9136f1410d07efc80f40c2a438621e43d18b8c6c Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Eevans <eev...@wikimedia.org> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits