)]}'
{"manifests/resource/service_identity.pp":[{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"4352960eaef017fd4a13788554a577d26f66ddb3","unresolved":false,"context_lines":[{"line_number":159,"context_line":"  }"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"  if $configure_user_role {"},{"line_number":162,"context_line":"    keystone_user_role { \"${auth_name}@${tenant}\":"},{"line_number":163,"context_line":"      ensure \u003d\u003e \u0027present\u0027,"},{"line_number":164,"context_line":"      roles  \u003d\u003e $roles,"},{"line_number":165,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":15,"id":"ba3cc151_d3dbc07a","line":162,"updated":"2015-07-08 21:29:53.000000000","message":"Why were these changed from ensure_resource to regular resources? I\u0027m in favor of the idea but I don\u0027t see how it\u0027s relevant to the commit and it could cause deployment problems if someone was declaring these resources elsewhere.","commit_id":"5f9c2aac3c4d0737fce0a2c343c449380aac056c"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ffaadb8b8a55d3432d0162bf49fcbbecd87ea09f","unresolved":false,"context_lines":[{"line_number":159,"context_line":"  }"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"  if $configure_user_role {"},{"line_number":162,"context_line":"    keystone_user_role { \"${auth_name}@${tenant}\":"},{"line_number":163,"context_line":"      ensure \u003d\u003e \u0027present\u0027,"},{"line_number":164,"context_line":"      roles  \u003d\u003e $roles,"},{"line_number":165,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":15,"id":"ba3cc151_e87ff2dc","line":162,"in_reply_to":"ba3cc151_2532e310","updated":"2015-07-08 22:27:50.000000000","message":"And for the keystone_user, keystone_service and keystone_endpoint resources?","commit_id":"5f9c2aac3c4d0737fce0a2c343c449380aac056c"},{"author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"change_message_id":"c97bd72264b56be05666a0ac99d575ba72e2a7b4","unresolved":false,"context_lines":[{"line_number":159,"context_line":"  }"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"  if $configure_user_role {"},{"line_number":162,"context_line":"    keystone_user_role { \"${auth_name}@${tenant}\":"},{"line_number":163,"context_line":"      ensure \u003d\u003e \u0027present\u0027,"},{"line_number":164,"context_line":"      roles  \u003d\u003e $roles,"},{"line_number":165,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":15,"id":"ba3cc151_2532e310","line":162,"in_reply_to":"ba3cc151_d3dbc07a","updated":"2015-07-08 22:08:14.000000000","message":"Done","commit_id":"5f9c2aac3c4d0737fce0a2c343c449380aac056c"},{"author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"change_message_id":"232be1ad97e8df9fc8d4dd799ed5835941a16a08","unresolved":false,"context_lines":[{"line_number":159,"context_line":"  }"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"  if $configure_user_role {"},{"line_number":162,"context_line":"    keystone_user_role { \"${auth_name}@${tenant}\":"},{"line_number":163,"context_line":"      ensure \u003d\u003e \u0027present\u0027,"},{"line_number":164,"context_line":"      roles  \u003d\u003e $roles,"},{"line_number":165,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":15,"id":"ba3cc151_99147644","line":162,"in_reply_to":"ba3cc151_e87ff2dc","updated":"2015-07-08 23:25:19.000000000","message":"Done","commit_id":"5f9c2aac3c4d0737fce0a2c343c449380aac056c"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"fbf9cc517b60a965eac067550feed10481beabf4","unresolved":false,"context_lines":[{"line_number":161,"context_line":"  if $configure_user_role {"},{"line_number":162,"context_line":"    ensure_resource(\u0027keystone_user_role\u0027, \"${auth_name}@${tenant}\", {"},{"line_number":163,"context_line":"      ensure \u003d\u003e \u0027present\u0027,"},{"line_number":164,"context_line":"      roles  \u003d\u003e $roles,"},{"line_number":165,"context_line":"    })"},{"line_number":166,"context_line":"  }"},{"line_number":167,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":16,"id":"ba3cc151_c8d9cef4","line":164,"updated":"2015-07-08 22:28:01.000000000","message":"These need to be restored to their original versions as well","commit_id":"9231bbde0c8da49d371a27b33b91d381fa8192e0"},{"author":{"_account_id":9983,"name":"Richard Megginson","email":"rmeggins@redhat.com","username":"rmeggins"},"change_message_id":"a330c266f4b7f2507c5033811ffa8be5baca807b","unresolved":false,"context_lines":[{"line_number":161,"context_line":"  if $configure_user_role {"},{"line_number":162,"context_line":"    ensure_resource(\u0027keystone_user_role\u0027, \"${auth_name}@${tenant}\", {"},{"line_number":163,"context_line":"      ensure \u003d\u003e \u0027present\u0027,"},{"line_number":164,"context_line":"      roles  \u003d\u003e $roles,"},{"line_number":165,"context_line":"    })"},{"line_number":166,"context_line":"  }"},{"line_number":167,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":16,"id":"ba3cc151_59310eec","line":164,"in_reply_to":"ba3cc151_c8d9cef4","updated":"2015-07-08 23:26:21.000000000","message":"Done\n\nI\u0027m assuming you mean with the single quotes.  I did this for all of the ensure_resources","commit_id":"9231bbde0c8da49d371a27b33b91d381fa8192e0"}]}