Giuseppe Lavagetto has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/350833 )
Change subject: scap::dsh::group: fix regression in conftool template
......................................................................
scap::dsh::group: fix regression in conftool template
I removed it by error in a preceding commit, but the "/" in the key path
was needed.
Change-Id: I34019d055ebe7ebc82a900e700593b56ce4f2867
---
M modules/scap/templates/dsh/dsh-group-conftool.tpl.erb
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Giuseppe Lavagetto: Verified; Looks good to me, approved
diff --git a/modules/scap/templates/dsh/dsh-group-conftool.tpl.erb
b/modules/scap/templates/dsh/dsh-group-conftool.tpl.erb
index 96a2259..b4aa72f 100644
--- a/modules/scap/templates/dsh/dsh-group-conftool.tpl.erb
+++ b/modules/scap/templates/dsh/dsh-group-conftool.tpl.erb
@@ -9,6 +9,6 @@
# List of hosts gathered from etcd
<%- @keys.each do |key| -%>
# etcd pool: <%= key %>
-{{range $node := ls "<%= key %>/"}}{{ $key := printf "<%= key %>%s" $node }}{{
$data := json (getv $key) }}
+{{range $node := ls "<%= key %>/"}}{{ $key := printf "<%= key %>/%s" $node
}}{{ $data := json (getv $key) }}
{{ if ne $data.pooled "inactive" }}{{ $node }}{{end}}{{end}}
<% end -%>
--
To view, visit https://gerrit.wikimedia.org/r/350833
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I34019d055ebe7ebc82a900e700593b56ce4f2867
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
