)]}'
{"openstack/cluster/v1/cluster.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"2d27f9afcb6ff15e1dbbdc61929ffaaecd1cee03","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    #: Name of the profile used by the cluster."},{"line_number":76,"context_line":"    profile_name \u003d resource.Body(\u0027profile_name\u0027)"},{"line_number":77,"context_line":"    #: Cluster update profile only parameter the bool value."},{"line_number":78,"context_line":"    profile_only \u003d resource.Body(\u0027profile_only\u0027)"},{"line_number":79,"context_line":"    #: A dictionary with dependency information of the cluster"},{"line_number":80,"context_line":"    dependents \u003d resource.Body(\u0027dependents\u0027, type\u003ddict)"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bff0334d_c90cf442","line":78,"updated":"2017-04-10 13:25:39.000000000","message":"If this is a bool we would prefix it with an is_ name. Should this be is_profile_only? Can you link to documentation of what this is so I can help come up with a docstring that is a little more clear on what this is?","commit_id":"7ceb2fcde9961cde7f9260fbc12111727d7d291d"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"1603eb1873911bb8205df4f50ca5a1f27f2c10b9","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    _query_mapping \u003d resource.QueryParameters("},{"line_number":33,"context_line":"        \u0027name\u0027, \u0027status\u0027, \u0027sort\u0027, \u0027global_project\u0027, \u0027profile_only\u0027,"},{"line_number":34,"context_line":"        is_profile_only\u003d\u0027profile_only\u0027)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    # Properties"},{"line_number":37,"context_line":"    #: The name of the cluster."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ff52f3f_6504707c","line":34,"range":{"start_line":34,"start_character":8,"end_line":34,"end_character":38},"updated":"2017-04-13 12:34:52.000000000","message":"profile_only is a parameter only meant to be used in the body of a PATCH /clusters/\u003ccluster_id\u003e command, it means the update operation is not supposed to change the existing profiles of cluster nodes. So this is not supposed to be a query parameter.","commit_id":"097d7bf7550bb5cb6d7457e495db894ccbe564df"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"e4da558666fcb43c834d90b9a8a984b6d321b53e","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    _query_mapping \u003d resource.QueryParameters("},{"line_number":33,"context_line":"        \u0027name\u0027, \u0027status\u0027, \u0027sort\u0027, \u0027global_project\u0027, \u0027profile_only\u0027,"},{"line_number":34,"context_line":"        is_profile_only\u003d\u0027profile_only\u0027)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    # Properties"},{"line_number":37,"context_line":"    #: The name of the cluster."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ff52f3f_855bf485","line":34,"range":{"start_line":34,"start_character":8,"end_line":34,"end_character":38},"in_reply_to":"9ff52f3f_6504707c","updated":"2017-04-13 12:37:04.000000000","message":"Should the other one that\u0027s in there be removed instead of me adding this one? I noticed it up there, but I couldn\u0027t find it in the documentation...and you just noticed this before I could ask about it :)","commit_id":"097d7bf7550bb5cb6d7457e495db894ccbe564df"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"84b578225755c9411bdb76c2ff390cf9a8bde253","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    _query_mapping \u003d resource.QueryParameters("},{"line_number":33,"context_line":"        \u0027name\u0027, \u0027status\u0027, \u0027sort\u0027, \u0027global_project\u0027, \u0027profile_only\u0027,"},{"line_number":34,"context_line":"        is_profile_only\u003d\u0027profile_only\u0027)"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    # Properties"},{"line_number":37,"context_line":"    #: The name of the cluster."}],"source_content_type":"text/x-python","patch_set":4,"id":"9ff52f3f_bdaa8882","line":34,"range":{"start_line":34,"start_character":8,"end_line":34,"end_character":38},"in_reply_to":"9ff52f3f_855bf485","updated":"2017-04-13 14:36:18.000000000","message":"both should be removed. it is only valid as a field of a request body.","commit_id":"097d7bf7550bb5cb6d7457e495db894ccbe564df"}],"openstack/tests/unit/cluster/v1/test_cluster.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"a99757f8664d6382fd88f928ebaf2797badadc86","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    \u0027max_size\u0027: 3,"},{"line_number":26,"context_line":"    \u0027min_size\u0027: 0,"},{"line_number":27,"context_line":"    \u0027name\u0027: FAKE_NAME,"},{"line_number":28,"context_line":"    \u0027profile_id\u0027: \u0027myserver\u0027,"},{"line_number":29,"context_line":"    \u0027metadata\u0027: {},"},{"line_number":30,"context_line":"    \u0027dependents\u0027: {},"},{"line_number":31,"context_line":"    \u0027timeout\u0027: None,"},{"line_number":32,"context_line":"    \u0027init_at\u0027: \u00272015-10-10T12:46:36.000000\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_0c11caac","line":29,"range":{"start_line":28,"start_character":0,"end_line":29,"end_character":19},"updated":"2017-04-10 13:38:57.000000000","message":"That should be up there","commit_id":"e1be59cd1e09fe9c080a3a8f3f25ed853f4c6cf3"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"a99757f8664d6382fd88f928ebaf2797badadc86","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        \u0027name\u0027: \u0027test_cluster\u0027,"},{"line_number":52,"context_line":"        \u0027nodes\u0027: [],"},{"line_number":53,"context_line":"        \u0027policies\u0027: [],"},{"line_number":54,"context_line":"        \u0027profile_id\u0027: \u0027560a8f9d-7596-4a32-85e8-03645fa7be13\u0027,"},{"line_number":55,"context_line":"        \u0027profile_name\u0027: \u0027myserver\u0027,"},{"line_number":56,"context_line":"        \u0027profile_only\u0027: True,"},{"line_number":57,"context_line":"        \u0027project\u0027: \u0027333acb15a43242f4a609a27cb097a8f2\u0027,"},{"line_number":58,"context_line":"        \u0027status\u0027: \u0027INIT\u0027,"},{"line_number":59,"context_line":"        \u0027status_reason\u0027: \u0027Initializing\u0027,"},{"line_number":60,"context_line":"        \u0027timeout\u0027: None,"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_6c7bb6f4","line":57,"range":{"start_line":54,"start_character":0,"end_line":57,"end_character":38},"updated":"2017-04-10 13:38:57.000000000","message":"instead of in here, missing some other values maybe in FAKE?","commit_id":"e1be59cd1e09fe9c080a3a8f3f25ed853f4c6cf3"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"a99757f8664d6382fd88f928ebaf2797badadc86","unresolved":false,"context_lines":[{"line_number":87,"context_line":"        self.assertEqual(FAKE[\u0027id\u0027], sot.id)"},{"line_number":88,"context_line":"        self.assertEqual(FAKE[\u0027name\u0027], sot.name)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        self.assertEqual(FAKE[\u0027profile_id\u0027], sot.profile_id)"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        self.assertEqual(FAKE[\u0027min_size\u0027], sot.min_size)"},{"line_number":93,"context_line":"        self.assertEqual(FAKE[\u0027max_size\u0027], sot.max_size)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_4c69121f","line":90,"range":{"start_line":90,"start_character":8,"end_line":90,"end_character":60},"updated":"2017-04-10 13:38:57.000000000","message":"test down here somewhere","commit_id":"e1be59cd1e09fe9c080a3a8f3f25ed853f4c6cf3"}]}