Ema has submitted this change and it was merged.

Change subject: etcd::client: sort keys in the config files
......................................................................


etcd::client: sort keys in the config files

So we can overcome ruby 1.8's limitations with dictionaries and order

Change-Id: Ib7349beb365b319a29d94535682b3d0d20b806b8
---
M modules/etcd/templates/client_config.erb
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Ema: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/etcd/templates/client_config.erb 
b/modules/etcd/templates/client_config.erb
index 729cee7..338c6bf 100644
--- a/modules/etcd/templates/client_config.erb
+++ b/modules/etcd/templates/client_config.erb
@@ -1,3 +1,5 @@
-<% @settings.each do |k,v| -%>
-<% if v != :undef %><%= k %>: <%= v %><% end %>
+<% @settings.keys.sort.each do |k| -%>
+<%- if @settings[k] != :undef -%>
+<%= k %>: <%= @settings[k] %>
+<%- end -%>
 <% end %>

-- 
To view, visit https://gerrit.wikimedia.org/r/263618
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib7349beb365b319a29d94535682b3d0d20b806b8
Gerrit-PatchSet: 5
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: Ema <e...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to