)]}'
{"cinder/volume/api.py":[{"author":{"_account_id":6153,"name":"stephen-mulcahy","email":"stephen.mulcahy@hpe.com","username":"stephen-mulcahy"},"change_message_id":"00f57e79d699c4b16753abb73898885dda45f799","unresolved":false,"context_lines":[{"line_number":302,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":303,"context_line":"        topic \u003d FLAGS.volume_topic"},{"line_number":304,"context_line":"        volume_services \u003d self.db.service_get_all_by_topic(ctxt, topic)"},{"line_number":305,"context_line":"        self.availability_zones \u003d set()"},{"line_number":306,"context_line":"        found \u003d False"},{"line_number":307,"context_line":"        for service in volume_services:"},{"line_number":308,"context_line":"            if service[\u0027availability_zone\u0027] \u003d\u003d availability_zone:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FwDU%3D","line":305,"updated":"2013-06-11 12:26:27.000000000","message":"this is already initialised in _init_ above","commit_id":"03c448d2eba35be5358a2dc09cef4a96777930d3"},{"author":{"_account_id":6153,"name":"stephen-mulcahy","email":"stephen.mulcahy@hpe.com","username":"stephen-mulcahy"},"change_message_id":"00f57e79d699c4b16753abb73898885dda45f799","unresolved":false,"context_lines":[{"line_number":307,"context_line":"        for service in volume_services:"},{"line_number":308,"context_line":"            if service[\u0027availability_zone\u0027] \u003d\u003d availability_zone:"},{"line_number":309,"context_line":"                found \u003d True"},{"line_number":310,"context_line":"            self.availability_zones.add(service[\u0027availability_zone\u0027])"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"        if not found:"},{"line_number":313,"context_line":"            msg \u003d _(\"Availability zone is unvalid\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FwDQ%3D","line":310,"updated":"2013-06-11 12:26:27.000000000","message":"consider just adding this availability_zone to the list and returning if service[\u0027availability_zone\u0027] \u003d\u003d availability_zone, eliminating need for found flag","commit_id":"03c448d2eba35be5358a2dc09cef4a96777930d3"},{"author":{"_account_id":6153,"name":"stephen-mulcahy","email":"stephen.mulcahy@hpe.com","username":"stephen-mulcahy"},"change_message_id":"5b9578920248794c2c2be4565369a5e1fb25328a","unresolved":false,"context_lines":[{"line_number":310,"context_line":"            self.availability_zones.add(service[\u0027availability_zone\u0027])"},{"line_number":311,"context_line":""},{"line_number":312,"context_line":"        if availability_zone not in self.availability_zones:"},{"line_number":313,"context_line":"            msg \u003d _(\"Availability zone is unvalid\")"},{"line_number":314,"context_line":"            LOG.warn(msg)"},{"line_number":315,"context_line":"            raise exception.InvalidInput(reason\u003dmsg)"},{"line_number":316,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPX%2F%2Fv%2BA%3D","line":313,"updated":"2013-06-11 13:19:58.000000000","message":"spelling: invalid (not unvalid)","commit_id":"7123265cd75bc1ab3596662140b12fda3ba0ed19"}]}