)]}'
{"cinder/scheduler/filter_scheduler.py":[{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"3789ada4dd299086a5234114014e2ab7e69390e1","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    def _max_attempts(self):"},{"line_number":103,"context_line":"        max_attempts \u003d FLAGS.scheduler_max_attempts"},{"line_number":104,"context_line":"        if max_attempts \u003c 1:"},{"line_number":105,"context_line":"            raise exception.CinderException(_(\"Invalid value for \""},{"line_number":106,"context_line":"                                            \"\u0027scheduler_max_attempts\u0027, \""},{"line_number":107,"context_line":"                                            \"must be \u003e\u003d 1\"))"},{"line_number":108,"context_line":"        return max_attempts"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUaQ%3D","line":105,"updated":"2013-01-29 04:56:15.000000000","message":"Please use a more specific exception.","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"af11cfc7bcc77e05a99f3cafac65a4e0033c88e5","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    def _max_attempts(self):"},{"line_number":103,"context_line":"        max_attempts \u003d FLAGS.scheduler_max_attempts"},{"line_number":104,"context_line":"        if max_attempts \u003c 1:"},{"line_number":105,"context_line":"            raise exception.CinderException(_(\"Invalid value for \""},{"line_number":106,"context_line":"                                            \"\u0027scheduler_max_attempts\u0027, \""},{"line_number":107,"context_line":"                                            \"must be \u003e\u003d 1\"))"},{"line_number":108,"context_line":"        return max_attempts"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUX8%3D","line":105,"in_reply_to":"AAAAM3%2F%2FUaQ%3D","updated":"2013-01-29 05:32:10.000000000","message":"Done","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"3789ada4dd299086a5234114014e2ab7e69390e1","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        \"\"\"Populate filter properties with history of retries for this"},{"line_number":129,"context_line":"        request. If maximum retries is exceeded, raise NoValidHost."},{"line_number":130,"context_line":"        \"\"\""},{"line_number":131,"context_line":"        max_attempts \u003d self._max_attempts()"},{"line_number":132,"context_line":"        retry \u003d filter_properties.pop(\u0027retry\u0027, {})"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        if max_attempts \u003d\u003d 1:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUaU%3D","line":131,"updated":"2013-01-29 04:56:15.000000000","message":"Does it make sense to check that the flag is \u003e\u003d1 every time it\u0027s accessed?  Is there a place to check it on init?","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"af11cfc7bcc77e05a99f3cafac65a4e0033c88e5","unresolved":false,"context_lines":[{"line_number":128,"context_line":"        \"\"\"Populate filter properties with history of retries for this"},{"line_number":129,"context_line":"        request. If maximum retries is exceeded, raise NoValidHost."},{"line_number":130,"context_line":"        \"\"\""},{"line_number":131,"context_line":"        max_attempts \u003d self._max_attempts()"},{"line_number":132,"context_line":"        retry \u003d filter_properties.pop(\u0027retry\u0027, {})"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        if max_attempts \u003d\u003d 1:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUXI%3D","line":131,"in_reply_to":"AAAAM3%2F%2FUaU%3D","updated":"2013-01-29 05:32:10.000000000","message":"Will move to __init__()","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"}],"cinder/tests/scheduler/test_filter_scheduler.py":[{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"3789ada4dd299086a5234114014e2ab7e69390e1","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        sched._schedule(self.context, request_spec,"},{"line_number":167,"context_line":"                        filter_properties\u003dfilter_properties)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        num_attempts \u003d filter_properties[\u0027retry\u0027][\u0027num_attempts\u0027]"},{"line_number":170,"context_line":"        self.assertEqual(2, num_attempts)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    def test_retry_exceeded_max_attempts(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUaE%3D","line":169,"updated":"2013-01-29 04:56:15.000000000","message":"Should we also check filter_properties[\u0027retry\u0027][\u0027hosts\u0027] here?","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"af11cfc7bcc77e05a99f3cafac65a4e0033c88e5","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        sched._schedule(self.context, request_spec,"},{"line_number":167,"context_line":"                        filter_properties\u003dfilter_properties)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        num_attempts \u003d filter_properties[\u0027retry\u0027][\u0027num_attempts\u0027]"},{"line_number":170,"context_line":"        self.assertEqual(2, num_attempts)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    def test_retry_exceeded_max_attempts(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUY0%3D","line":169,"in_reply_to":"AAAAM3%2F%2FUaE%3D","updated":"2013-01-29 05:32:10.000000000","message":"Well, I think test_add_retry_host() case has done this, right?","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"162170e558aacd0cec0554be50968df95daa82ee","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        sched._schedule(self.context, request_spec,"},{"line_number":167,"context_line":"                        filter_properties\u003dfilter_properties)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        num_attempts \u003d filter_properties[\u0027retry\u0027][\u0027num_attempts\u0027]"},{"line_number":170,"context_line":"        self.assertEqual(2, num_attempts)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    def test_retry_exceeded_max_attempts(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FUKs%3D","line":169,"updated":"2013-01-29 09:40:26.000000000","message":"What I meant here about testing filter_properties[\u0027retry\u0027][\u0027hosts\u0027] is that the other test tests that _add_retry_host() adds a host - here we should check that _schedule() calls _add_retry_host() which adds it.","commit_id":"00d4f7a1805953d678d1e1a1eba9616e552e2420"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"ec82530f77776cad7ed569d08182eff6fa565b40","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        sched._schedule(self.context, request_spec,"},{"line_number":167,"context_line":"                        filter_properties\u003dfilter_properties)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        num_attempts \u003d filter_properties[\u0027retry\u0027][\u0027num_attempts\u0027]"},{"line_number":170,"context_line":"        self.assertEqual(2, num_attempts)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    def test_retry_exceeded_max_attempts(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FTeo%3D","line":169,"in_reply_to":"AAAAM3%2F%2FUKs%3D","updated":"2013-01-29 18:07:50.000000000","message":"Done","commit_id":"00d4f7a1805953d678d1e1a1eba9616e552e2420"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"56de7a6cf0e925d5939c237c6c3ebec9d503d7c2","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        sched._schedule(self.context, request_spec,"},{"line_number":167,"context_line":"                        filter_properties\u003dfilter_properties)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        num_attempts \u003d filter_properties[\u0027retry\u0027][\u0027num_attempts\u0027]"},{"line_number":170,"context_line":"        self.assertEqual(2, num_attempts)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"    def test_retry_exceeded_max_attempts(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2FTd0%3D","line":169,"in_reply_to":"AAAAM3%2F%2FUKs%3D","updated":"2013-01-29 18:15:22.000000000","message":"Please ignore my previous comment... _schedule() doesn\u0027t call _add_retry_host() actually, it\u0027s _post_select_populate_filter_properties() which is called after _schedule() that calls _add_retry_host().","commit_id":"00d4f7a1805953d678d1e1a1eba9616e552e2420"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"4d93cd8530608c0e5bb06b910904d222b9baa2c4","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Tests For Filter Scheduler."},{"line_number":17,"context_line":"\"\"\""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from cinder import context"},{"line_number":20,"context_line":"from cinder import db"},{"line_number":21,"context_line":"from cinder import exception"},{"line_number":22,"context_line":"from cinder import test"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FQWY%3D","line":19,"updated":"2013-01-31 08:17:00.000000000","message":"db is not used anywhere.","commit_id":"ed0bb358ed6726d10d3856970bc511633c5d37c5"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"a5f513076ba87b429c7b674ba5ab6736e41c85bf","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Tests For Filter Scheduler."},{"line_number":17,"context_line":"\"\"\""},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from cinder import context"},{"line_number":20,"context_line":"from cinder import db"},{"line_number":21,"context_line":"from cinder import exception"},{"line_number":22,"context_line":"from cinder import test"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FP2E%3D","line":19,"in_reply_to":"AAAAM3%2F%2FQWY%3D","updated":"2013-01-31 15:08:18.000000000","message":"Done","commit_id":"ed0bb358ed6726d10d3856970bc511633c5d37c5"}],"cinder/volume/api.py":[{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"4d93cd8530608c0e5bb06b910904d222b9baa2c4","unresolved":false,"context_lines":[{"line_number":251,"context_line":"                                             volume_ref,"},{"line_number":252,"context_line":"                                             volume_ref[\u0027host\u0027],"},{"line_number":253,"context_line":"                                             request_spec,"},{"line_number":254,"context_line":"                                             filter_properties,"},{"line_number":255,"context_line":"                                             False,"},{"line_number":256,"context_line":"                                             snapshot_id,"},{"line_number":257,"context_line":"                                             image_id)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FQXU%3D","line":254,"updated":"2013-01-31 08:17:00.000000000","message":"I\u0027d really like to see this be more obvious of what False is being used for so I don\u0027t have to look up the definition.","commit_id":"ed0bb358ed6726d10d3856970bc511633c5d37c5"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"a5f513076ba87b429c7b674ba5ab6736e41c85bf","unresolved":false,"context_lines":[{"line_number":251,"context_line":"                                             volume_ref,"},{"line_number":252,"context_line":"                                             volume_ref[\u0027host\u0027],"},{"line_number":253,"context_line":"                                             request_spec,"},{"line_number":254,"context_line":"                                             filter_properties,"},{"line_number":255,"context_line":"                                             False,"},{"line_number":256,"context_line":"                                             snapshot_id,"},{"line_number":257,"context_line":"                                             image_id)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FP68%3D","line":254,"in_reply_to":"AAAAM3%2F%2FQXU%3D","updated":"2013-01-31 15:08:18.000000000","message":"Done","commit_id":"ed0bb358ed6726d10d3856970bc511633c5d37c5"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"4d93cd8530608c0e5bb06b910904d222b9baa2c4","unresolved":false,"context_lines":[{"line_number":267,"context_line":"                                             volume_ref,"},{"line_number":268,"context_line":"                                             volume_ref[\u0027host\u0027],"},{"line_number":269,"context_line":"                                             request_spec,"},{"line_number":270,"context_line":"                                             filter_properties,"},{"line_number":271,"context_line":"                                             False,"},{"line_number":272,"context_line":"                                             snapshot_id,"},{"line_number":273,"context_line":"                                             image_id,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FQXQ%3D","line":270,"updated":"2013-01-31 08:17:00.000000000","message":"same as above","commit_id":"ed0bb358ed6726d10d3856970bc511633c5d37c5"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"a5f513076ba87b429c7b674ba5ab6736e41c85bf","unresolved":false,"context_lines":[{"line_number":267,"context_line":"                                             volume_ref,"},{"line_number":268,"context_line":"                                             volume_ref[\u0027host\u0027],"},{"line_number":269,"context_line":"                                             request_spec,"},{"line_number":270,"context_line":"                                             filter_properties,"},{"line_number":271,"context_line":"                                             False,"},{"line_number":272,"context_line":"                                             snapshot_id,"},{"line_number":273,"context_line":"                                             image_id,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAM3%2F%2FP1w%3D","line":270,"in_reply_to":"AAAAM3%2F%2FQXQ%3D","updated":"2013-01-31 15:08:18.000000000","message":"Done","commit_id":"ed0bb358ed6726d10d3856970bc511633c5d37c5"}],"cinder/volume/manager.py":[{"author":{"_account_id":4355,"name":"Avishay Traeger","email":"avishay@stratoscale.com","username":"avishay-il"},"change_message_id":"3789ada4dd299086a5234114014e2ab7e69390e1","unresolved":false,"context_lines":[{"line_number":181,"context_line":"                                           image_id)"},{"line_number":182,"context_line":"                status \u003d \u0027available\u0027"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"        if model_update:"},{"line_number":185,"context_line":"            self.db.volume_update(context, volume_ref[\u0027id\u0027], model_update)"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"        return status, cloned"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUZ0%3D","line":184,"updated":"2013-01-29 04:56:15.000000000","message":"Consider returning model_update to the calling function rather than doing the update here - I think it would be best to have _only_ the volume creation code here.","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"af11cfc7bcc77e05a99f3cafac65a4e0033c88e5","unresolved":false,"context_lines":[{"line_number":181,"context_line":"                                           image_id)"},{"line_number":182,"context_line":"                status \u003d \u0027available\u0027"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"        if model_update:"},{"line_number":185,"context_line":"            self.db.volume_update(context, volume_ref[\u0027id\u0027], model_update)"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"        return status, cloned"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FUYc%3D","line":184,"in_reply_to":"AAAAM3%2F%2FUZ0%3D","updated":"2013-01-29 05:32:10.000000000","message":"Yeah, that\u0027s a good idea to isolate DB exception from driver one.  Will do.","commit_id":"d49c74f877bfe4878fce3be3db6555790af42cd1"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25baf66fe5cc4a5bdb2516b9f2d60f095a6b426f","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        cloned \u003d None"},{"line_number":156,"context_line":"        model_update \u003d False"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"        if all(x is None for x in(snapshot_ref, image_location, srcvol_ref)):"},{"line_number":159,"context_line":"            model_update \u003d self.driver.create_volume(volume_ref)"},{"line_number":160,"context_line":"        elif snapshot_ref is not None:"},{"line_number":161,"context_line":"            model_update \u003d self.driver.create_volume_from_snapshot("}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2B0kM%3D","line":158,"updated":"2013-02-18 16:09:45.000000000","message":"image_id was here previously.","commit_id":"d17cc23c648f98d0764748e9007ff4b8742e6572"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25baf66fe5cc4a5bdb2516b9f2d60f095a6b426f","unresolved":false,"context_lines":[{"line_number":169,"context_line":"            cloned \u003d self.driver.clone_image(volume_ref, image_location)"},{"line_number":170,"context_line":"            if not cloned:"},{"line_number":171,"context_line":"                model_update \u003d self.driver.create_volume(volume_ref)"},{"line_number":172,"context_line":"                #copy the image onto the volume."},{"line_number":173,"context_line":"                status \u003d \u0027downloading\u0027"},{"line_number":174,"context_line":"                self.db.volume_update(context,"},{"line_number":175,"context_line":"                                      volume_ref[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2B0kA%3D","line":172,"updated":"2013-02-18 16:09:45.000000000","message":"We should update the db with model_update.","commit_id":"d17cc23c648f98d0764748e9007ff4b8742e6572"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"2f8fa0fe741bf435d59c0c9839c2454f77df0950","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                                                          volume_ref[\u0027id\u0027],"},{"line_number":269,"context_line":"                                                          snapshot_id)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        if image_id and not cloned:"},{"line_number":272,"context_line":"            if image_meta:"},{"line_number":273,"context_line":"                # Copy all of the Glance image properties to the"},{"line_number":274,"context_line":"                # volume_glance_metadata table for future reference."}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2BuuI%3D","line":271,"updated":"2013-02-20 15:28:45.000000000","message":"cloned might not be defined here, if an exception happened above:\nTraceback (most recent call last):\n  File \"/opt/stack/cinder/cinder/openstack/common/rpc/amqp.py\", line 275, in _process_data\n    rval \u003d self.proxy.dispatch(ctxt, version, method, **args)\n  File \"/opt/stack/cinder/cinder/openstack/common/rpc/dispatcher.py\", line 145, in dispatch\n    return getattr(proxyobj, method)(ctxt, **kwargs)\n  File \"/opt/stack/cinder/cinder/volume/manager.py\", line 281, in create_volume\n    if image_id and not cloned:\nUnboundLocalError: local variable \u0027cloned\u0027 referenced before assignment","commit_id":"d17cc23c648f98d0764748e9007ff4b8742e6572"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"2f8fa0fe741bf435d59c0c9839c2454f77df0950","unresolved":false,"context_lines":[{"line_number":300,"context_line":"        type_, value, tb \u003d exc_info"},{"line_number":301,"context_line":"        LOG.error(_(\u0027Error: %s\u0027) %"},{"line_number":302,"context_line":"                  traceback.format_exception(type_, value, tb),"},{"line_number":303,"context_line":"                  volume_id\u003dvolume_id)"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"    def _reschedule_or_reraise(self, context, volume_id, exc_info,"},{"line_number":306,"context_line":"                               snapshot_id, image_id, request_spec,"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2BuuA%3D","line":303,"updated":"2013-02-20 15:28:45.000000000","message":"This exception handling is totally broken:\n[-] Exception during message handling\nTraceback (most recent call last):\n  File \"/opt/stack/cinder/cinder/openstack/common/rpc/amqp.py\", line 275, in _process_data\n    rval \u003d self.proxy.dispatch(ctxt, version, method, **args)\n  File \"/opt/stack/cinder/cinder/openstack/common/rpc/dispatcher.py\", line 145, in dispatch\n    return getattr(proxyobj, method)(ctxt, **kwargs)\n  File \"/opt/stack/cinder/cinder/volume/manager.py\", line 273, in create_volume\n    volume_ref[\u0027id\u0027], {\u0027status\u0027: \u0027error\u0027})\n  File \"/usr/lib/python2.7/contextlib.py\", line 24, in __exit__\n    self.gen.next()\n  File \"/opt/stack/cinder/cinder/volume/manager.py\", line 254, in create_volume\n    allow_reschedule)\n  File \"/opt/stack/cinder/cinder/volume/manager.py\", line 343, in _reschedule_or_reraise\n    self._log_original_error(exc_info, volume_id)\n  File \"/opt/stack/cinder/cinder/volume/manager.py\", line 313, in _log_original_error\n    volume_id\u003dvolume_id)\n  File \"/usr/lib/python2.7/logging/__init__.py\", line 1436, in error\n    self.logger.error(msg, *args, **kwargs)\n  File \"/usr/lib/python2.7/logging/__init__.py\", line 1166, in error\n    self._log(ERROR, msg, args, **kwargs)\nTypeError: _log() got an unexpected keyword argument \u0027volume_id\u0027","commit_id":"d17cc23c648f98d0764748e9007ff4b8742e6572"},{"author":{"_account_id":5044,"name":"Mate Lakat","email":"mlakat@suse.com","username":"mate-lakat"},"change_message_id":"25baf66fe5cc4a5bdb2516b9f2d60f095a6b426f","unresolved":false,"context_lines":[{"line_number":325,"context_line":""},{"line_number":326,"context_line":"        except Exception:"},{"line_number":327,"context_line":"            rescheduled \u003d False"},{"line_number":328,"context_line":"            LOG.exception(_(\"Error trying to reschedule\"),"},{"line_number":329,"context_line":"                          volume_id\u003dvolume_id)"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"        if rescheduled:"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAM3%2F%2B0kU%3D","line":328,"updated":"2013-02-18 16:09:45.000000000","message":"TypeError: _log() got an unexpected keyword argument \u0027volume_id\u0027","commit_id":"d17cc23c648f98d0764748e9007ff4b8742e6572"}]}