Dzahn has uploaded a new change for review.
https://gerrit.wikimedia.org/r/269886
Change subject: Revert "cassandra: fix top-scope vars without namespaces"
......................................................................
Revert "cassandra: fix top-scope vars without namespaces"
This reverts commit c5dedb643886f455c7b056cb1c1021ad16eb8a59.
Change-Id: I6fdb30d2141424ed499d400f363c7320b6f9421b
---
M modules/cassandra/manifests/init.pp
M modules/cassandra/manifests/instance.pp
2 files changed, 8 insertions(+), 7 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/86/269886/1
diff --git a/modules/cassandra/manifests/init.pp
b/modules/cassandra/manifests/init.pp
index 03b1b0a..17d4a46 100644
--- a/modules/cassandra/manifests/init.pp
+++ b/modules/cassandra/manifests/init.pp
@@ -274,9 +274,9 @@
$application_password = undef,
$auto_bootstrap = true,
- $yaml_template = "${::module}/cassandra.yaml.erb",
- $env_template = "${::module}/cassandra-env.sh.erb",
- $rackdc_template =
"${::module}/cassandra-rackdc.properties.erb",
+ $yaml_template = "${module}/cassandra.yaml.erb",
+ $env_template = "${module}/cassandra-env.sh.erb",
+ $rackdc_template =
"${module}/cassandra-rackdc.properties.erb",
$logstash_host = 'logstash1003.eqiad.wmnet',
$logstash_port = 11514,
@@ -286,6 +286,7 @@
validate_absolute_path($commitlog_directory)
validate_absolute_path($saved_caches_directory)
+ validate_string($initial_token)
validate_string($endpoint_snitch)
validate_re($rpc_server_type, '^(hsha|sync|async)$')
diff --git a/modules/cassandra/manifests/instance.pp
b/modules/cassandra/manifests/instance.pp
index b25cece..6dd6119 100644
--- a/modules/cassandra/manifests/instance.pp
+++ b/modules/cassandra/manifests/instance.pp
@@ -59,9 +59,9 @@
$data_directory_base = '/var/lib/cassandra'
$config_directory = '/etc/cassandra'
$service_name = 'cassandra'
- $tls_hostname = $::hostname
+ $tls_hostname = $hostname
$pid_file = '/var/run/cassandra/cassandra.pid'
- $instance_id = $::hostname
+ $instance_id = $hostname
$data_file_directories = $this_instance['data_file_directories']
$commitlog_directory = $this_instance['commitlog_directory']
$heapdump_directory = $this_instance['heapdump_directory']
@@ -70,9 +70,9 @@
$data_directory_base = "/srv/cassandra-${instance_name}"
$config_directory = "/etc/cassandra-${instance_name}"
$service_name = "cassandra-${instance_name}"
- $tls_hostname = "${::hostname}-${::instance_name}"
+ $tls_hostname = "${hostname}-${instance_name}"
$pid_file =
"/var/run/cassandra/cassandra-${instance_name}.pid"
- $instance_id = "${::hostname}-${::instance_name}"
+ $instance_id = "${hostname}-${instance_name}"
$data_file_directories = ["${data_directory_base}/data"]
$commitlog_directory = "${data_directory_base}/commitlog"
$heapdump_directory = $data_directory_base
--
To view, visit https://gerrit.wikimedia.org/r/269886
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6fdb30d2141424ed499d400f363c7320b6f9421b
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
