)]}'
{"nova/tests/virt/xenapi/test_vmops.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"b97664876e945626f901189c272263c0339bc9a6","unresolved":false,"context_lines":[{"line_number":844,"context_line":"        vm_rec, vm_ref \u003d self.create_vm(\u0027test\u0027)"},{"line_number":845,"context_line":"        xenapi_fake.create_vif(0, vm_ref)"},{"line_number":846,"context_line":"        xenapi_fake.create_vif(2, vm_ref)"},{"line_number":847,"context_line":"        result \u003d self.vmops._find_next_vif_device(self.instance)"},{"line_number":848,"context_line":"        self.assertEqual(3, result)"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F9G%2Bc%3D","line":847,"updated":"2013-12-16 18:22:04.000000000","message":"any reason why this is not 1? that is, should it not be the first free device?","commit_id":"4d1f183e362c8c49be456380ef2eebb1891675a6"}],"nova/virt/xenapi/vmops.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"b97664876e945626f901189c272263c0339bc9a6","unresolved":false,"context_lines":[{"line_number":1725,"context_line":"            if device \u003e max_device:"},{"line_number":1726,"context_line":"                max_device \u003d device"},{"line_number":1727,"context_line":""},{"line_number":1728,"context_line":"        return max_device + 1"},{"line_number":1729,"context_line":""},{"line_number":1730,"context_line":"    def hot_unplug_vif(self, instance, vif):"},{"line_number":1731,"context_line":"        # NOTE(johngarbutt) synchronize on hotplug so we"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAATn%2F9G6A%3D","line":1728,"updated":"2013-12-16 18:22:04.000000000","message":"should this not be the first free device?","commit_id":"4d1f183e362c8c49be456380ef2eebb1891675a6"},{"author":{"_account_id":688,"name":"Sandy Walsh","email":"sandy.walsh@rackspace.com","username":"sandy-walsh"},"change_message_id":"9f3e6988c50eb92e7f3087709af7006ff7ca3af0","unresolved":false,"context_lines":[{"line_number":1725,"context_line":"            if device \u003e max_device:"},{"line_number":1726,"context_line":"                max_device \u003d device"},{"line_number":1727,"context_line":""},{"line_number":1728,"context_line":"        return max_device + 1"},{"line_number":1729,"context_line":""},{"line_number":1730,"context_line":"    def hot_unplug_vif(self, instance, vif):"},{"line_number":1731,"context_line":"        # NOTE(johngarbutt) synchronize on hotplug so we"}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAU3%2F%2FcRs%3D","line":1728,"in_reply_to":"AAAATn%2F9G6A%3D","updated":"2014-02-26 16:13:04.000000000","message":"Good point ... is there a reason we want to skip \"holes\" in the vif\u0027s vs. filling them in?","commit_id":"4d1f183e362c8c49be456380ef2eebb1891675a6"}]}