)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3c177877abdeae00cbf93bc62024a492029a3ec8","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Cinder volume driver for Veritas HyperScale currently stores product"},{"line_number":10,"context_line":"specific metadata in Cinder database."},{"line_number":11,"context_line":"This bug is to remove dependency from Cinder database and store"},{"line_number":12,"context_line":"required metadata in product specific database."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1749719"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa0c359_4cdd8f6e","line":12,"updated":"2018-02-20 16:41:12.000000000","message":"Can you give more info here about how this works and where it is stored instead?","commit_id":"0d504fe66f23427029d1204e57d6ec648ca3e283"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"39e0ba9b2b52a5602b6ef80a0b7442560c68f230","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Cinder volume driver for Veritas HyperScale currently stores product"},{"line_number":10,"context_line":"specific metadata in Cinder database."},{"line_number":11,"context_line":"This bug is to remove dependency from Cinder database and store"},{"line_number":12,"context_line":"required metadata in product specific database."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1749719"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa0c359_6e9038ee","line":12,"in_reply_to":"3fa0c359_4cdd8f6e","updated":"2018-02-23 07:36:44.000000000","message":"HyperScale will have this information in its own database instead of depending on cinder database. HyperScale will refer this metadata as part of the backend call from HyperScale Cinder driver.","commit_id":"0d504fe66f23427029d1204e57d6ec648ca3e283"}],"cinder/volume/drivers/veritas/utils.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"db0be3cf2cbf5256e7dcb743d7d5626dd015d067","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import json"},{"line_number":16,"context_line":"import six"},{"line_number":17,"context_line":"import uuid"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from oslo_concurrency import processutils as putils"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_2f609ace","line":16,"updated":"2018-02-28 15:42:07.000000000","message":"six import should be in the section with other 3rd party libraries","commit_id":"9f817466bc5cc22aa30c5ed2c7580cd3420dc96f"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"0021b662704359c65ddf20d3f2bfd2830f72adf2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import json"},{"line_number":16,"context_line":"import six"},{"line_number":17,"context_line":"import uuid"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from oslo_concurrency import processutils as putils"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_d4b73e86","line":16,"in_reply_to":"1f9dbf25_2f609ace","updated":"2018-03-01 09:34:23.000000000","message":"Done","commit_id":"9f817466bc5cc22aa30c5ed2c7580cd3420dc96f"}],"cinder/volume/drivers/veritas/vrtshyperscale.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"d57dede59f367dd4b097be846f9518e55c575d7b","unresolved":false,"context_lines":[{"line_number":44,"context_line":"HYPERSCALE_VERSION \u003d \u00271.0.0\u0027"},{"line_number":45,"context_line":"PROVIDER_LOCATION_MNT \u003d \"/hyperscale\""},{"line_number":46,"context_line":"PROVIDER_LOCATION \u003d \u0027hyperscale-sv:\u0027 + PROVIDER_LOCATION_MNT"},{"line_number":47,"context_line":"g_images_lock \u003d threading.Lock()"},{"line_number":48,"context_line":"g_cached_images \u003d {}"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"@interface.volumedriver"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_7182d7ca","line":48,"range":{"start_line":47,"start_character":0,"end_line":48,"end_character":20},"updated":"2018-02-17 09:57:18.000000000","message":"These variables should be named in upper-case notation","commit_id":"23fb4fbf06854cbe4e50816814966997b9ead159"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"b21a352baedbbddd2f3f43ad1230803e6d64b2fa","unresolved":false,"context_lines":[{"line_number":44,"context_line":"HYPERSCALE_VERSION \u003d \u00271.0.0\u0027"},{"line_number":45,"context_line":"PROVIDER_LOCATION_MNT \u003d \"/hyperscale\""},{"line_number":46,"context_line":"PROVIDER_LOCATION \u003d \u0027hyperscale-sv:\u0027 + PROVIDER_LOCATION_MNT"},{"line_number":47,"context_line":"g_images_lock \u003d threading.Lock()"},{"line_number":48,"context_line":"g_cached_images \u003d {}"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"@interface.volumedriver"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_ed536038","line":48,"range":{"start_line":47,"start_character":0,"end_line":48,"end_character":20},"in_reply_to":"3fa0c359_7182d7ca","updated":"2018-02-19 05:59:59.000000000","message":"Done","commit_id":"23fb4fbf06854cbe4e50816814966997b9ead159"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"d57dede59f367dd4b097be846f9518e55c575d7b","unresolved":false,"context_lines":[{"line_number":291,"context_line":"        \"\"\"Creates a hyperscale volume.\"\"\""},{"line_number":292,"context_line":""},{"line_number":293,"context_line":"        model_update \u003d {}"},{"line_number":294,"context_line":"        LOG.debug(\"*** Create volume ***\")"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        try:"},{"line_number":297,"context_line":"            volume_metadata \u003d self._get_volume_metadata(volume)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_111cb3bf","line":294,"updated":"2018-02-17 09:57:18.000000000","message":"Please, remove \u0027*\u0027 from log message and add volume id.","commit_id":"23fb4fbf06854cbe4e50816814966997b9ead159"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"b21a352baedbbddd2f3f43ad1230803e6d64b2fa","unresolved":false,"context_lines":[{"line_number":291,"context_line":"        \"\"\"Creates a hyperscale volume.\"\"\""},{"line_number":292,"context_line":""},{"line_number":293,"context_line":"        model_update \u003d {}"},{"line_number":294,"context_line":"        LOG.debug(\"*** Create volume ***\")"},{"line_number":295,"context_line":""},{"line_number":296,"context_line":"        try:"},{"line_number":297,"context_line":"            volume_metadata \u003d self._get_volume_metadata(volume)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa0c359_8d309c09","line":294,"in_reply_to":"3fa0c359_111cb3bf","updated":"2018-02-19 05:59:59.000000000","message":"Done","commit_id":"23fb4fbf06854cbe4e50816814966997b9ead159"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3c177877abdeae00cbf93bc62024a492029a3ec8","unresolved":false,"context_lines":[{"line_number":539,"context_line":""},{"line_number":540,"context_line":"        LOG.debug(\"Get volume status\")"},{"line_number":541,"context_line":""},{"line_number":542,"context_line":"        try:"},{"line_number":543,"context_line":"            self._stats \u003d self._fetch_volume_status()"},{"line_number":544,"context_line":"            new_total \u003d self._stats[\u0027total_capacity_gb\u0027]"},{"line_number":545,"context_line":"            new_free \u003d self._stats[\u0027free_capacity_gb\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa0c359_8c7ab774","line":542,"updated":"2018-02-20 16:41:12.000000000","message":"Is this related to what\u0027s described in the commit message?","commit_id":"0d504fe66f23427029d1204e57d6ec648ca3e283"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"39e0ba9b2b52a5602b6ef80a0b7442560c68f230","unresolved":false,"context_lines":[{"line_number":539,"context_line":""},{"line_number":540,"context_line":"        LOG.debug(\"Get volume status\")"},{"line_number":541,"context_line":""},{"line_number":542,"context_line":"        try:"},{"line_number":543,"context_line":"            self._stats \u003d self._fetch_volume_status()"},{"line_number":544,"context_line":"            new_total \u003d self._stats[\u0027total_capacity_gb\u0027]"},{"line_number":545,"context_line":"            new_free \u003d self._stats[\u0027free_capacity_gb\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa0c359_ae05e047","line":542,"in_reply_to":"3fa0c359_8c7ab774","updated":"2018-02-23 07:36:44.000000000","message":"Moving of the try statement will handle any exceptions in the _fetch_volume_status().","commit_id":"0d504fe66f23427029d1204e57d6ec648ca3e283"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"3c177877abdeae00cbf93bc62024a492029a3ec8","unresolved":false,"context_lines":[{"line_number":674,"context_line":"                        \u0027service\u0027: six.text_type(image_service),"},{"line_number":675,"context_line":"                        \u0027id\u0027: six.text_type(image_id)})"},{"line_number":676,"context_line":""},{"line_number":677,"context_line":"        with G_IMAGES_LOCK:"},{"line_number":678,"context_line":"            if G_CACHED_IMAGES.get(image_id) is None:"},{"line_number":679,"context_line":"                G_CACHED_IMAGES[image_id] \u003d threading.Lock()"},{"line_number":680,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa0c359_eceedb20","line":677,"updated":"2018-02-20 16:41:12.000000000","message":"This should be using coordination.synchronized instead of a custom threading.Lock.","commit_id":"0d504fe66f23427029d1204e57d6ec648ca3e283"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"39e0ba9b2b52a5602b6ef80a0b7442560c68f230","unresolved":false,"context_lines":[{"line_number":674,"context_line":"                        \u0027service\u0027: six.text_type(image_service),"},{"line_number":675,"context_line":"                        \u0027id\u0027: six.text_type(image_id)})"},{"line_number":676,"context_line":""},{"line_number":677,"context_line":"        with G_IMAGES_LOCK:"},{"line_number":678,"context_line":"            if G_CACHED_IMAGES.get(image_id) is None:"},{"line_number":679,"context_line":"                G_CACHED_IMAGES[image_id] \u003d threading.Lock()"},{"line_number":680,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa0c359_61b60941","line":677,"in_reply_to":"3fa0c359_eceedb20","updated":"2018-02-23 07:36:44.000000000","message":"Yes, we can explore some more options here, but, for now, we will revert this change and submit this particular change as part of a separate bug. Furthermore, this is not directly related to metadata changes.","commit_id":"0d504fe66f23427029d1204e57d6ec648ca3e283"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"db0be3cf2cbf5256e7dcb743d7d5626dd015d067","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"},{"line_number":21,"context_line":"from oslo_log import log as logging"},{"line_number":22,"context_line":"from oslo_utils import excutils"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_4fae46e8","side":"PARENT","line":19,"updated":"2018-02-28 15:42:07.000000000","message":"It\u0027s not related to this change. Please, leave this line as is","commit_id":"4f778ee01ec40276e235efaa339dd00381a1fa41"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"0021b662704359c65ddf20d3f2bfd2830f72adf2","unresolved":false,"context_lines":[{"line_number":16,"context_line":"\"\"\""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import os"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from oslo_config import cfg"},{"line_number":21,"context_line":"from oslo_log import log as logging"},{"line_number":22,"context_line":"from oslo_utils import excutils"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_d49e5e06","side":"PARENT","line":19,"in_reply_to":"1f9dbf25_4fae46e8","updated":"2018-03-01 09:34:23.000000000","message":"Done","commit_id":"4f778ee01ec40276e235efaa339dd00381a1fa41"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"db0be3cf2cbf5256e7dcb743d7d5626dd015d067","unresolved":false,"context_lines":[{"line_number":402,"context_line":"            message_body[\u0027snapshot_name\u0027] \u003d snapshot[\u0027name\u0027]"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":"            if workflow_snap \u003d\u003d 1:"},{"line_number":405,"context_line":"                LOG.debug(\u0027__help request for WORKFLOW snapshot\u0027)"},{"line_number":406,"context_line":"                message_body[\u0027workflow_snapshot\u0027] \u003d 1"},{"line_number":407,"context_line":"                message_body[\u0027TYPE\u0027] \u003d TYPE_WORKFLOW_SNAP"},{"line_number":408,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_6f94628c","line":405,"updated":"2018-02-28 15:42:07.000000000","message":"Do we really need underscores here?","commit_id":"9f817466bc5cc22aa30c5ed2c7580cd3420dc96f"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"0021b662704359c65ddf20d3f2bfd2830f72adf2","unresolved":false,"context_lines":[{"line_number":402,"context_line":"            message_body[\u0027snapshot_name\u0027] \u003d snapshot[\u0027name\u0027]"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":"            if workflow_snap \u003d\u003d 1:"},{"line_number":405,"context_line":"                LOG.debug(\u0027__help request for WORKFLOW snapshot\u0027)"},{"line_number":406,"context_line":"                message_body[\u0027workflow_snapshot\u0027] \u003d 1"},{"line_number":407,"context_line":"                message_body[\u0027TYPE\u0027] \u003d TYPE_WORKFLOW_SNAP"},{"line_number":408,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_f4a35a4e","line":405,"in_reply_to":"1f9dbf25_6f94628c","updated":"2018-03-01 09:34:23.000000000","message":"Done","commit_id":"9f817466bc5cc22aa30c5ed2c7580cd3420dc96f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"db0be3cf2cbf5256e7dcb743d7d5626dd015d067","unresolved":false,"context_lines":[{"line_number":713,"context_line":""},{"line_number":714,"context_line":"            path \u003d util.get_image_path(image_id)"},{"line_number":715,"context_line":""},{"line_number":716,"context_line":"            LOG.debug(\"get_image_path: %s\", path)"},{"line_number":717,"context_line":""},{"line_number":718,"context_line":"            image_utils.upload_volume(context,"},{"line_number":719,"context_line":"                                      image_service,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_aff7caf5","line":716,"updated":"2018-02-28 15:42:07.000000000","message":"Please, add more user friendly debug message","commit_id":"9f817466bc5cc22aa30c5ed2c7580cd3420dc96f"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"0021b662704359c65ddf20d3f2bfd2830f72adf2","unresolved":false,"context_lines":[{"line_number":713,"context_line":""},{"line_number":714,"context_line":"            path \u003d util.get_image_path(image_id)"},{"line_number":715,"context_line":""},{"line_number":716,"context_line":"            LOG.debug(\"get_image_path: %s\", path)"},{"line_number":717,"context_line":""},{"line_number":718,"context_line":"            image_utils.upload_volume(context,"},{"line_number":719,"context_line":"                                      image_service,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1f9dbf25_f44a7a7e","line":716,"in_reply_to":"1f9dbf25_aff7caf5","updated":"2018-03-01 09:34:23.000000000","message":"Done","commit_id":"9f817466bc5cc22aa30c5ed2c7580cd3420dc96f"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"73b4f674747dfb2b5be1d4acc4d4df606cc264ae","unresolved":false,"context_lines":[{"line_number":521,"context_line":"                exception.UnableToProcessHyperScaleCmdOutput):"},{"line_number":522,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":523,"context_line":"                LOG.exception(\u0027Exception in creating volume from\u0027"},{"line_number":524,"context_line":"                              \u0027 snapshot\u0027)"},{"line_number":525,"context_line":"        except exception.InvalidMetadataType:"},{"line_number":526,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":527,"context_line":"                LOG.exception(\u0027Exception updating metadata in create\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f9dbf25_0d6594b5","line":524,"updated":"2018-03-01 14:53:50.000000000","message":"It looks like not related to the patch and not needed","commit_id":"8e3bdb9d912b59a344166fef148e70f4de51069a"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"ffdd9f08f79427d16b38fdf6feac74bb9d740310","unresolved":false,"context_lines":[{"line_number":521,"context_line":"                exception.UnableToProcessHyperScaleCmdOutput):"},{"line_number":522,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":523,"context_line":"                LOG.exception(\u0027Exception in creating volume from\u0027"},{"line_number":524,"context_line":"                              \u0027 snapshot\u0027)"},{"line_number":525,"context_line":"        except exception.InvalidMetadataType:"},{"line_number":526,"context_line":"            with excutils.save_and_reraise_exception():"},{"line_number":527,"context_line":"                LOG.exception(\u0027Exception updating metadata in create\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f9dbf25_70ac238c","line":524,"in_reply_to":"1f9dbf25_0d6594b5","updated":"2018-03-02 06:26:58.000000000","message":"Done","commit_id":"8e3bdb9d912b59a344166fef148e70f4de51069a"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"73b4f674747dfb2b5be1d4acc4d4df606cc264ae","unresolved":false,"context_lines":[{"line_number":675,"context_line":"                        \u0027service\u0027: six.text_type(image_service),"},{"line_number":676,"context_line":"                        \u0027id\u0027: six.text_type(image_id)})"},{"line_number":677,"context_line":""},{"line_number":678,"context_line":"        with G_IMAGES_LOCK:"},{"line_number":679,"context_line":"            if G_CACHED_IMAGES.get(image_id) is None:"},{"line_number":680,"context_line":"                G_CACHED_IMAGES[image_id] \u003d threading.Lock()"},{"line_number":681,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f9dbf25_2d1bb01b","line":678,"updated":"2018-03-01 14:53:50.000000000","message":"Can we use cinder.synchronized instead?","commit_id":"8e3bdb9d912b59a344166fef148e70f4de51069a"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"ffdd9f08f79427d16b38fdf6feac74bb9d740310","unresolved":false,"context_lines":[{"line_number":675,"context_line":"                        \u0027service\u0027: six.text_type(image_service),"},{"line_number":676,"context_line":"                        \u0027id\u0027: six.text_type(image_id)})"},{"line_number":677,"context_line":""},{"line_number":678,"context_line":"        with G_IMAGES_LOCK:"},{"line_number":679,"context_line":"            if G_CACHED_IMAGES.get(image_id) is None:"},{"line_number":680,"context_line":"                G_CACHED_IMAGES[image_id] \u003d threading.Lock()"},{"line_number":681,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f9dbf25_d09f374e","line":678,"in_reply_to":"1f9dbf25_2d1bb01b","updated":"2018-03-02 06:26:58.000000000","message":"We removed this in \"Patch Set 2\", but, mistakenly, reintroduced it in yesterday\u0027s check in. Please refer to my comments on Patch Set 2. Pasting it here again.\n\n\"Yes, we can explore some more options here, but, for now, we will revert this change and submit this particular change as part of a separate bug. Furthermore, this is not directly related to metadata changes.\"","commit_id":"8e3bdb9d912b59a344166fef148e70f4de51069a"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"87e00f7b99397bc0e3396f2d6f797a8043e2e28d","unresolved":false,"context_lines":[{"line_number":704,"context_line":""},{"line_number":705,"context_line":"        try:"},{"line_number":706,"context_line":"            vol_glance_metadata \u003d volume.volume_glance_metadata"},{"line_number":707,"context_line":"            LOG.debug(\"Volume glance metadata: %s\", str(vol_glance_metadata))"},{"line_number":708,"context_line":""},{"line_number":709,"context_line":"            for i in vol_glance_metadata:"},{"line_number":710,"context_line":"                if i[\u0027key\u0027] \u003d\u003d \u0027image_id\u0027:"}],"source_content_type":"text/x-python","patch_set":5,"id":"df7087c5_51b9aa2b","line":707,"updated":"2018-03-12 14:12:25.000000000","message":"Please, change str to six.text_type","commit_id":"a8eb0a7a0f5b64766771fe71f8761849c9383c61"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"0e59196940658950984319562c201721fe01b506","unresolved":false,"context_lines":[{"line_number":704,"context_line":""},{"line_number":705,"context_line":"        try:"},{"line_number":706,"context_line":"            vol_glance_metadata \u003d volume.volume_glance_metadata"},{"line_number":707,"context_line":"            LOG.debug(\"Volume glance metadata: %s\", str(vol_glance_metadata))"},{"line_number":708,"context_line":""},{"line_number":709,"context_line":"            for i in vol_glance_metadata:"},{"line_number":710,"context_line":"                if i[\u0027key\u0027] \u003d\u003d \u0027image_id\u0027:"}],"source_content_type":"text/x-python","patch_set":5,"id":"df7087c5_5c30c4e8","line":707,"in_reply_to":"df7087c5_51b9aa2b","updated":"2018-03-13 09:28:51.000000000","message":"Done","commit_id":"a8eb0a7a0f5b64766771fe71f8761849c9383c61"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"8d55d37f31faddfca171607414a3bd7eb52074c9","unresolved":false,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        # Get metadata for volume"},{"line_number":342,"context_line":"        metadata \u003d self._get_volume_metadata(volume)"},{"line_number":343,"context_line":"        LOG.debug(\"Delete volume with id metadata: %s \", metadata)"},{"line_number":344,"context_line":"        LOG.debug(\"Delete volume with id message_body: %s \", message_body)"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"        try:"},{"line_number":347,"context_line":"            util.message_controller("}],"source_content_type":"text/x-python","patch_set":7,"id":"df7087c5_3450edd4","line":344,"range":{"start_line":343,"start_character":7,"end_line":344,"end_character":10},"updated":"2018-03-19 19:01:15.000000000","message":"Please, add volume id to the log message","commit_id":"5ebcd27666a31942610556cb13059dbeb7cf910f"},{"author":{"_account_id":27839,"name":"atul swami","email":"atul.swami@veritas.com","username":"atulswami"},"change_message_id":"5769fd379cb7f371edd205f7735a7db31bc9bb09","unresolved":false,"context_lines":[{"line_number":340,"context_line":""},{"line_number":341,"context_line":"        # Get metadata for volume"},{"line_number":342,"context_line":"        metadata \u003d self._get_volume_metadata(volume)"},{"line_number":343,"context_line":"        LOG.debug(\"Delete volume with id metadata: %s \", metadata)"},{"line_number":344,"context_line":"        LOG.debug(\"Delete volume with id message_body: %s \", message_body)"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":"        try:"},{"line_number":347,"context_line":"            util.message_controller("}],"source_content_type":"text/x-python","patch_set":7,"id":"df7087c5_0788e7c3","line":344,"range":{"start_line":343,"start_character":7,"end_line":344,"end_character":10},"in_reply_to":"df7087c5_3450edd4","updated":"2018-03-21 11:08:19.000000000","message":"Done","commit_id":"5ebcd27666a31942610556cb13059dbeb7cf910f"}]}