)]}'
{"nova/tests/virt/vmwareapi/test_vmwareapi.py":[{"author":{"_account_id":8027,"name":"Vui Lam","email":"vui@vmware.com","username":"vui"},"change_message_id":"49e0caad45a3b7b18e0919e8c22d49f38afce884","unresolved":false,"context_lines":[{"line_number":158,"context_line":"                  \u0027root_gb\u0027: 10,"},{"line_number":159,"context_line":"                  }"},{"line_number":160,"context_line":"        self.instance \u003d db.instance_create(None, values)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    def _create_vm(self, node\u003dNone, num_instances\u003d1):"},{"line_number":163,"context_line":"        \"\"\"Create and spawn the VM.\"\"\""},{"line_number":164,"context_line":"        if not node:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F3jU%3D","line":161,"updated":"2013-09-18 00:20:27.000000000","message":"Perhaps we should have another test that mocks up _extend_virtual_disk and validates that it gets called on when spawning a linked clone or regular vm.","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"08e250d54df7e590b8b6992fb494e9e72eb1ce7a","unresolved":false,"context_lines":[{"line_number":158,"context_line":"                  \u0027root_gb\u0027: 10,"},{"line_number":159,"context_line":"                  }"},{"line_number":160,"context_line":"        self.instance \u003d db.instance_create(None, values)"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    def _create_vm(self, node\u003dNone, num_instances\u003d1):"},{"line_number":163,"context_line":"        \"\"\"Create and spawn the VM.\"\"\""},{"line_number":164,"context_line":"        if not node:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F2gY%3D","line":161,"in_reply_to":"AAAAS3%2F%2F3jU%3D","updated":"2013-09-18 08:36:06.000000000","message":"Done","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"}],"nova/virt/vmwareapi/vmops.py":[{"author":{"_account_id":8027,"name":"Vui Lam","email":"vui@vmware.com","username":"vui"},"change_message_id":"49e0caad45a3b7b18e0919e8c22d49f38afce884","unresolved":false,"context_lines":[{"line_number":459,"context_line":"            # Extend the disk size if necessary"},{"line_number":460,"context_line":"            if not linked_clone:"},{"line_number":461,"context_line":"                root_vmdk_path \u003d uploaded_vmdk_path"},{"line_number":462,"context_line":"                self._extend_virtual_disk(instance, root_gb_in_kb,"},{"line_number":463,"context_line":"                                          vmdk_file_size_in_kb,"},{"line_number":464,"context_line":"                                          root_vmdk_path, dc_ref)"},{"line_number":465,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F3fs%3D","line":462,"updated":"2013-09-18 00:20:27.000000000","message":"Nit: (feel free to ignore) it may be clearer to do:\nif root_gb_in_kb \u003e vmdk_file_size_in_kb:\n   self._extend_virtual_disk(instance, root_gb_in_kb, root_vmdk_path, dc_ref)","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"08e250d54df7e590b8b6992fb494e9e72eb1ce7a","unresolved":false,"context_lines":[{"line_number":459,"context_line":"            # Extend the disk size if necessary"},{"line_number":460,"context_line":"            if not linked_clone:"},{"line_number":461,"context_line":"                root_vmdk_path \u003d uploaded_vmdk_path"},{"line_number":462,"context_line":"                self._extend_virtual_disk(instance, root_gb_in_kb,"},{"line_number":463,"context_line":"                                          vmdk_file_size_in_kb,"},{"line_number":464,"context_line":"                                          root_vmdk_path, dc_ref)"},{"line_number":465,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F2jc%3D","line":462,"in_reply_to":"AAAAS3%2F%2F3fs%3D","updated":"2013-09-18 08:36:06.000000000","message":"Done","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":1313,"name":"yaguang","display_name":"Yaguang Tang","email":"heut2008@gmail.com","username":"heut2008"},"change_message_id":"d5386e7d0469f71767b89d0e5abcaf17cd7307f8","unresolved":false,"context_lines":[{"line_number":485,"context_line":"                        destSpec\u003dcopy_spec)"},{"line_number":486,"context_line":"                    self._session._wait_for_task(instance[\u0027uuid\u0027],"},{"line_number":487,"context_line":"                                                 vmdk_copy_task)"},{"line_number":488,"context_line":"                    self._extend_virtual_disk(instance, root_gb_in_kb,"},{"line_number":489,"context_line":"                                              vmdk_file_size_in_kb,"},{"line_number":490,"context_line":"                                              root_vmdk_path, dc_ref)"},{"line_number":491,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F3Jg%3D","line":488,"updated":"2013-09-18 04:43:30.000000000","message":"if root_gb_in_kb \u003e vmdk_file_size_in_kb:\n   self._extend_virtual_disk(instance, root_gb_in_kb, root_vmdk_path, dc_ref)","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"08e250d54df7e590b8b6992fb494e9e72eb1ce7a","unresolved":false,"context_lines":[{"line_number":485,"context_line":"                        destSpec\u003dcopy_spec)"},{"line_number":486,"context_line":"                    self._session._wait_for_task(instance[\u0027uuid\u0027],"},{"line_number":487,"context_line":"                                                 vmdk_copy_task)"},{"line_number":488,"context_line":"                    self._extend_virtual_disk(instance, root_gb_in_kb,"},{"line_number":489,"context_line":"                                              vmdk_file_size_in_kb,"},{"line_number":490,"context_line":"                                              root_vmdk_path, dc_ref)"},{"line_number":491,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F2jY%3D","line":488,"in_reply_to":"AAAAS3%2F%2F3EU%3D","updated":"2013-09-18 08:36:06.000000000","message":"Done","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":8027,"name":"Vui Lam","email":"vui@vmware.com","username":"vui"},"change_message_id":"bf8685d18e0a82758d88ff8ad8053bbade2e1353","unresolved":false,"context_lines":[{"line_number":485,"context_line":"                        destSpec\u003dcopy_spec)"},{"line_number":486,"context_line":"                    self._session._wait_for_task(instance[\u0027uuid\u0027],"},{"line_number":487,"context_line":"                                                 vmdk_copy_task)"},{"line_number":488,"context_line":"                    self._extend_virtual_disk(instance, root_gb_in_kb,"},{"line_number":489,"context_line":"                                              vmdk_file_size_in_kb,"},{"line_number":490,"context_line":"                                              root_vmdk_path, dc_ref)"},{"line_number":491,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F3EU%3D","line":488,"in_reply_to":"AAAAS3%2F%2F3Jg%3D","updated":"2013-09-18 05:41:47.000000000","message":"Yep definitely meant to say this too when I comment on the previous call site. Do both or none, obviously.","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":8027,"name":"Vui Lam","email":"vui@vmware.com","username":"vui"},"change_message_id":"49e0caad45a3b7b18e0919e8c22d49f38afce884","unresolved":false,"context_lines":[{"line_number":489,"context_line":"                                              vmdk_file_size_in_kb,"},{"line_number":490,"context_line":"                                              root_vmdk_path, dc_ref)"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":"            # Attach the vmdk uploaded to the VM."},{"line_number":493,"context_line":"            self._volumeops.attach_disk_to_vm("},{"line_number":494,"context_line":"                                vm_ref, instance,"},{"line_number":495,"context_line":"                                adapter_type, disk_type, root_vmdk_path,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F3g0%3D","line":492,"updated":"2013-09-18 00:20:27.000000000","message":"\"vmdk uploaded\" -\u003e \"root disk\"\nmay be more apt here.","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"08e250d54df7e590b8b6992fb494e9e72eb1ce7a","unresolved":false,"context_lines":[{"line_number":489,"context_line":"                                              vmdk_file_size_in_kb,"},{"line_number":490,"context_line":"                                              root_vmdk_path, dc_ref)"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":"            # Attach the vmdk uploaded to the VM."},{"line_number":493,"context_line":"            self._volumeops.attach_disk_to_vm("},{"line_number":494,"context_line":"                                vm_ref, instance,"},{"line_number":495,"context_line":"                                adapter_type, disk_type, root_vmdk_path,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F2jQ%3D","line":492,"in_reply_to":"AAAAS3%2F%2F3g0%3D","updated":"2013-09-18 08:36:06.000000000","message":"Done","commit_id":"7f648ee50c699f7b9691b035a5a442db4c04cd5d"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"41fbd44f93bc904e3d0e4e623795f50bcb9c8a26","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d max(root_gb * 1024 * 1024,"},{"line_number":212,"context_line":"                            vmdk_file_size_in_kb)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        vm_folder_ref \u003d self._get_vmfolder_ref()"},{"line_number":215,"context_line":"        node_mo_id \u003d vm_util.get_mo_id_from_instance(instance)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2Fts8%3D","line":212,"updated":"2013-09-20 17:49:48.000000000","message":"What if the vmdk image size is larger than the flavor size?  The driver should raise an error in that case.  It looks like that\u0027s not handled here.","commit_id":"9357c46e49beac1aee00ee71db522277cc988a2a"},{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"5cdd5052177bce8e2cb5a00bc3c4c285c006034c","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d max(root_gb * 1024 * 1024,"},{"line_number":212,"context_line":"                            vmdk_file_size_in_kb)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        vm_folder_ref \u003d self._get_vmfolder_ref()"},{"line_number":215,"context_line":"        node_mo_id \u003d vm_util.get_mo_id_from_instance(instance)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2Frok%3D","line":212,"in_reply_to":"AAAAS3%2F%2Frwo%3D","updated":"2013-09-22 07:00:19.000000000","message":"I am wondering this check would take any effect at here, since the https://github.com/openstack/nova/blob/master/nova/compute/api.py#L591 will check the image size with root_gb, a exception will be throw before gose to here, sorry if I miss something.","commit_id":"9357c46e49beac1aee00ee71db522277cc988a2a"},{"author":{"_account_id":8027,"name":"Vui Lam","email":"vui@vmware.com","username":"vui"},"change_message_id":"aaefe87ac4d08f300e95e0851ddf736527d1c026","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d max(root_gb * 1024 * 1024,"},{"line_number":212,"context_line":"                            vmdk_file_size_in_kb)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        vm_folder_ref \u003d self._get_vmfolder_ref()"},{"line_number":215,"context_line":"        node_mo_id \u003d vm_util.get_mo_id_from_instance(instance)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2Frwo%3D","line":212,"in_reply_to":"AAAAS3%2F%2FseI%3D","updated":"2013-09-22 05:14:04.000000000","message":"I was initially okay with the idea of just using the image size if flavor\u0027s root disk size is smaller, but for consistency, looks like we should throw and exception.InstanceTypeDiskTooSmall instead. (I looked a couple of other drivers like libvirt, hyperv, xenapi; while they do things in their own different ways, they will eventually error out if the image cannot be used to create a disk with the size dictated by the flavor). If we decide to go the route of throwing, be sure to exclude the case where instance[\u0027root_gb\u0027] is 0 (the disk should be created with the image vmdk\u0027s size in that case). Thanks","commit_id":"9357c46e49beac1aee00ee71db522277cc988a2a"},{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"change_message_id":"b6a5930ac4b79b5a25f12e31290807021ae1e022","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d max(root_gb * 1024 * 1024,"},{"line_number":212,"context_line":"                            vmdk_file_size_in_kb)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        vm_folder_ref \u003d self._get_vmfolder_ref()"},{"line_number":215,"context_line":"        node_mo_id \u003d vm_util.get_mo_id_from_instance(instance)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2FsFs%3D","line":212,"in_reply_to":"AAAAS3%2F%2FseI%3D","updated":"2013-09-21 19:37:25.000000000","message":"I would have guessed that an error and failure would be appropriate here.  What do other drivers like libvirt do in a similar situation?  Seems like consistency would be the best goal here.","commit_id":"9357c46e49beac1aee00ee71db522277cc988a2a"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"313be23bcabf8ddbf51576b7c08e32a519373d26","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d max(root_gb * 1024 * 1024,"},{"line_number":212,"context_line":"                            vmdk_file_size_in_kb)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        vm_folder_ref \u003d self._get_vmfolder_ref()"},{"line_number":215,"context_line":"        node_mo_id \u003d vm_util.get_mo_id_from_instance(instance)"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAS3%2F%2FseI%3D","line":212,"in_reply_to":"AAAAS3%2F%2Fts8%3D","updated":"2013-09-21 08:46:12.000000000","message":"My thinking is that that is too extreme at the moment. I would prefer that we enable the user to deploy an instance and a warning be logged for the administrator.\nA change like this would break existing installations that work.","commit_id":"9357c46e49beac1aee00ee71db522277cc988a2a"},{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"bf648c5d7455de78e060b5128a1e6ddfa791ac93","unresolved":false,"context_lines":[{"line_number":139,"context_line":"                datacenter\u003ddatacenter,"},{"line_number":140,"context_line":"                newCapacityKb\u003drequested_size,"},{"line_number":141,"context_line":"                eagerZero\u003dFalse)"},{"line_number":142,"context_line":"        self._session._wait_for_task(instance[\u0027uuid\u0027],"},{"line_number":143,"context_line":"                                     vmdk_extend_task)"},{"line_number":144,"context_line":"        LOG.debug(_(\"Extended root virtual disk\"))"},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2FjwY%3D","line":142,"updated":"2013-09-24 18:55:05.000000000","message":"consider: instance.get(\u0027uuid\u0027)","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"158d0af93fe2357fcb91c1c8b60d203eaedb7d40","unresolved":false,"context_lines":[{"line_number":139,"context_line":"                datacenter\u003ddatacenter,"},{"line_number":140,"context_line":"                newCapacityKb\u003drequested_size,"},{"line_number":141,"context_line":"                eagerZero\u003dFalse)"},{"line_number":142,"context_line":"        self._session._wait_for_task(instance[\u0027uuid\u0027],"},{"line_number":143,"context_line":"                                     vmdk_extend_task)"},{"line_number":144,"context_line":"        LOG.debug(_(\"Extended root virtual disk\"))"},{"line_number":145,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2Fiww%3D","line":142,"in_reply_to":"AAAAS3%2F%2FjwY%3D","updated":"2013-09-24 22:26:28.000000000","message":"Why? Using .get() is wrong in a lot of cases, and always for uuid, IMHO.","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"},{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"bf648c5d7455de78e060b5128a1e6ddfa791ac93","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        (vmdk_file_size_in_kb, os_type, adapter_type,"},{"line_number":208,"context_line":"            disk_type, vif_model, image_linked_clone) \u003d _get_image_properties()"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d root_gb * 1024 * 1024"},{"line_number":212,"context_line":"        if root_gb_in_kb and vmdk_file_size_in_kb \u003e root_gb_in_kb:"},{"line_number":213,"context_line":"            reason \u003d _(\"Image disk size greater than requested disk size\")"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2Fjb4%3D","line":210,"updated":"2013-09-24 18:55:05.000000000","message":"consider: instance.get(\u0027root_gb\u0027, 0)","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"158d0af93fe2357fcb91c1c8b60d203eaedb7d40","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        (vmdk_file_size_in_kb, os_type, adapter_type,"},{"line_number":208,"context_line":"            disk_type, vif_model, image_linked_clone) \u003d _get_image_properties()"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d root_gb * 1024 * 1024"},{"line_number":212,"context_line":"        if root_gb_in_kb and vmdk_file_size_in_kb \u003e root_gb_in_kb:"},{"line_number":213,"context_line":"            reason \u003d _(\"Image disk size greater than requested disk size\")"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2Fiwk%3D","line":210,"in_reply_to":"AAAAS3%2F%2Fjb4%3D","updated":"2013-09-24 22:26:28.000000000","message":"Why? That should always be there. It\u0027s a bug if not, and this would paper over it.","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"},{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"bf648c5d7455de78e060b5128a1e6ddfa791ac93","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d root_gb * 1024 * 1024"},{"line_number":212,"context_line":"        if root_gb_in_kb and vmdk_file_size_in_kb \u003e root_gb_in_kb:"},{"line_number":213,"context_line":"            reason \u003d _(\"Image disk size greater than requested disk size\")"},{"line_number":214,"context_line":"            raise exception.InstanceUnacceptable(instance_id\u003dinstance[\u0027uuid\u0027],"},{"line_number":215,"context_line":"                                                 reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2Fjbw%3D","line":212,"updated":"2013-09-24 18:55:05.000000000","message":"if root_gb_in_kb is guaranteed to be 0 then this test can be simpler.","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"43ab2f7ba5548e65b12ebe6439772688861c97d3","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d root_gb * 1024 * 1024"},{"line_number":212,"context_line":"        if root_gb_in_kb and vmdk_file_size_in_kb \u003e root_gb_in_kb:"},{"line_number":213,"context_line":"            reason \u003d _(\"Image disk size greater than requested disk size\")"},{"line_number":214,"context_line":"            raise exception.InstanceUnacceptable(instance_id\u003dinstance[\u0027uuid\u0027],"},{"line_number":215,"context_line":"                                                 reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2FiIM%3D","line":212,"in_reply_to":"AAAAS3%2F%2FiwY%3D","updated":"2013-09-25 05:11:56.000000000","message":"if the root disk size is 0 then we wan to use the vmdk as it is irrespective of it size.","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"158d0af93fe2357fcb91c1c8b60d203eaedb7d40","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"        root_gb \u003d instance[\u0027root_gb\u0027]"},{"line_number":211,"context_line":"        root_gb_in_kb \u003d root_gb * 1024 * 1024"},{"line_number":212,"context_line":"        if root_gb_in_kb and vmdk_file_size_in_kb \u003e root_gb_in_kb:"},{"line_number":213,"context_line":"            reason \u003d _(\"Image disk size greater than requested disk size\")"},{"line_number":214,"context_line":"            raise exception.InstanceUnacceptable(instance_id\u003dinstance[\u0027uuid\u0027],"},{"line_number":215,"context_line":"                                                 reason\u003dreason)"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAS3%2F%2FiwY%3D","line":212,"in_reply_to":"AAAAS3%2F%2Fjbw%3D","updated":"2013-09-24 22:26:28.000000000","message":"Did you mean nonzero?","commit_id":"823bee9d44ceb7aacae02d90c929a1ba135b9e9e"}]}