)]}'
{"lib/tempest":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"0cdda165ed9513dec6f5572dde0ea0f7c04f48ab","unresolved":false,"context_lines":[{"line_number":144,"context_line":"    # Tempest creates instane types for himself"},{"line_number":145,"context_line":"    if  [[ -z \"$DEFAULT_INSTANCE_TYPE\" ]]; then"},{"line_number":146,"context_line":"        available_flavors\u003d$(nova flavor-list)"},{"line_number":147,"context_line":"        if [[ ! ( $available_flavors \u003d~ \u0027m1.nano\u0027 ) ]]; then"},{"line_number":148,"context_line":"            nova flavor-create m1.nano 42 64 0 1"},{"line_number":149,"context_line":"        fi"},{"line_number":150,"context_line":"        flavor_ref\u003d42"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAATn%2F%2Budg%3D","line":147,"updated":"2013-11-21 02:09:03.000000000","message":"I would have done\n\n if ! nova flavor-show m1.nano 2\u003e /dev/null; then\n     nova flavor-create...\n fi","commit_id":"3b80bde8c7345a5e8c217b6c5c256c2f83aa7900"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"e36bf7819e215590969e2ef324a2257b2f7d492b","unresolved":false,"context_lines":[{"line_number":144,"context_line":"    # Tempest creates instane types for himself"},{"line_number":145,"context_line":"    if  [[ -z \"$DEFAULT_INSTANCE_TYPE\" ]]; then"},{"line_number":146,"context_line":"        available_flavors\u003d$(nova flavor-list)"},{"line_number":147,"context_line":"        if [[ ! ( $available_flavors \u003d~ \u0027m1.nano\u0027 ) ]]; then"},{"line_number":148,"context_line":"            nova flavor-create m1.nano 42 64 0 1"},{"line_number":149,"context_line":"        fi"},{"line_number":150,"context_line":"        flavor_ref\u003d42"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAATn%2F%2BuaA%3D","line":147,"in_reply_to":"AAAATn%2F%2BucY%3D","updated":"2013-11-21 02:26:57.000000000","message":"Yes, but each invocation has less work to do, since it\u0027s a direct lookup.","commit_id":"3b80bde8c7345a5e8c217b6c5c256c2f83aa7900"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"a84fc5ae970275a81e89a015a5abc8c25c04899c","unresolved":false,"context_lines":[{"line_number":144,"context_line":"    # Tempest creates instane types for himself"},{"line_number":145,"context_line":"    if  [[ -z \"$DEFAULT_INSTANCE_TYPE\" ]]; then"},{"line_number":146,"context_line":"        available_flavors\u003d$(nova flavor-list)"},{"line_number":147,"context_line":"        if [[ ! ( $available_flavors \u003d~ \u0027m1.nano\u0027 ) ]]; then"},{"line_number":148,"context_line":"            nova flavor-create m1.nano 42 64 0 1"},{"line_number":149,"context_line":"        fi"},{"line_number":150,"context_line":"        flavor_ref\u003d42"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAATn%2F%2BucY%3D","line":147,"in_reply_to":"AAAATn%2F%2Budg%3D","updated":"2013-11-21 02:16:05.000000000","message":"Wouldn\u0027t that result in an extra invocation (in the m1.micro block below) that can be avoided by caching the result (which the patch does, as written)?","commit_id":"3b80bde8c7345a5e8c217b6c5c256c2f83aa7900"}]}