)]}'
{"manifests/init.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"f05c823e1e3a1da5af39116ab122dadabf1fdc6a","unresolved":false,"context_lines":[{"line_number":410,"context_line":"  }"},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"  cinder_config {"},{"line_number":413,"context_line":"    \u0027oslo_concurrency/lock_path\u0027:   value \u003d\u003e  $lock_path;"},{"line_number":414,"context_line":"  }"},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"da6ed579_48e23b05","line":413,"updated":"2016-01-12 12:47:58.000000000","message":"why do you create a new cinder_config call? It\u0027s useless, you can use the previous one.","commit_id":"65233fb73a51104be16d2232bce01d8af6aad42b"},{"author":{"_account_id":16226,"name":"tobe","email":"chendihao@unitedstack.com","username":"tobe"},"change_message_id":"d2da575f2310d58cda6041d4ef657cc26d963fa2","unresolved":false,"context_lines":[{"line_number":410,"context_line":"  }"},{"line_number":411,"context_line":""},{"line_number":412,"context_line":"  cinder_config {"},{"line_number":413,"context_line":"    \u0027oslo_concurrency/lock_path\u0027:   value \u003d\u003e  $lock_path;"},{"line_number":414,"context_line":"  }"},{"line_number":415,"context_line":""},{"line_number":416,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"da6ed579_f3a0b792","line":413,"in_reply_to":"da6ed579_48e23b05","updated":"2016-01-12 16:24:32.000000000","message":"I think the configuration \"lock_path\" should be in section \"oslo_concurrency\" but not \"DEFAULT\".","commit_id":"65233fb73a51104be16d2232bce01d8af6aad42b"}]}