)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"e8cd059561eb15b4283b58be64788cbcc18023ef","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Hao Chen \u003chao.chen1@hitachivantara.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-01-27 15:38:43 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Hitachi: ADR plus GAD"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iefe36c30af35a2ead566e1e232a09fcd7a8a746c"},{"line_number":10,"context_line":"Signed-off-by: Hao Chen \u003chao.chen1@hitachivantara.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"91746577_31e3b0ad","line":7,"updated":"2026-01-29 19:41:02.000000000","message":"This is a bit too terse, I don\u0027t know what this means, why the change is necessary, if it addresses a bug, etc...","commit_id":"593b120cfed87cb213f7ca09cfc8d89318c2bc8f"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"ad5bc529f05b0705ac536f12baa09052bccbba3c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Hao Chen \u003chao.chen1@hitachivantara.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2026-01-27 15:38:43 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Hitachi: ADR plus GAD"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iefe36c30af35a2ead566e1e232a09fcd7a8a746c"},{"line_number":10,"context_line":"Signed-off-by: Hao Chen \u003chao.chen1@hitachivantara.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"06c22d03_f072437f","line":7,"in_reply_to":"91746577_31e3b0ad","updated":"2026-01-30 18:25:12.000000000","message":"Hi Jon!\n\nI\u0027ve updated the commit message so you know what\u0027s going on.  \n\nQuick notes: \n1. We\u0027re going to add release notes today (they were forgotten)\n2. Our storage array that we use for Tempest CI is having some issues, which is why our CI has been down the last few days.  We\u0027re working on it to get it back and running ASAP.","commit_id":"593b120cfed87cb213f7ca09cfc8d89318c2bc8f"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"8542a6a68d92e708892ce58a959fe7425a23f669","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Hitachi: Allow for ADR with GAD support."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The current version of the driver does not allow GAD support when ADR"},{"line_number":10,"context_line":"is enabled for a volume (deduplication + compression) as in some"},{"line_number":11,"context_line":"scenarios it can cause a performance issue on the storage (despite"},{"line_number":12,"context_line":"being a legitimate configuration)."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9c4a2c74_4efe9d68","line":9,"updated":"2026-02-13 17:54:27.000000000","message":"Nit: Would you mind adding the full names for GAD and ADR acronyms just for this first sentence as you did for reno? I think it would help people who aren\u0027t familiar with Hitachi systems understand the goal of this patch.","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"c36cfed996a2522a860c9c4a3eb3b7c1293c4d8e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Hitachi: Allow for ADR with GAD support."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The current version of the driver does not allow GAD support when ADR"},{"line_number":10,"context_line":"is enabled for a volume (deduplication + compression) as in some"},{"line_number":11,"context_line":"scenarios it can cause a performance issue on the storage (despite"},{"line_number":12,"context_line":"being a legitimate configuration)."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"2faadc01_7840aa7d","line":9,"in_reply_to":"9c4a2c74_4efe9d68","updated":"2026-02-14 10:39:44.000000000","message":"Acknowledged","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"e8cd059561eb15b4283b58be64788cbcc18023ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cec05003_87696968","updated":"2026-01-29 19:41:02.000000000","message":"changes look fine, can we have a bit more in the commit message please.","commit_id":"593b120cfed87cb213f7ca09cfc8d89318c2bc8f"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"632e61830209c31ee14da8523c1c1d7c50bd1450","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"39787bac_c7a7f67f","updated":"2026-01-28 15:01:37.000000000","message":"recheck cinder-plugin-ceph-tempest","commit_id":"593b120cfed87cb213f7ca09cfc8d89318c2bc8f"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"ad5bc529f05b0705ac536f12baa09052bccbba3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"34f39375_0ca5191f","updated":"2026-01-30 18:25:12.000000000","message":"Update Commit message","commit_id":"95c00d41170aba6d991b0439bc590b55a43b75a5"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"8542a6a68d92e708892ce58a959fe7425a23f669","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1d5d42c4_b56bc917","updated":"2026-02-13 17:54:27.000000000","message":"I commented a few nits inline, but the code looks good. I don\u0027t see any Hitachi CI results yet. Can we confirm that it\u0027s passing the automated tests?","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"251ace4fd5111b60981632f6b8b98b168443b092","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b0c86d74_53bffe89","updated":"2026-02-11 14:04:43.000000000","message":"recheck","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"339dee47d44da11b592fd37ea954aa6b9f4d4c9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ae2bebdd_2f767a92","updated":"2026-02-10 00:54:11.000000000","message":"recheck tempest-slow-py3","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"781d2eb7dd9ca4b80322cf8982f5429dfcaf7ac7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e2f9b609_d5bb95de","updated":"2026-01-31 07:06:12.000000000","message":"recheck tempest-slow-py3","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"7ea719fc7038525de5ecf51204787da910e57d5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f118e396_d461bb15","updated":"2026-02-11 12:50:13.000000000","message":"recheck tempest-slow-py3 - this has timed out multiple times on this patch, but not others.  This patch is tiny and should not be materially different.","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"c36cfed996a2522a860c9c4a3eb3b7c1293c4d8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5fb8b16a_7a04248e","in_reply_to":"1d5d42c4_b56bc917","updated":"2026-02-14 10:39:44.000000000","message":"Yeah, we\u0027re having trouble with our CI system FC network, and it\u0027s going through layers of bureaucracy. I had thought this one made it through in the first patch set, but clearly I was wrong.  It\u0027s actively being worked on and should be back soon. \n\nI\u0027m going to speak to my colleague about getting iSCSI results uploaded in the meantime.","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"4b4358d967bb5a1daa348583881f0504fe6fd64b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"867a0d06_60286585","in_reply_to":"f118e396_d461bb15","updated":"2026-02-11 12:50:43.000000000","message":"It also worked before the only change was the commit message.","commit_id":"4349af4ff10f18fa0f992ba72691784eb1f5b873"},{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"806656b3a1ecfbb7467a3ab70731254ef1e760e4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c446772a_7f229302","updated":"2026-02-16 12:13:53.000000000","message":"+1 code looks good. I will check tempest output once it is posted in comments.\n\nNot a issue but regarding the commit title and description, I would suggest removing the full names from title and keep them just in the description.","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ef4df45028c94adaeb807da4adaee56709f7d789","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6db4f8da_39bba806","updated":"2026-02-26 17:15:24.000000000","message":"Code change looks OK, but I wonder whether there should be a new driver version for this, or piggyback on one of the other new versions so that you can say in the release note, beginning with driver version X.Y, the driver no longer prevents GAD support for ADR volumes.  This seems like a big change to me, and it\u0027s not something that is transparent to an operator as far as I can tell.","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"f6bdb938447191d76452065c68fd65b0159e38fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0fb7e70a_a9613873","updated":"2026-02-17 19:45:24.000000000","message":"Respond to Fernando about CI being uplaoded.","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"c36cfed996a2522a860c9c4a3eb3b7c1293c4d8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ff8c4ed0_92c39616","updated":"2026-02-14 10:39:44.000000000","message":"Thanks for the review, Fernando.  I\u0027ve modified the commit message, and we\u0027re still working on the CI.  Hopefully iSCSI results will be up early next week, with FC to follow.","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"f7585b47f48f07f673abc46d58af111663c373cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9e0ba5ef_f8975a52","updated":"2026-02-25 12:27:43.000000000","message":"code changes look good. Hopefully zuul would pass soon.","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"1e1e36c9c6e91e3141bd09b87dff1ac9e28e0f07","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7f44a63f_50fce7ae","updated":"2026-02-25 00:04:28.000000000","message":"recheck zuul (I think it\u0027s back?)","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"91b37460e2191b0c87d05b71544df642a6db2e38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7bb0e397_499d8e93","in_reply_to":"6db4f8da_39bba806","updated":"2026-02-26 23:02:37.000000000","message":"I have published a new edit with your suggestions, thank you!","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"f6bdb938447191d76452065c68fd65b0159e38fc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"385a5262_b5673342","in_reply_to":"c446772a_7f229302","updated":"2026-02-17 19:45:24.000000000","message":"If we need another patch set and I remember I will do that.\n\nThe CI has been posted now (shortly before you made this comment, so feel free to check it out!","commit_id":"4565e831faf5a5405b5dc02da6c6e6a8357f122b"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"91b37460e2191b0c87d05b71544df642a6db2e38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ee803385_ed402acd","updated":"2026-02-26 23:02:37.000000000","message":"Update version number as per Brian\u0027s suggestion. This is one of our pain points (forgetting version bumps).  Thanks Brian :-)","commit_id":"a0bb0244e665a84a73d4f0863fd7cd6c5088843d"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"300ac909a657ee9e64555b21611b284674985ce7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5d6f5f5f_9759c05d","updated":"2026-02-26 23:30:17.000000000","message":"Revision LGTM.","commit_id":"c61eee49bfdd175fe5002f4f71cf5833fe039c33"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"14ba5187dede68b4c66d7d1be7003b3ec2a4f23e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"4ecce0b5_06da8496","updated":"2026-02-28 22:55:54.000000000","message":"Renewing my +2 after rebase.","commit_id":"66dfc1862045f0c90230a551602afc653c900300"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"0b538e2bb91119f90e1238817381bea8c5f3f602","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"0b6fec6b_53e840db","updated":"2026-02-27 22:47:15.000000000","message":"recheck cinder-plugin-ceph-tempest failed loading modules unrelated to driver","commit_id":"66dfc1862045f0c90230a551602afc653c900300"}],"cinder/volume/drivers/hitachi/hbsd_common.py":[{"author":{"_account_id":30555,"name":"Fernando Ferraz","display_name":"Fernando Ferraz","email":"fesilva@redhat.com","username":"fernandoperches"},"change_message_id":"8542a6a68d92e708892ce58a959fe7425a23f669","unresolved":true,"context_lines":[{"line_number":329,"context_line":"                           \u0027:drs\u0027)"},{"line_number":330,"context_line":"        drs \u003d extra_specs.get(extra_specs_drs)"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        extra_specs_drs \u003d (self.driver_info[\u0027driver_dir_name\u0027] +"},{"line_number":333,"context_line":"                           \u0027:drs\u0027)"},{"line_number":334,"context_line":"        drs \u003d extra_specs.get(extra_specs_drs)"},{"line_number":335,"context_line":"        extra_specs_csv \u003d (self.driver_info[\u0027driver_dir_name\u0027] +"}],"source_content_type":"text/x-python","patch_set":3,"id":"e697d4df_d3111b55","side":"PARENT","line":332,"updated":"2026-02-13 17:54:27.000000000","message":"Nit: Code is repeating itself so removing this line make sense, but it seems unrelated to the current patch. To keep changes minimal and focused, I\u0027d recommend splitting this into a separate cleanup patch.","commit_id":"798b4fc29a02737eb703d4db3e52e0a471a1726d"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"c36cfed996a2522a860c9c4a3eb3b7c1293c4d8e","unresolved":false,"context_lines":[{"line_number":329,"context_line":"                           \u0027:drs\u0027)"},{"line_number":330,"context_line":"        drs \u003d extra_specs.get(extra_specs_drs)"},{"line_number":331,"context_line":""},{"line_number":332,"context_line":"        extra_specs_drs \u003d (self.driver_info[\u0027driver_dir_name\u0027] +"},{"line_number":333,"context_line":"                           \u0027:drs\u0027)"},{"line_number":334,"context_line":"        drs \u003d extra_specs.get(extra_specs_drs)"},{"line_number":335,"context_line":"        extra_specs_csv \u003d (self.driver_info[\u0027driver_dir_name\u0027] +"}],"source_content_type":"text/x-python","patch_set":3,"id":"ce4fcbc3_6795817a","side":"PARENT","line":332,"in_reply_to":"e697d4df_d3111b55","updated":"2026-02-14 10:39:44.000000000","message":"\u003e Nit: Code is repeating itself so removing this line make sense, but it seems unrelated to the current patch. To keep changes minimal and focused, I\u0027d recommend splitting this into a separate cleanup patch.\n\nYeah, this was brought up in a prior patch and then it somehow made it back in.  It will be removed in a future patch - thanks for catching its return.","commit_id":"798b4fc29a02737eb703d4db3e52e0a471a1726d"}]}