)]}'
{"manifests/haproxy.pp":[{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"fddb7f3e0f1c65bacaae8f7abe728966325418ca","unresolved":false,"context_lines":[{"line_number":487,"context_line":"      },"},{"line_number":488,"context_line":"      public_ssl_port   \u003d\u003e $ports[keystone_admin_api_ssl_port],"},{"line_number":489,"context_line":"    }"},{"line_number":490,"context_line":"    $keystone_admin_rules \u003d {"},{"line_number":491,"context_line":"      \u0027keystone_admin_haproxy_ssl\u0027 \u003d\u003e {"},{"line_number":492,"context_line":"        \u0027dport\u0027 \u003d\u003e $ports[keystone_admin_api_ssl_port],"},{"line_number":493,"context_line":"      },"},{"line_number":494,"context_line":"    }"},{"line_number":495,"context_line":"  }"},{"line_number":496,"context_line":""},{"line_number":497,"context_line":"  if $keystone_public {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5a9d85d2_dba076a8","line":494,"range":{"start_line":490,"start_character":0,"end_line":494,"end_character":5},"updated":"2016-06-20 04:45:47.000000000","message":"shouldn\u0027t we need to enable the non-ssl ports as well?","commit_id":"280bdfc01521c3751cc3f9beaa1e9ccf7425f70e"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"0cfad90fd4f40902bf3c9f727bf9913f843a28a1","unresolved":false,"context_lines":[{"line_number":487,"context_line":"      },"},{"line_number":488,"context_line":"      public_ssl_port   \u003d\u003e $ports[keystone_admin_api_ssl_port],"},{"line_number":489,"context_line":"    }"},{"line_number":490,"context_line":"    $keystone_admin_rules \u003d {"},{"line_number":491,"context_line":"      \u0027keystone_admin_haproxy_ssl\u0027 \u003d\u003e {"},{"line_number":492,"context_line":"        \u0027dport\u0027 \u003d\u003e $ports[keystone_admin_api_ssl_port],"},{"line_number":493,"context_line":"      },"},{"line_number":494,"context_line":"    }"},{"line_number":495,"context_line":"  }"},{"line_number":496,"context_line":""},{"line_number":497,"context_line":"  if $keystone_public {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5a9d85d2_b2b1f5e3","line":494,"range":{"start_line":490,"start_character":0,"end_line":494,"end_character":5},"in_reply_to":"5a9d85d2_dba076a8","updated":"2016-06-21 15:52:57.000000000","message":"Done","commit_id":"280bdfc01521c3751cc3f9beaa1e9ccf7425f70e"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"7953e97f1e3d2adf8e0015fbfc38c655157a0207","unresolved":false,"context_lines":[{"line_number":487,"context_line":"      },"},{"line_number":488,"context_line":"      public_ssl_port   \u003d\u003e $ports[keystone_admin_api_ssl_port],"},{"line_number":489,"context_line":"    }"},{"line_number":490,"context_line":"    $keystone_admin_rules \u003d {"},{"line_number":491,"context_line":"      \u0027keystone_admin_haproxy_ssl\u0027 \u003d\u003e {"},{"line_number":492,"context_line":"        \u0027dport\u0027 \u003d\u003e $ports[keystone_admin_api_ssl_port],"},{"line_number":493,"context_line":"      },"},{"line_number":494,"context_line":"    }"},{"line_number":495,"context_line":"  }"},{"line_number":496,"context_line":""},{"line_number":497,"context_line":"  if $keystone_public {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5a9d85d2_4260f6ce","line":494,"range":{"start_line":490,"start_character":0,"end_line":494,"end_character":5},"in_reply_to":"5a9d85d2_dba076a8","updated":"2016-06-20 17:57:18.000000000","message":"right, I missed something here, nice catch","commit_id":"280bdfc01521c3751cc3f9beaa1e9ccf7425f70e"}],"manifests/profile/base/keystone.pp":[{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"9e7aabe5a28b16b7e9545172b7c651725be7d2c0","unresolved":false,"context_lines":[{"line_number":52,"context_line":"  $step             \u003d hiera(\u0027step\u0027),"},{"line_number":53,"context_line":") {"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"  if hiera(\u0027manage_firewall\u0027, false) {"},{"line_number":56,"context_line":"    include ::tripleo::firewall"},{"line_number":57,"context_line":"    $firewall_rules \u003d {"},{"line_number":58,"context_line":"      \u0027keystone_public\u0027 \u003d\u003e {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5a9d85d2_ce9fed01","line":55,"updated":"2016-06-20 21:13:25.000000000","message":"Nit: Would be nice to just default these to true while we\u0027re making changes.","commit_id":"280bdfc01521c3751cc3f9beaa1e9ccf7425f70e"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"0cfad90fd4f40902bf3c9f727bf9913f843a28a1","unresolved":false,"context_lines":[{"line_number":52,"context_line":"  $step             \u003d hiera(\u0027step\u0027),"},{"line_number":53,"context_line":") {"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"  if hiera(\u0027manage_firewall\u0027, false) {"},{"line_number":56,"context_line":"    include ::tripleo::firewall"},{"line_number":57,"context_line":"    $firewall_rules \u003d {"},{"line_number":58,"context_line":"      \u0027keystone_public\u0027 \u003d\u003e {"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"5a9d85d2_d2c64148","line":55,"in_reply_to":"5a9d85d2_ce9fed01","updated":"2016-06-21 15:52:57.000000000","message":"Done","commit_id":"280bdfc01521c3751cc3f9beaa1e9ccf7425f70e"},{"author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"change_message_id":"1e3bf37f7bfc50c27b6d383ebef485a813cfd407","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  if hiera(\u0027manage_firewall\u0027, true) {"},{"line_number":51,"context_line":"    include ::tripleo::firewall"},{"line_number":52,"context_line":"    $firewall_rules \u003d {"},{"line_number":53,"context_line":"      \u0027100 keystone_public\u0027 \u003d\u003e {"},{"line_number":54,"context_line":"        \u0027dport\u0027 \u003d\u003e [\u00275000\u0027],"},{"line_number":55,"context_line":"      },"},{"line_number":56,"context_line":"      \u0027100 keystone_admin\u0027 \u003d\u003e {"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"1aa78d24_a6568a59","line":53,"updated":"2016-07-06 18:53:07.000000000","message":"Rather","commit_id":"77076c8d218d1669f8aba97c611c09908fef9b0b"}]}