)]}'
{"lib/puppet/provider/nova.rb":[{"author":{"_account_id":10540,"name":"Clayton O\u0027Neill","email":"clayton.oneill@charter.com","username":"coneill"},"change_message_id":"d0f11f4872b85f84db331e0bb6228209113de7e9","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        auth_keys.all?{|k| !conf[\u0027keystone_authtoken\u0027][k].nil?}"},{"line_number":47,"context_line":"      creds \u003d Hash[ auth_keys.map \\"},{"line_number":48,"context_line":"                   { |k| [k, conf[\u0027keystone_authtoken\u0027][k].strip] } ]"},{"line_number":49,"context_line":"      if conf[\u0027neutron\u0027] and !conf[\u0027neutron\u0027][\u0027region_name\u0027].nil?"},{"line_number":50,"context_line":"        creds[\u0027region_name\u0027] \u003d conf[\u0027neutron\u0027][\u0027region_name\u0027].strip"},{"line_number":51,"context_line":"      end"},{"line_number":52,"context_line":"      return creds"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"ba8a016a_ef22bd51","line":49,"updated":"2015-11-18 11:46:04.000000000","message":"These two clauses should match in style.  I\u0027d change it to:\n    if conf[\u0027neutron\u0027] and conf[\u0027neutron\u0027][\u0027region_name\u0027]","commit_id":"14c460a56313a6522fc2d79034e5e278e3efdc3e"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"e0da6dbc9780527ec231baa0e4864e53ffcd8667","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        auth_keys.all?{|k| !conf[\u0027keystone_authtoken\u0027][k].nil?}"},{"line_number":47,"context_line":"      creds \u003d Hash[ auth_keys.map \\"},{"line_number":48,"context_line":"                   { |k| [k, conf[\u0027keystone_authtoken\u0027][k].strip] } ]"},{"line_number":49,"context_line":"      if conf[\u0027neutron\u0027] and conf[\u0027neutron\u0027][\u0027region_name\u0027]"},{"line_number":50,"context_line":"        creds[\u0027region_name\u0027] \u003d conf[\u0027neutron\u0027][\u0027region_name\u0027].strip"},{"line_number":51,"context_line":"      end"},{"line_number":52,"context_line":"      return creds"}],"source_content_type":"text/x-ruby","patch_set":3,"id":"ba8a016a_6d920b4e","line":49,"updated":"2015-11-19 15:15:06.000000000","message":"What\u0027s about this http://docs.openstack.org/liberty/config-reference/content/networking-options-nova.html or keystone_authtoken/region_name\n\nWhy only neutron?","commit_id":"b58a02139d362e510c6916da80e28898643e4fa3"}]}