)]}'
{"ceilometer/compute/pollsters/util.py":[{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"2cea50076a065a4027ded7d00a7a3ab73af02897","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        \u0027flavor\u0027: instance.flavor,"},{"line_number":47,"context_line":"        \u0027status\u0027: instance.status.lower(),"},{"line_number":48,"context_line":"        # Image properties"},{"line_number":49,"context_line":"        \u0027image\u0027: (instance.image if instance.image else None),"},{"line_number":50,"context_line":"        \u0027image_ref\u0027: (instance.image[\u0027id\u0027] if instance.image else None),"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWn%2F%2F5Wc%3D","line":49,"updated":"2014-04-08 16:54:03.000000000","message":"Does instance.image contain empty string ?","commit_id":"62c0f682c3637e48666b0a5ad741a9e02fb12987"},{"author":{"_account_id":10242,"name":"Emilio Recio","email":"emiliorecio@gmail.com","username":"emiliorecio"},"change_message_id":"13d309e0cefdfe0b24a5323f3133a45aaaf17c8b","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        \u0027flavor\u0027: instance.flavor,"},{"line_number":47,"context_line":"        \u0027status\u0027: instance.status.lower(),"},{"line_number":48,"context_line":"        # Image properties"},{"line_number":49,"context_line":"        \u0027image\u0027: (instance.image if instance.image else None),"},{"line_number":50,"context_line":"        \u0027image_ref\u0027: (instance.image[\u0027id\u0027] if instance.image else None),"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWn%2F%2F5K0%3D","line":49,"updated":"2014-04-08 17:16:42.000000000","message":"The description of the bug says \n\"Ensure instance.image is not None ...Fix to ensure instance.image is not None so we can avoid AttributeErrors\"\nMakes no  sense with the fix u propose :-)","commit_id":"62c0f682c3637e48666b0a5ad741a9e02fb12987"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"cfe9c1b8fc91c1ab3c76da359bd6e643ffa68c45","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        \u0027flavor\u0027: instance.flavor,"},{"line_number":47,"context_line":"        \u0027status\u0027: instance.status.lower(),"},{"line_number":48,"context_line":"        # Image properties"},{"line_number":49,"context_line":"        \u0027image\u0027: (instance.image if instance.image else None),"},{"line_number":50,"context_line":"        \u0027image_ref\u0027: (instance.image[\u0027id\u0027] if instance.image else None),"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWn%2F%2F47s%3D","line":49,"in_reply_to":"AAAAWn%2F%2F5K0%3D","updated":"2014-04-08 17:47:37.000000000","message":"agreed :) Fixed!","commit_id":"62c0f682c3637e48666b0a5ad741a9e02fb12987"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"612ba7c73857a8d5a19eef00189996d1677de2d4","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        \u0027flavor\u0027: instance.flavor,"},{"line_number":47,"context_line":"        \u0027status\u0027: instance.status.lower(),"},{"line_number":48,"context_line":"        # Image properties"},{"line_number":49,"context_line":"        \u0027image\u0027: (instance.image if instance.image else None),"},{"line_number":50,"context_line":"        \u0027image_ref\u0027: (instance.image[\u0027id\u0027] if instance.image else None),"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2F96U%3D","line":49,"updated":"2014-04-09 07:33:07.000000000","message":"What does instance.image contains here when instance.image is unavailable ?\n\nThe code that fill instance.image in compute/nova_client.py seems already set this to None when image is unavailable. Perhaps this hide an other issue ?","commit_id":"8898d6c6be1e07200bbb29aee3eda88908bce4b9"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"6e2f024c1f43d5e213c866496646ae48244bac19","unresolved":false,"context_lines":[{"line_number":46,"context_line":"        \u0027flavor\u0027: instance.flavor,"},{"line_number":47,"context_line":"        \u0027status\u0027: instance.status.lower(),"},{"line_number":48,"context_line":"        # Image properties"},{"line_number":49,"context_line":"        \u0027image\u0027: (instance.image if instance.image else None),"},{"line_number":50,"context_line":"        \u0027image_ref\u0027: (instance.image[\u0027id\u0027] if instance.image else None),"},{"line_number":51,"context_line":"    }"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2F42Q%3D","line":49,"in_reply_to":"AAAAW3%2F%2F96U%3D","updated":"2014-04-09 17:15:42.000000000","message":"What you think about this new update. I think this addresses Thang\u0027s concern as well as. I optimized number of time we check if instance.image exists. We seem to be doing that atleast 3 times which is unnecessary.","commit_id":"8898d6c6be1e07200bbb29aee3eda88908bce4b9"}]}