)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"448ea23d3fdb9f2c54570553fb11fbc9fc0dc76d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Lee Yarwood \u003clyarwood@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-03-31 14:04:11 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"encryptors: Add and default to using missing noop constant"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Ica8bb121192e737acebf766876ef0d99621d6f42"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bff0334d_2d63e8de","line":7,"updated":"2017-04-06 00:34:20.000000000","message":"can we get a better description of what this patch is about please?","commit_id":"488d148b72e8368dc00fa1e17f43bf6af242d160"}],"os_brick/encryptors/__init__.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"5d409cb1c89a1538d7c91070d3f64db7d4bf76ca","unresolved":false,"context_lines":[{"line_number":25,"context_line":"FORMAT_TO_FRONTEND_ENCRYPTOR_MAP \u003d {"},{"line_number":26,"context_line":"    LUKS: \u0027os_brick.encryptors.luks.LuksEncryptor\u0027,"},{"line_number":27,"context_line":"    PLAIN: \u0027os_brick.encryptors.cryptsetup.CryptsetupEncryptor\u0027,"},{"line_number":28,"context_line":"    None: \u0027os_brick.encryptors.nop.NoOpEncryptor\u0027"},{"line_number":29,"context_line":"}"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"LEGACY_PROVIDER_CLASS_TO_FORMAT_MAP \u003d {"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_d842783e","line":28,"range":{"start_line":28,"start_character":4,"end_line":28,"end_character":8},"updated":"2017-04-10 15:03:35.000000000","message":"Is there a reason we are using \u0027None\u0027 here instead of NONE which would follow the convention of the other entries.","commit_id":"a781a7f79aafccf0ec2391a541f71d8e1d9a7e5d"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"d9f623398e12349eb52e116d025f7ce8de681531","unresolved":false,"context_lines":[{"line_number":25,"context_line":"FORMAT_TO_FRONTEND_ENCRYPTOR_MAP \u003d {"},{"line_number":26,"context_line":"    LUKS: \u0027os_brick.encryptors.luks.LuksEncryptor\u0027,"},{"line_number":27,"context_line":"    PLAIN: \u0027os_brick.encryptors.cryptsetup.CryptsetupEncryptor\u0027,"},{"line_number":28,"context_line":"    None: \u0027os_brick.encryptors.nop.NoOpEncryptor\u0027"},{"line_number":29,"context_line":"}"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"LEGACY_PROVIDER_CLASS_TO_FORMAT_MAP \u003d {"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_f8ad3c9b","line":28,"range":{"start_line":28,"start_character":4,"end_line":28,"end_character":8},"in_reply_to":"bff0334d_d842783e","updated":"2017-04-10 15:13:51.000000000","message":"Only that the below LEGACY_PROVIDER_CLASS_TO_FORMAT_MAP maps NoOpEncryptor to None already. \n\nAllowing provider\u003dNone as a kwarg to get_volume_encryptor to also return NoOpEncryptor also feels like the right thing to do but I\u0027m likely missing something trivial.","commit_id":"a781a7f79aafccf0ec2391a541f71d8e1d9a7e5d"}]}