)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"8b171ddec8010ef60eb085031b27b6824de0e005","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Parse filter item \"name\" correctly for snapshot-list"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"cinderclient will parse filter item \"name\" to \"display_name\""},{"line_number":10,"context_line":"when get snapshot list via v2 api, this will make cinder"},{"line_number":11,"context_line":"remove the filter as an invalid filter item and return the"},{"line_number":12,"context_line":"full snapshot list, so parse the \"name\" to \"name\" will fix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_49328211","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":12},"updated":"2016-03-09 09:12:41.000000000","message":"You should mention in the commit message that this is a bug only for *no-admin* users. So the reviewers can reproduce the bug.","commit_id":"08e8b46ebd845c9e60e67ca7080b165d15bd4b7a"},{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"8b171ddec8010ef60eb085031b27b6824de0e005","unresolved":false,"context_lines":[{"line_number":13,"context_line":"the problem."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I94439b082bf6ad56d3e84d3301e1869343ceae55"},{"line_number":16,"context_line":"Closes-Bug: #1554538"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_f556c1d0","line":16,"range":{"start_line":16,"start_character":13,"end_line":16,"end_character":20},"updated":"2016-03-09 09:12:41.000000000","message":"You should edit the Bug Description of it, provide a detailed reproduce step.\nThe key step may be this:\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n$ env | grep demo\nOS_USERNAME\u003ddemo\nOS_TENANT_NAME\u003ddemo\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nMake sure the user is not \"admin\"","commit_id":"08e8b46ebd845c9e60e67ca7080b165d15bd4b7a"},{"author":{"_account_id":16426,"name":"Cheney Dong","email":"cheneydc@gmail.com","username":"cheneydc"},"change_message_id":"4659d7745dc90edcade00e8757b390535690e998","unresolved":false,"context_lines":[{"line_number":13,"context_line":"the problem."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I94439b082bf6ad56d3e84d3301e1869343ceae55"},{"line_number":16,"context_line":"Closes-Bug: #1554538"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5aef4532_d5a62552","line":16,"range":{"start_line":16,"start_character":13,"end_line":16,"end_character":20},"in_reply_to":"5aef4532_f556c1d0","updated":"2016-03-09 09:20:02.000000000","message":"Yes, you\u0027re right, that\u0027s why others cannot reproduce it, thanks so much.","commit_id":"08e8b46ebd845c9e60e67ca7080b165d15bd4b7a"}],"cinderclient/tests/unit/v2/test_shell.py":[{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"8b171ddec8010ef60eb085031b27b6824de0e005","unresolved":false,"context_lines":[{"line_number":467,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/snapshots/detail?name\u003d123\u0027)"},{"line_number":468,"context_line":""},{"line_number":469,"context_line":"    def test_snapshot_list_filter_display_name(self):"},{"line_number":470,"context_line":"        self.run_command(\u0027snapshot-list --display-name\u003d123\u0027)"},{"line_number":471,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/snapshots/detail?name\u003d123\u0027)"},{"line_number":472,"context_line":""},{"line_number":473,"context_line":"    def test_snapshot_list_filter_status_and_volume_id(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5aef4532_ba0d4640","line":470,"range":{"start_line":470,"start_character":40,"end_line":470,"end_character":55},"updated":"2016-03-09 09:12:41.000000000","message":"The --display-name is suppressed, I think this is not necessary now.","commit_id":"08e8b46ebd845c9e60e67ca7080b165d15bd4b7a"},{"author":{"_account_id":20105,"name":"Cao ShuFeng","email":"caosf.fnst@cn.fujitsu.com","username":"caosf.fnst"},"change_message_id":"edad0330b4dd6ea1167469c230973f858b559a30","unresolved":false,"context_lines":[{"line_number":467,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/snapshots/detail?name\u003d123\u0027)"},{"line_number":468,"context_line":""},{"line_number":469,"context_line":"    def test_snapshot_list_filter_display_name(self):"},{"line_number":470,"context_line":"        self.run_command(\u0027snapshot-list --display-name\u003d123\u0027)"},{"line_number":471,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/snapshots/detail?name\u003d123\u0027)"},{"line_number":472,"context_line":""},{"line_number":473,"context_line":"    def test_snapshot_list_filter_status_and_volume_id(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"5aef4532_4dd449bc","line":470,"range":{"start_line":470,"start_character":39,"end_line":470,"end_character":54},"updated":"2016-03-09 09:39:30.000000000","message":"The --display-name is suppressed, I don\u0027t think this is necessary.\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n$ cinder help snapshot-list\nusage: cinder snapshot-list [--all-tenants [\u003c0|1\u003e]] [--name \u003cname\u003e]\n                            [--status \u003cstatus\u003e] [--volume-id \u003cvolume-id\u003e]\n                            [--marker \u003cmarker\u003e] [--limit \u003climit\u003e]\n                            [--sort \u003ckey\u003e[:\u003cdirection\u003e]]\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nThere is no help message for \"--display-name\" although it works.\nThis will encourage other to use \"--display-name\" rather than recommended \"--name\".","commit_id":"8451f10e8178f67047af0b57c10c051d222720b1"}]}