)]}'
{"openstackclient/compute/v2/server.py":[{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"0c6688e87628dd10366e76d8b14c39087833c550","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    \"\"\""},{"line_number":118,"context_line":"    info \u003d server._info.copy()"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    # Call .get() to retrieve all of the server information"},{"line_number":121,"context_line":"    # as findall(name\u003dblah) and REST /details are not the same"},{"line_number":122,"context_line":"    # and do not return flavor and image information."},{"line_number":123,"context_line":"    server \u003d compute_client.servers.get(info[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_8f6db69a","side":"PARENT","line":120,"updated":"2016-05-09 12:29:47.000000000","message":"The comment sounds like find should not be used because it doesn\u0027t return flavor and image information.  Have you confirmed that the same REST API is called in both cases?","commit_id":"4cd702c3610e4e6b40695ff9ea8eecf7c0735c68"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"762de564c889525fd97c02896e2172f888dbbc91","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    \"\"\""},{"line_number":118,"context_line":"    info \u003d server._info.copy()"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    # Call .get() to retrieve all of the server information"},{"line_number":121,"context_line":"    # as findall(name\u003dblah) and REST /details are not the same"},{"line_number":122,"context_line":"    # and do not return flavor and image information."},{"line_number":123,"context_line":"    server \u003d compute_client.servers.get(info[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_5eceab2e","side":"PARENT","line":120,"in_reply_to":"dab17558_10642b47","updated":"2016-05-10 11:28:44.000000000","message":"If that is the case, then why not just do the .get() call?","commit_id":"4cd702c3610e4e6b40695ff9ea8eecf7c0735c68"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"e15c58e076e968925bb5b1998a5e58415834a1ce","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    \"\"\""},{"line_number":118,"context_line":"    info \u003d server._info.copy()"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    # Call .get() to retrieve all of the server information"},{"line_number":121,"context_line":"    # as findall(name\u003dblah) and REST /details are not the same"},{"line_number":122,"context_line":"    # and do not return flavor and image information."},{"line_number":123,"context_line":"    server \u003d compute_client.servers.get(info[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_79fba0c1","side":"PARENT","line":120,"in_reply_to":"dab17558_5eceab2e","updated":"2016-05-11 02:47:15.000000000","message":"find_resource() calls .get() directly, and it gives more try if .get() failed to find the resoruce. If .get() could find the resource in the very beginning, there is no difference between find_resource() and .get(). I prefer find_resource() here so that we don\u0027t need to mock servers.get() in the unit test any more.","commit_id":"4cd702c3610e4e6b40695ff9ea8eecf7c0735c68"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"b348698f7cb7c9e255beeca14ab61f30b038505f","unresolved":false,"context_lines":[{"line_number":117,"context_line":"    \"\"\""},{"line_number":118,"context_line":"    info \u003d server._info.copy()"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    # Call .get() to retrieve all of the server information"},{"line_number":121,"context_line":"    # as findall(name\u003dblah) and REST /details are not the same"},{"line_number":122,"context_line":"    # and do not return flavor and image information."},{"line_number":123,"context_line":"    server \u003d compute_client.servers.get(info[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_10642b47","side":"PARENT","line":120,"in_reply_to":"dab17558_8f6db69a","updated":"2016-05-10 02:50:06.000000000","message":"I have confirmed that find_resource() will call exactly .get() to return a server obj the same as the previous one. It contains flavor and image info.","commit_id":"4cd702c3610e4e6b40695ff9ea8eecf7c0735c68"}]}
