)]}'
{"manifests/api.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"1a60450cf70b937189f8f3ff23e552ac776d7497","unresolved":false,"context_lines":[{"line_number":48,"context_line":"#   Defaults to \u0027127.0.0.1\u0027"},{"line_number":49,"context_line":"#"},{"line_number":50,"context_line":"# [*auth_port*]"},{"line_number":51,"context_line":"#   (optional) DEPRECATED. The port to use when authenticating against Keystone"},{"line_number":52,"context_line":"#   Defaults to 35357"},{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*auth_protocol*]"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba15a1d1_e8553f1d","line":51,"updated":"2015-09-14 14:41:56.000000000","message":"move please all deprecated parameters under # DEPRECATED PARAMETERS line","commit_id":"1b192e6ef11da4872d087f06c3f77c0b0250613c"},{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"b5e89e9721d162e329d74ac72848c9813d31ca10","unresolved":false,"context_lines":[{"line_number":48,"context_line":"#   Defaults to \u0027127.0.0.1\u0027"},{"line_number":49,"context_line":"#"},{"line_number":50,"context_line":"# [*auth_port*]"},{"line_number":51,"context_line":"#   (optional) DEPRECATED. The port to use when authenticating against Keystone"},{"line_number":52,"context_line":"#   Defaults to 35357"},{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*auth_protocol*]"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba15a1d1_e53a3dcd","line":51,"in_reply_to":"ba15a1d1_e8553f1d","updated":"2015-09-14 17:31:15.000000000","message":"good call, I\u0027ve made these changes in patchset 2.","commit_id":"1b192e6ef11da4872d087f06c3f77c0b0250613c"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"1a60450cf70b937189f8f3ff23e552ac776d7497","unresolved":false,"context_lines":[{"line_number":96,"context_line":"  $max_limit         \u003d \u00271000\u0027,"},{"line_number":97,"context_line":"  $auth_host         \u003d \u0027127.0.0.1\u0027,"},{"line_number":98,"context_line":"  $auth_port         \u003d \u002735357\u0027,"},{"line_number":99,"context_line":"  $auth_protocol     \u003d \u0027http\u0027,"},{"line_number":100,"context_line":"  $auth_uri          \u003d false,"},{"line_number":101,"context_line":"  $identity_uri      \u003d false,"},{"line_number":102,"context_line":"  $auth_admin_prefix \u003d false,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"ba15a1d1_68492f8b","line":99,"updated":"2015-09-14 14:41:56.000000000","message":"same here","commit_id":"1b192e6ef11da4872d087f06c3f77c0b0250613c"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"0282cdf88d6cc8d1f3f85db092980d2c29782159","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  if $identity_uri {"},{"line_number":163,"context_line":"    ironic_config { \u0027keystone_authtoken/identity_uri\u0027: value \u003d\u003e $identity_uri; }"},{"line_number":164,"context_line":"  } else {"},{"line_number":165,"context_line":"    ironic_config { \u0027keystone_authtoken/identity_uri\u0027: ensure \u003d\u003e absent; }"},{"line_number":166,"context_line":"  }"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"  if $auth_version {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ba15a1d1_1db2cf06","line":165,"updated":"2015-09-16 12:50:30.000000000","message":"did the \u0027\u003cSERVICE DEFAULTS\u003e\u0027 get into ironic module? If so, let\u0027s use it please.","commit_id":"490699ff66e26d4291c3460813170d69fa1556f5"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"f5c5ec785ed78e34970f7eadcfcf092ca784212f","unresolved":false,"context_lines":[{"line_number":162,"context_line":"  if $identity_uri {"},{"line_number":163,"context_line":"    ironic_config { \u0027keystone_authtoken/identity_uri\u0027: value \u003d\u003e $identity_uri; }"},{"line_number":164,"context_line":"  } else {"},{"line_number":165,"context_line":"    ironic_config { \u0027keystone_authtoken/identity_uri\u0027: ensure \u003d\u003e absent; }"},{"line_number":166,"context_line":"  }"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"  if $auth_version {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"ba15a1d1_2f0d1ae8","line":165,"in_reply_to":"ba15a1d1_1db2cf06","updated":"2015-09-16 13:06:44.000000000","message":"Bogdan, for now, we try to use  \u0027\u003cSERVICE DEFAULTS\u003e\u0027 only for Cinder, as Canary module.","commit_id":"490699ff66e26d4291c3460813170d69fa1556f5"}]}