)]}'
{"README.md":[{"author":{"_account_id":9500,"name":"Matt Fischer","email":"matt@mattfischer.com","username":"mfisch"},"change_message_id":"ce799ca0080b93b303a2ec6b2c1ac0663f38fa4d","unresolved":false,"context_lines":[{"line_number":67,"context_line":"# Remove the admin_token_auth paste pipeline."},{"line_number":68,"context_line":"# After the first puppet run this requires setting keystone v3"},{"line_number":69,"context_line":"# admin credentials via /root/openrc or as environment variables."},{"line_number":70,"context_line":"include keystone::pipeline"},{"line_number":71,"context_line":"```"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"**Leveraging the Native Types**"}],"source_content_type":"text/x-gfm","patch_set":6,"id":"3a57f1b5_5e45e0b0","line":70,"updated":"2016-02-11 04:35:03.000000000","message":"did you mean keystone::disable_admin_token_auth?","commit_id":"068e3bf6ecf6432a9d50fe125faffa34b49aed5a"}],"manifests/disable_admin_token_auth.pp":[{"author":{"_account_id":9500,"name":"Matt Fischer","email":"matt@mattfischer.com","username":"mfisch"},"change_message_id":"ce799ca0080b93b303a2ec6b2c1ac0663f38fa4d","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    section    \u003d\u003e \u0027pipeline:public_api\u0027,"},{"line_number":27,"context_line":"    setting    \u003d\u003e \u0027pipeline\u0027,"},{"line_number":28,"context_line":"    subsetting \u003d\u003e \u0027admin_token_auth\u0027,"},{"line_number":29,"context_line":"    ensure     \u003d\u003e absent,"},{"line_number":30,"context_line":"  }"},{"line_number":31,"context_line":"  ini_subsetting { \u0027admin_api/admin_token_auth\u0027:"},{"line_number":32,"context_line":"    path       \u003d\u003e \u0027/etc/keystone/keystone-paste.ini\u0027,"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"3a57f1b5_be0c1495","line":29,"updated":"2016-02-11 04:35:03.000000000","message":"see main comment","commit_id":"068e3bf6ecf6432a9d50fe125faffa34b49aed5a"},{"author":{"_account_id":9500,"name":"Matt Fischer","email":"matt@mattfischer.com","username":"mfisch"},"change_message_id":"ce799ca0080b93b303a2ec6b2c1ac0663f38fa4d","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    section    \u003d\u003e \u0027pipeline:admin_api\u0027,"},{"line_number":34,"context_line":"    setting    \u003d\u003e \u0027pipeline\u0027,"},{"line_number":35,"context_line":"    subsetting \u003d\u003e \u0027admin_token_auth\u0027,"},{"line_number":36,"context_line":"    ensure     \u003d\u003e absent,"},{"line_number":37,"context_line":"  }"},{"line_number":38,"context_line":"  ini_subsetting { \u0027api_v3/admin_token_auth\u0027:"},{"line_number":39,"context_line":"    path       \u003d\u003e \u0027/etc/keystone/keystone-paste.ini\u0027,"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"3a57f1b5_de0f9086","line":36,"updated":"2016-02-11 04:35:03.000000000","message":"see main comment","commit_id":"068e3bf6ecf6432a9d50fe125faffa34b49aed5a"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"84447db5fb1cd0a36b767fbe41bd997247efd3c5","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"  if $::keystone::manage_service and $::keystone::enabled {"},{"line_number":19,"context_line":"    Ini_subsetting {"},{"line_number":20,"context_line":"      notify  \u003d\u003e Exec[\u0027restart_keystone\u0027],"},{"line_number":21,"context_line":"    }"},{"line_number":22,"context_line":"  }"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":8,"id":"3a57f1b5_4dabad75","line":20,"range":{"start_line":20,"start_character":13,"end_line":20,"end_character":14},"updated":"2016-02-12 10:49:03.000000000","message":"extra space :)","commit_id":"60259a616c03eac54a713cd0d53b76ce0c943691"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"84447db5fb1cd0a36b767fbe41bd997247efd3c5","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    }"},{"line_number":22,"context_line":"  }"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"  ini_subsetting { \u0027public_api/admin_token_auth\u0027:"},{"line_number":25,"context_line":"    ensure     \u003d\u003e absent,"},{"line_number":26,"context_line":"    path       \u003d\u003e \u0027/etc/keystone/keystone-paste.ini\u0027,"},{"line_number":27,"context_line":"    section    \u003d\u003e \u0027pipeline:public_api\u0027,"}],"source_content_type":"text/x-puppet","patch_set":8,"id":"3a57f1b5_ed5a99f1","line":24,"range":{"start_line":24,"start_character":2,"end_line":24,"end_character":16},"updated":"2016-02-12 10:49:03.000000000","message":"why you are not using keystone_paste_ini provider here?\n\nhttps://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_paste_ini/ini_setting.rb","commit_id":"60259a616c03eac54a713cd0d53b76ce0c943691"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"413a4b8c583e5f18b80b2cabe807235e7688c702","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    }"},{"line_number":22,"context_line":"  }"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"  ini_subsetting { \u0027public_api/admin_token_auth\u0027:"},{"line_number":25,"context_line":"    ensure     \u003d\u003e absent,"},{"line_number":26,"context_line":"    path       \u003d\u003e \u0027/etc/keystone/keystone-paste.ini\u0027,"},{"line_number":27,"context_line":"    section    \u003d\u003e \u0027pipeline:public_api\u0027,"}],"source_content_type":"text/x-puppet","patch_set":8,"id":"3a57f1b5_6601c737","line":24,"range":{"start_line":24,"start_character":2,"end_line":24,"end_character":16},"in_reply_to":"3a57f1b5_ed5a99f1","updated":"2016-02-12 21:07:39.000000000","message":"See Cody\u0027s suggestions and earlier patchsets. keystone_paste_ini inherits from ini_setting. We need ini_subsetting rather than ini_setting to get this level of granularity, to just remove one keyword from a string rather than the whole string.","commit_id":"60259a616c03eac54a713cd0d53b76ce0c943691"}],"manifests/init.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"54e39c7e3a633d863d9afac31d42df86901d5631","unresolved":false,"context_lines":[{"line_number":990,"context_line":"    }"},{"line_number":991,"context_line":"  }"},{"line_number":992,"context_line":""},{"line_number":993,"context_line":"  # and if we manage the service to be enabled."},{"line_number":994,"context_line":"  if $manage_service and $enabled {"},{"line_number":995,"context_line":"    exec { \u0027restart_keystone\u0027:"},{"line_number":996,"context_line":"      path        \u003d\u003e [\u0027/usr/sbin\u0027, \u0027/usr/bin\u0027, \u0027/sbin\u0027, \u0027/bin/\u0027],"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"7a5de9d1_e42c72ad","line":993,"updated":"2016-01-27 21:46:16.000000000","message":"the comment does not make really sense to me now :-)","commit_id":"4a421dbc5b8684ed757354d014ddfadffb62d254"}]}