)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"f2b0f937688d51cff9575d9030954c5002d0c239","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Cluster get operation now supports show_events flag to display provision"},{"line_number":10,"context_line":"progress."},{"line_number":11,"context_line":"Removed events endpoint as it is going to be removed from the API."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I6892c8be2f8daaa215732f5e58524620cb9a1f85"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ba7be1f8_a146664d","line":12,"updated":"2015-03-02 15:03:32.000000000","message":"is there a bug or blueprint to reference here?","commit_id":"04aa5aa457e1d419ad4398bbd703a4d32c02a71e"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"6e31c2038e59fa90abc4f80ddd9f02799a1f60fa","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Cluster get operation now supports show_events flag to display provision"},{"line_number":10,"context_line":"progress."},{"line_number":11,"context_line":"Removed events endpoint as it is going to be removed from the API."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I6892c8be2f8daaa215732f5e58524620cb9a1f85"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ba7be1f8_1bc4f690","line":12,"updated":"2015-03-02 17:34:59.000000000","message":"is there a bug or blueprint associated with this change?","commit_id":"2a5effbed67d17b5025d93f15ae5995c3c99c0ca"},{"author":{"_account_id":6786,"name":"Sergey Lukjanov","email":"me@slukjanov.name","username":"slukjanov"},"change_message_id":"c7b160b3b4186944b31a2333368102bd9a03f38c","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Cluster get operation now supports show_events flag to display provision"},{"line_number":10,"context_line":"progress."},{"line_number":11,"context_line":"Removed events endpoint as it is going to be removed from the API."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I6892c8be2f8daaa215732f5e58524620cb9a1f85"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9a80dd14_078fc075","line":11,"updated":"2015-03-10 10:46:23.000000000","message":"Please, add link to the bug or blueprint related to it.","commit_id":"fedd03b6c7348008a7368c31d44195b0a0cd789c"}],"saharaclient/api/clusters.py":[{"author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"change_message_id":"60541261fa0c4e4081dd3be72cc50a407b70ddf3","unresolved":false,"context_lines":[{"line_number":58,"context_line":"    def get(self, cluster_id, show_progress\u003dFalse):"},{"line_number":59,"context_line":"        url \u003d \u0027/clusters/%s\u0027 % cluster_id"},{"line_number":60,"context_line":"        if show_progress:"},{"line_number":61,"context_line":"            url +\u003d \"?show_progress\u003dtrue\""},{"line_number":62,"context_line":"        return self._get(url, \u0027cluster\u0027)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def delete(self, cluster_id):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9a80dd14_833c1fe4","line":61,"range":{"start_line":61,"start_character":21,"end_line":61,"end_character":39},"updated":"2015-03-05 17:35:09.000000000","message":"use urllib.urlencode for creating arguments","commit_id":"1ba797a5deb92eedcf27a847eb03f9697e632afb"},{"author":{"_account_id":12707,"name":"Jacob Bin Wang","email":"wangbin@awcloud.com","username":"Jacob"},"change_message_id":"2a6105559c68436e70e13ba6e32d15b3c3385f6b","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        return self._list(\u0027/clusters%s\u0027 % query, \u0027clusters\u0027)"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def get(self, cluster_id, show_progress\u003dFalse):"},{"line_number":61,"context_line":"        url \u003d \u0027/clusters/%(cluster_id)s?%(params)s\u0027 % \\"},{"line_number":62,"context_line":"              {\"cluster_id\": cluster_id,"},{"line_number":63,"context_line":"               \"params\": urllib.urlencode({\"show_progress\": show_progress})}"},{"line_number":64,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_2a5b122f","line":61,"updated":"2015-03-06 15:02:47.000000000","message":"\\ is not recommended now","commit_id":"e50876c0c58ca6cb6bd857a3254f34e0afa8e720"},{"author":{"_account_id":7710,"name":"Sergey Reshetnyak","email":"sreshetniak@mirantis.com","username":"sreshetniak"},"change_message_id":"8d7feba59233c1f9fe5a00d4643ac7420f42e14f","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    def get(self, cluster_id, show_progress\u003dFalse):"},{"line_number":61,"context_line":"        url \u003d (\u0027/clusters/%(cluster_id)s?%(params)s\u0027 %"},{"line_number":62,"context_line":"               {\"cluster_id\": cluster_id,"},{"line_number":63,"context_line":"                \"params\": urllib.urlencode({\"show_progress\": show_progress})})"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        return self._get(url, \u0027cluster\u0027)"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"9a80dd14_3e480a87","line":63,"range":{"start_line":63,"start_character":26,"end_line":63,"end_character":42},"updated":"2015-03-17 16:24:18.000000000","message":"Is not compatible with python 3","commit_id":"2c93b71ed820c4ea3676f76914af38a51b083501"}],"saharaclient/api/shell.py":[{"author":{"_account_id":8091,"name":"Trevor McKay","email":"tmckay@redhat.com","username":"tmckay"},"change_message_id":"313e42afea35ed9ac8f690a0150849a35ca12ff0","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    if not (name or id):"},{"line_number":115,"context_line":"        raise exceptions.CommandError(\"either NAME or ID is required\")"},{"line_number":116,"context_line":"    if id:"},{"line_number":117,"context_line":"        return manager.get(id, **kwargs)"},{"line_number":118,"context_line":"    ls \u003d manager.find(name\u003dname, **kwargs)"},{"line_number":119,"context_line":"    if len(ls) \u003d\u003d 0:"},{"line_number":120,"context_line":"        raise exceptions.CommandError(\"%s \u0027%s\u0027 not found\" %"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_56c3e389","line":117,"updated":"2015-02-27 20:24:30.000000000","message":"won\u0027t this cause an unexpected keyword argument in all of the managers unless\nthey all have get() fixed up to take **kwargs?","commit_id":"10282d2760e1391ca4189d55ea3d6a91d34907d8"},{"author":{"_account_id":7132,"name":"Nikita Konovalov","email":"nkonovalov@mirantis.com","username":"nkonovalov"},"change_message_id":"e23f2c16acd1dab002b750fdb0685d45de19cfd7","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    if not (name or id):"},{"line_number":115,"context_line":"        raise exceptions.CommandError(\"either NAME or ID is required\")"},{"line_number":116,"context_line":"    if id:"},{"line_number":117,"context_line":"        return manager.get(id, **kwargs)"},{"line_number":118,"context_line":"    ls \u003d manager.find(name\u003dname, **kwargs)"},{"line_number":119,"context_line":"    if len(ls) \u003d\u003d 0:"},{"line_number":120,"context_line":"        raise exceptions.CommandError(\"%s \u0027%s\u0027 not found\" %"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7be1f8_40b5b2ec","line":117,"in_reply_to":"ba7be1f8_56c3e389","updated":"2015-03-02 10:42:01.000000000","message":"Do you suggest adding a check for empty **kwargs?\nAll managers except clusters do not support charges, however nothing wrong is going to happen .get is called with empty kwargs.","commit_id":"10282d2760e1391ca4189d55ea3d6a91d34907d8"},{"author":{"_account_id":12707,"name":"Jacob Bin Wang","email":"wangbin@awcloud.com","username":"Jacob"},"change_message_id":"2a6105559c68436e70e13ba6e32d15b3c3385f6b","unresolved":false,"context_lines":[{"line_number":271,"context_line":"# Clusters"},{"line_number":272,"context_line":"# ~~~~~~~~"},{"line_number":273,"context_line":"# cluster-list"},{"line_number":274,"context_line":"#"},{"line_number":275,"context_line":"# cluster-show --name \u003ccluster\u003e|--id \u003ccluster_id\u003e [--json]"},{"line_number":276,"context_line":"#"},{"line_number":277,"context_line":"# cluster-create [--json \u003cfile\u003e]"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_0f43c06c","line":274,"updated":"2015-03-06 15:02:47.000000000","message":"Should add description here","commit_id":"e50876c0c58ca6cb6bd857a3254f34e0afa8e720"},{"author":{"_account_id":7132,"name":"Nikita Konovalov","email":"nkonovalov@mirantis.com","username":"nkonovalov"},"change_message_id":"cbc40ee5ca98e1ae73af3bef6600e9cba995b2e5","unresolved":false,"context_lines":[{"line_number":271,"context_line":"# Clusters"},{"line_number":272,"context_line":"# ~~~~~~~~"},{"line_number":273,"context_line":"# cluster-list"},{"line_number":274,"context_line":"#"},{"line_number":275,"context_line":"# cluster-show --name \u003ccluster\u003e|--id \u003ccluster_id\u003e [--json]"},{"line_number":276,"context_line":"#"},{"line_number":277,"context_line":"# cluster-create [--json \u003cfile\u003e]"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a80dd14_a5a81578","line":274,"in_reply_to":"9a80dd14_0f43c06c","updated":"2015-03-06 15:24:40.000000000","message":"The list operation does not support the new flag.","commit_id":"e50876c0c58ca6cb6bd857a3254f34e0afa8e720"},{"author":{"_account_id":12707,"name":"Jacob Bin Wang","email":"wangbin@awcloud.com","username":"Jacob"},"change_message_id":"68dd2fc50db0e5da3621bb015a1a69818883df63","unresolved":false,"context_lines":[{"line_number":272,"context_line":"# ~~~~~~~~"},{"line_number":273,"context_line":"# cluster-list"},{"line_number":274,"context_line":"#"},{"line_number":275,"context_line":"# cluster-show --name \u003ccluster\u003e|--id \u003ccluster_id\u003e [--json]"},{"line_number":276,"context_line":"#"},{"line_number":277,"context_line":"# cluster-create [--json \u003cfile\u003e]"},{"line_number":278,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a80dd14_13049916","line":275,"updated":"2015-03-09 10:23:54.000000000","message":"Sorry, I mean the cluster-show. Should add description about --show-progress here.","commit_id":"fedd03b6c7348008a7368c31d44195b0a0cd789c"}],"saharaclient/tests/unit/test_clusters.py":[{"author":{"_account_id":12707,"name":"Jacob Bin Wang","email":"wangbin@awcloud.com","username":"Jacob"},"change_message_id":"68dd2fc50db0e5da3621bb015a1a69818883df63","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        self.assertFields(self.body, resp[0])"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def test_clusters_get(self):"},{"line_number":68,"context_line":"        url \u003d self.URL + \u0027/clusters/id?show_progress\u003dFalse\u0027"},{"line_number":69,"context_line":"        self.responses.get(url, json\u003d{\u0027cluster\u0027: self.body})"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        resp \u003d self.client.clusters.get(\u0027id\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a80dd14_627fc015","line":68,"updated":"2015-03-09 10:23:54.000000000","message":"Should we add a testcase that show_progress is True?","commit_id":"fedd03b6c7348008a7368c31d44195b0a0cd789c"},{"author":{"_account_id":10670,"name":"Michael McCune","email":"elmiko@redhat.com","username":"mimccune"},"change_message_id":"7d4fe19ebd2086ae1d7582628820941abc757319","unresolved":false,"context_lines":[{"line_number":65,"context_line":"        self.assertFields(self.body, resp[0])"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def test_clusters_get(self):"},{"line_number":68,"context_line":"        url \u003d self.URL + \u0027/clusters/id?show_progress\u003dFalse\u0027"},{"line_number":69,"context_line":"        self.responses.get(url, json\u003d{\u0027cluster\u0027: self.body})"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        resp \u003d self.client.clusters.get(\u0027id\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a80dd14_744a0c56","line":68,"in_reply_to":"9a80dd14_627fc015","updated":"2015-03-09 16:27:54.000000000","message":"+1","commit_id":"fedd03b6c7348008a7368c31d44195b0a0cd789c"}]}