)]}'
{"nova/conf/crypto.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"77e060c412fd119a3cbc4d581f40be41b41a0eba","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from nova.conf import paths"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"crypto_opts_group \u003d cfg.OptGroup("},{"line_number":22,"context_line":"    \"crypto\","},{"line_number":23,"context_line":"    title\u003d\"Crypto Options\")"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"crypto_opts \u003d ["}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_2e98ce77","line":22,"updated":"2016-06-23 22:56:51.000000000","message":"It\u0027s not necessary to change \u0027 to \"... In the future, please try to refrain from these stylistic-only changes as it adds cognitive work for reviewers who need to check each line of the patch to determine what has been changed of substance.","commit_id":"43bffb0d5c0f2c65aaf4cdf299728774f7c2e05b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"66a89c00413febbf62309a1af910ba8eb7fbb894","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from nova.conf import paths"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"crypto_opts_group \u003d cfg.OptGroup("},{"line_number":22,"context_line":"    \"crypto\","},{"line_number":23,"context_line":"    title\u003d\"Crypto Options\")"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"crypto_opts \u003d ["}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_55479076","line":22,"in_reply_to":"3aaa91ec_2e98ce77","updated":"2016-06-24 10:25:25.000000000","message":"+1","commit_id":"43bffb0d5c0f2c65aaf4cdf299728774f7c2e05b"},{"author":{"_account_id":19590,"name":"Sarafraj Singh","email":"Sarafraj.Singh@intel.com","username":"sarafrajsingh"},"change_message_id":"53f380343375f0a426f56dbc88fa41ec26f37519","unresolved":false,"context_lines":[{"line_number":19,"context_line":"from nova.conf import paths"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"crypto_opts_group \u003d cfg.OptGroup("},{"line_number":22,"context_line":"    \"crypto\","},{"line_number":23,"context_line":"    title\u003d\"Crypto Options\")"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"crypto_opts \u003d ["}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_8d8ba8c2","line":22,"in_reply_to":"3aaa91ec_2e98ce77","updated":"2016-06-24 14:08:53.000000000","message":"Will do. Thanks for the review.","commit_id":"43bffb0d5c0f2c65aaf4cdf299728774f7c2e05b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"66a89c00413febbf62309a1af910ba8eb7fbb894","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"crypto_opts_group \u003d cfg.OptGroup("},{"line_number":22,"context_line":"    \"crypto\","},{"line_number":23,"context_line":"    title\u003d\"Crypto Options\")"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"crypto_opts \u003d ["},{"line_number":26,"context_line":"    cfg.StrOpt("}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_f832f3f0","line":23,"updated":"2016-06-24 10:25:25.000000000","message":"We are missing the description for the crypto options, lets add that in a follow up change.\n\nIt would be good to cover what is actually using these certificate options. I don\u0027t remember myself.","commit_id":"43bffb0d5c0f2c65aaf4cdf299728774f7c2e05b"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"66a89c00413febbf62309a1af910ba8eb7fbb894","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        deprecated_group\u003d\"DEFAULT\","},{"line_number":30,"context_line":"        help\u003d\"\"\""},{"line_number":31,"context_line":"Filename of root CA (Certificate Authority). This is a container format"},{"line_number":32,"context_line":" and includes root certificates."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"* Possible values:"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_b527d4c8","line":32,"range":{"start_line":32,"start_character":0,"end_line":32,"end_character":1},"updated":"2016-06-24 10:25:25.000000000","message":"Nit: I don\u0027t think we want this space here.","commit_id":"43bffb0d5c0f2c65aaf4cdf299728774f7c2e05b"}]}