)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dc597d77788f79e912e18c14309cfe61e9a80848","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"978977d0_f75efc81","updated":"2021-10-12 21:01:52.000000000","message":"LGTM and PowerMax CI is green.","commit_id":"28e7a46afea688bd9639fbfc0892f296e3bf1252"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"22f4df99d848c1c709877a13ead2a32540c34d18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d5bc744_4b9b4829","updated":"2021-10-13 17:43:52.000000000","message":"Passing CI.  Looks ok.","commit_id":"28e7a46afea688bd9639fbfc0892f296e3bf1252"},{"author":{"_account_id":19904,"name":"Digvijay Ukirde","email":"digvijay.ukirde@in.ibm.com","username":"digvijay2016"},"change_message_id":"4183cc668b7fa1fa7bf6f65bb8c6a8b7b090a836","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bc24bccf_a0849a02","updated":"2021-10-13 03:53:57.000000000","message":"run ibmgpfs-ci","commit_id":"28e7a46afea688bd9639fbfc0892f296e3bf1252"}],"cinder/volume/drivers/dell_emc/powermax/common.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6ac63c450a20b2779b8b956aad14c22ab321927f","unresolved":true,"context_lines":[{"line_number":4250,"context_line":"                if rep_config:"},{"line_number":4251,"context_line":"                    port_group_name \u003d rep_config.get(\u0027portgroup\u0027)"},{"line_number":4252,"context_line":"                port_group_label \u003d self.utils.get_port_name_label("},{"line_number":4253,"context_line":"                    port_group_name, self.powermax_port_group_name_template)"},{"line_number":4254,"context_line":"            elif volume.attach_status \u003d\u003d \u0027attached\u0027 and not remote:"},{"line_number":4255,"context_line":"                is_attached_vol \u003d True"},{"line_number":4256,"context_line":"                port_group_label \u003d self.utils.get_port_name_label("}],"source_content_type":"text/x-python","patch_set":2,"id":"155072ff_1ce77045","line":4253,"range":{"start_line":4253,"start_character":20,"end_line":4253,"end_character":35},"updated":"2021-09-15 16:37:17.000000000","message":"The utils can handle a None value here, but i think if line 4251 isn\u0027t executed, you\u0027ll get a NameError here when you make the call (unless it\u0027s already been used somewhere above, but i don\u0027t see it).","commit_id":"3c28be75c7ba6c620b9c7f0223d4a044cadb273c"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"ed3235ab471c9ffa344fd3c1d2d127d0abc120cf","unresolved":true,"context_lines":[{"line_number":4250,"context_line":"                if rep_config:"},{"line_number":4251,"context_line":"                    port_group_name \u003d rep_config.get(\u0027portgroup\u0027)"},{"line_number":4252,"context_line":"                port_group_label \u003d self.utils.get_port_name_label("},{"line_number":4253,"context_line":"                    port_group_name, self.powermax_port_group_name_template)"},{"line_number":4254,"context_line":"            elif volume.attach_status \u003d\u003d \u0027attached\u0027 and not remote:"},{"line_number":4255,"context_line":"                is_attached_vol \u003d True"},{"line_number":4256,"context_line":"                port_group_label \u003d self.utils.get_port_name_label("}],"source_content_type":"text/x-python","patch_set":2,"id":"79b1f11b_a8ff0e91","line":4253,"range":{"start_line":4253,"start_character":20,"end_line":4253,"end_character":35},"in_reply_to":"155072ff_1ce77045","updated":"2021-09-15 17:03:25.000000000","message":"Sure, I will look into this.","commit_id":"3c28be75c7ba6c620b9c7f0223d4a044cadb273c"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"c7e0045c67cd7b8ddef780ad63a2876a5aebafd7","unresolved":false,"context_lines":[{"line_number":4250,"context_line":"                if rep_config:"},{"line_number":4251,"context_line":"                    port_group_name \u003d rep_config.get(\u0027portgroup\u0027)"},{"line_number":4252,"context_line":"                port_group_label \u003d self.utils.get_port_name_label("},{"line_number":4253,"context_line":"                    port_group_name, self.powermax_port_group_name_template)"},{"line_number":4254,"context_line":"            elif volume.attach_status \u003d\u003d \u0027attached\u0027 and not remote:"},{"line_number":4255,"context_line":"                is_attached_vol \u003d True"},{"line_number":4256,"context_line":"                port_group_label \u003d self.utils.get_port_name_label("}],"source_content_type":"text/x-python","patch_set":2,"id":"3bcaae95_57c6873b","line":4253,"range":{"start_line":4253,"start_character":20,"end_line":4253,"end_character":35},"in_reply_to":"79b1f11b_a8ff0e91","updated":"2021-10-12 10:41:54.000000000","message":"Done","commit_id":"3c28be75c7ba6c620b9c7f0223d4a044cadb273c"}]}
