)]}'
{"tests/integration/api/v1/clients/clusters_client.py":[{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"3712ab5b76138dc80dd8d10d272f82fcf976f31e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2015 OpenStack Foundation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":1,"id":"9af37de9_d09da85a","line":1,"updated":"2015-05-11 11:12:53.000000000","message":"This copyright doesn\u0027t change, not sure why it\u0027s OpenStack Foundation in the first place, but that said, you should always add to Copyright lines, not change them.  Please add a new line for recent changes.","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"3712ab5b76138dc80dd8d10d272f82fcf976f31e","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    def list_clusters(self, params\u003dNone, url\u003d\u0027clusters\u0027):"},{"line_number":34,"context_line":"        \"\"\"List all clusters"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        :param params: Optional parameters for listing cluster"},{"line_number":37,"context_line":"        \"\"\""},{"line_number":38,"context_line":"        if params:"},{"line_number":39,"context_line":"            url +\u003d \u0027?%s\u0027 % urllib.urlencode(params)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9af37de9_50aad871","line":36,"updated":"2015-05-11 11:12:53.000000000","message":"Please add documentation for the url parameter.","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"b64ed7e446bf4ef569d0141855222306e5276a53","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":52,"context_line":"        return rest_client.ResponseBody(resp, self._parse_resp(body))"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    def create_cluster(self, name, flavor, network_id):"},{"line_number":55,"context_line":"        \"\"\"Create a new cluster with one node"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"        :param name: The name of the cluster"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ad8896b_a248a688","line":54,"updated":"2015-05-11 18:28:08.000000000","message":"should also pass in size","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"8683e95f180b69ca25b4df9843e6e87bcc8aeaae","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        resp, body \u003d self.post(\u0027clusters\u0027, post_body)"},{"line_number":73,"context_line":"        return rest_client.ResponseBody(resp, self._parse_resp(body))"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def create_cluster_from_body(self, post_body):"},{"line_number":76,"context_line":"        \"\"\"Create a new cluster with one node"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        :param post_body: The cluster info body to post"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ad8896b_5d3c472f","line":75,"updated":"2015-05-11 18:42:46.000000000","message":"this function does the same thing as the above one, probably better to make use of the above function instead (update to take size field as well)","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"},{"author":{"_account_id":10584,"name":"Min Pae","username":"sputnik13","inactive":true},"change_message_id":"3712ab5b76138dc80dd8d10d272f82fcf976f31e","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        return rest_client.ResponseBody(resp, self._parse_resp(body))"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def create_cluster_from_body(self, post_body):"},{"line_number":76,"context_line":"        \"\"\"Create a new cluster with one node"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        :param post_body: The cluster info body to post"},{"line_number":79,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"9af37de9_f0e66ce7","line":76,"updated":"2015-05-11 11:12:53.000000000","message":"Is this an accurate description of the function?  What distinguishes this from create_cluster?","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"}],"tests/integration/api/v1/clusters/test_clusters.py":[{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"6eab8598b4d4ba36a0aa73ba0be8557994623082","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                              self.client.create_cluster_from_body,"},{"line_number":75,"context_line":"                              bad)"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    # Not specifying volume_size does not throw a BadRequest exception"},{"line_number":78,"context_line":"    # Todo https://bugs.launchpad.net/cue/+bug/1452980"},{"line_number":79,"context_line":"    def test_create_cluster_missing_attributes(self):"},{"line_number":80,"context_line":"        # Missing Name"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ad8896b_1daa3f71","line":77,"updated":"2015-05-11 18:40:27.000000000","message":"this is ok since it is not a required field","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"},{"author":{"_account_id":13771,"name":"Davide Agnello","email":"dagnello@hp.com","username":"dagnello"},"change_message_id":"b64ed7e446bf4ef569d0141855222306e5276a53","unresolved":false,"context_lines":[{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    # Not specifying volume_size does not throw a BadRequest exception"},{"line_number":78,"context_line":"    # Todo https://bugs.launchpad.net/cue/+bug/1452980"},{"line_number":79,"context_line":"    def test_create_cluster_missing_attributes(self):"},{"line_number":80,"context_line":"        # Missing Name"},{"line_number":81,"context_line":"        post_body \u003d {"},{"line_number":82,"context_line":"            \u0027size\u0027: 1,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ad8896b_7d96e395","line":79,"updated":"2015-05-11 18:28:08.000000000","message":"should separate each into it\u0027s own test, since we can add tests that miss combination of parameters if needed too","commit_id":"c6ed3b1ce5a6e4334821d31406d909b1d3372a21"}]}