)]}'
{"ironic/nova/virt/ironic/driver.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"4d5139a239ee4124b9bd3af536934fbbf21701ce","unresolved":false,"context_lines":[{"line_number":461,"context_line":"                \u0027cpu_time\u0027: 0"},{"line_number":462,"context_line":"                }"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"    def deallocate_networks_on_reschedule(self, instance):"},{"line_number":465,"context_line":"        return True"},{"line_number":466,"context_line":""},{"line_number":467,"context_line":"    def macs_for_instance(self, instance):"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_b1f11902","line":464,"updated":"2014-08-11 09:30:33.000000000","message":"Nit: All non-private methods in the driver has a docstring, and I don\u0027t think this one should be an exception\n\nE.g\n\n def deallocate_networks_on_reschedule(self, instance):\n    \"\"\"Does the driver want networks deallocated on reschedule?\n   \n    :param instance: the instance object.\n    :returns: Boolean value. If True deallocate networks on reschedule.\n\n    \"\"\"","commit_id":"891420c51f2c17a7d4acd60bef364f510fa96b97"}]}