)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sven mark \u003cmasw@awcloud.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-14 16:50:47 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_684ae6ad","line":7,"range":{"start_line":7,"start_character":52,"end_line":7,"end_character":53},"updated":"2017-02-14 10:48:55.000000000","message":"extra space","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sven mark \u003cmasw@awcloud.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-14 16:50:47 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_c858328d","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":4},"updated":"2017-02-14 10:48:55.000000000","message":"remove this padding from the commit message","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sven mark \u003cmasw@awcloud.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-14 16:50:47 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_e8d416be","line":7,"range":{"start_line":7,"start_character":55,"end_line":7,"end_character":77},"updated":"2017-02-14 10:48:55.000000000","message":"title should be shorter than 53 chars","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_48fb6247","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":45},"updated":"2017-02-14 10:48:55.000000000","message":"add a space here","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_a89cde0f","line":9,"range":{"start_line":9,"start_character":69,"end_line":9,"end_character":77},"updated":"2017-02-14 10:48:55.000000000","message":"don\u0027t go beyond 72 chars.","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_0857aa58","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"updated":"2017-02-14 10:48:55.000000000","message":"here and below","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_2802ae3d","line":9,"range":{"start_line":9,"start_character":22,"end_line":9,"end_character":28},"updated":"2017-02-14 10:48:55.000000000","message":"manila?","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_68a66654","line":10,"range":{"start_line":10,"start_character":58,"end_line":10,"end_character":59},"updated":"2017-02-14 10:48:55.000000000","message":"remove this space","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    add attr access_key to access view returned from  api share.access_list()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_e87096de","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":57},"updated":"2017-02-14 10:48:55.000000000","message":"you seem to be describing the problem, but what about the solution?","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    when mount cephfs malina share instance,we need access_key to auth cephx,"},{"line_number":10,"context_line":"    but find object access has no access_key attr returned ."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    Closes-Bug:#1664462"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifcab81d172e165071994dd74c83056c57683e19d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa31d9ce_28b0ee89","line":12,"range":{"start_line":12,"start_character":14,"end_line":12,"end_character":16},"updated":"2017-02-14 10:48:55.000000000","message":"add a space between : and #","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"}],"manila/api/views/share_accesses.py":[{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":40,"context_line":"            \u0027access_to\u0027: access.get(\u0027access_to\u0027),"},{"line_number":41,"context_line":"            \u0027access_type\u0027: access.get(\u0027access_type\u0027),"},{"line_number":42,"context_line":"            \u0027state\u0027: access.get(\u0027state\u0027),"},{"line_number":43,"context_line":"            \u0027access_key\u0027: access.get(\u0027access_key\u0027),"},{"line_number":44,"context_line":"        }"},{"line_number":45,"context_line":"        self.update_versioned_resource_dict("},{"line_number":46,"context_line":"            request, access_dict, access)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_c83912f4","line":43,"range":{"start_line":43,"start_character":12,"end_line":43,"end_character":51},"updated":"2017-02-14 10:48:55.000000000","message":"you cannot add a field to the view without an API microversion bump.","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"4ec8c7a443da7dd420432ad54288eee996f51137","unresolved":false,"context_lines":[{"line_number":40,"context_line":"            \u0027access_to\u0027: access.get(\u0027access_to\u0027),"},{"line_number":41,"context_line":"            \u0027access_type\u0027: access.get(\u0027access_type\u0027),"},{"line_number":42,"context_line":"            \u0027state\u0027: access.get(\u0027state\u0027),"},{"line_number":43,"context_line":"            \u0027access_key\u0027: access.get(\u0027access_key\u0027),"},{"line_number":44,"context_line":"        }"},{"line_number":45,"context_line":"        self.update_versioned_resource_dict("},{"line_number":46,"context_line":"            request, access_dict, access)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_b9b43282","line":43,"range":{"start_line":43,"start_character":12,"end_line":43,"end_character":51},"in_reply_to":"fa31d9ce_c83912f4","updated":"2017-02-14 11:56:04.000000000","message":"Rodrigo is right. It should be done as it is ALREADY done on lines 64-66. And this field is viewable starting with microversion 2.21","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"97f9adb33fc0b5f82a167835d50be1224622bae9","unresolved":false,"context_lines":[{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    @common.ViewBuilder.versioned_method(\"2.21\")"},{"line_number":65,"context_line":"    def add_access_key(self, context, access_dict, access):"},{"line_number":66,"context_line":"        access_dict[\u0027access_key\u0027] \u003d access.get(\u0027access_key\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    @common.ViewBuilder.versioned_method(\"1.0\", \"2.27\")"},{"line_number":69,"context_line":"    def translate_transitional_statuses(self, context, access_dict, access):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_482e02a6","line":66,"range":{"start_line":66,"start_character":8,"end_line":66,"end_character":60},"updated":"2017-02-14 10:48:55.000000000","message":"what you are looking for is already here. I have taken a look at the bug entry and there is no output or steps to reproduce, I am not sure what the problem is.","commit_id":"e49214dcc1498847d3e23015b856c8168829ae0c"}]}