)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"d70df696ee4b4687cee9bb77ef5fe39ea7262689","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1bdb85aa_b20d524a","updated":"2026-05-08 08:53:01.000000000","message":"LGTM","commit_id":"1d7b0671ff2458dcebf5d788657610a1c986fdc5"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"ff110e87aa2daf566ed54329a7b8d3a03930bd04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"081e6070_bcc96680","updated":"2026-05-08 14:18:14.000000000","message":"I had a \"merge bug\" and have fixed it.  Zuul should pass now.","commit_id":"b51082a6b75e09d2f65aa4d2e4b365a6fb69d161"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"f9f868418e64f5b1525f80c1ca9db1db569b0373","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"60847bb6_a996d615","in_reply_to":"081e6070_bcc96680","updated":"2026-05-08 19:17:28.000000000","message":"LOL, of course there was a second one.","commit_id":"b51082a6b75e09d2f65aa4d2e4b365a6fb69d161"}],"cinder/volume/drivers/hitachi/hbsd_common.py":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"06e523a562ae81281189d171b815e87e88d9251e","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        min\u003d1, max\u003d600,"},{"line_number":130,"context_line":"        help\u003d\u0027Interval in seconds to check asynchronous copying status during \u0027"},{"line_number":131,"context_line":"             \u0027a copy pair deletion or data restoration.\u0027),"},{"line_number":132,"context_line":"    cfg.BoolOpt("},{"line_number":133,"context_line":"        \u0027hitachi_manage_drs_volumes\u0027,"},{"line_number":134,"context_line":"        default\u003dTrue,"},{"line_number":135,"context_line":"        help\u003d\u0027If true, the driver will create a driver managed vClone parent \u0027"},{"line_number":136,"context_line":"             \u0027for each non-cloned DRS volume it creates. It is recommended \u0027"},{"line_number":137,"context_line":"             \u0027to keep this value True to avoid issues with volume expansion \u0027"},{"line_number":138,"context_line":"             \u0027while using DRS volumes.\u0027),"},{"line_number":139,"context_line":"    cfg.BoolOpt("},{"line_number":140,"context_line":"        \u0027hitachi_use_drs_volumes\u0027,"},{"line_number":141,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":5,"id":"3c0a9e1b_590bf626","line":138,"range":{"start_line":132,"start_character":3,"end_line":138,"end_character":41},"updated":"2026-05-07 12:54:17.000000000","message":"How does this affect the upgrade scenario? Will the customers be mandated to add this value or are we initializing it as True during driver/client initialization?","commit_id":"ffb2eb23581c2263f16369d4f232d8ace0458ba0"},{"author":{"_account_id":38081,"name":"Anthony Galica","display_name":"agalica","email":"anthony.galica@hitachivantara.com","username":"agalica","status":"Hitachi Vantara"},"change_message_id":"7540b3b0eb5c0db3e9c4f5731bc5f64b2962f544","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        min\u003d1, max\u003d600,"},{"line_number":130,"context_line":"        help\u003d\u0027Interval in seconds to check asynchronous copying status during \u0027"},{"line_number":131,"context_line":"             \u0027a copy pair deletion or data restoration.\u0027),"},{"line_number":132,"context_line":"    cfg.BoolOpt("},{"line_number":133,"context_line":"        \u0027hitachi_manage_drs_volumes\u0027,"},{"line_number":134,"context_line":"        default\u003dTrue,"},{"line_number":135,"context_line":"        help\u003d\u0027If true, the driver will create a driver managed vClone parent \u0027"},{"line_number":136,"context_line":"             \u0027for each non-cloned DRS volume it creates. It is recommended \u0027"},{"line_number":137,"context_line":"             \u0027to keep this value True to avoid issues with volume expansion \u0027"},{"line_number":138,"context_line":"             \u0027while using DRS volumes.\u0027),"},{"line_number":139,"context_line":"    cfg.BoolOpt("},{"line_number":140,"context_line":"        \u0027hitachi_use_drs_volumes\u0027,"},{"line_number":141,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7d6b6345_86b06f0d","line":138,"range":{"start_line":132,"start_character":3,"end_line":138,"end_character":41},"in_reply_to":"3c0a9e1b_590bf626","updated":"2026-05-07 13:07:39.000000000","message":"Hi Anoop!\n\nThanks for bringing this up as it\u0027s an important area to scrutinize.\n\nThe way this option works is that you can freely toggle it on and off -- that\u0027s not likely to occur, but volumes created with it set to True are completely compatible with volumes created with it set to False.  The driver uses a special naming tag to know whether or not it owns a \"hidden\" parent volume, so it\u0027s completely safe to change and there are no upgrade issues.  \n\nThe original reason we defaulted this to False is because it uses extra array resources.  However, there are at least two features that are made needlessly complex (deletion and expansion) with that configuration, and it would probably be both unacceptable to users and difficult to work around (though possible).  So, it was deemed to be more important to default to True.\n\nWe\u0027re still working on getting the Zuul stuff working here, as you might have noticed with the -1 verified :)","commit_id":"ffb2eb23581c2263f16369d4f232d8ace0458ba0"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"d70df696ee4b4687cee9bb77ef5fe39ea7262689","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        min\u003d1, max\u003d600,"},{"line_number":130,"context_line":"        help\u003d\u0027Interval in seconds to check asynchronous copying status during \u0027"},{"line_number":131,"context_line":"             \u0027a copy pair deletion or data restoration.\u0027),"},{"line_number":132,"context_line":"    cfg.BoolOpt("},{"line_number":133,"context_line":"        \u0027hitachi_manage_drs_volumes\u0027,"},{"line_number":134,"context_line":"        default\u003dTrue,"},{"line_number":135,"context_line":"        help\u003d\u0027If true, the driver will create a driver managed vClone parent \u0027"},{"line_number":136,"context_line":"             \u0027for each non-cloned DRS volume it creates. It is recommended \u0027"},{"line_number":137,"context_line":"             \u0027to keep this value True to avoid issues with volume expansion \u0027"},{"line_number":138,"context_line":"             \u0027while using DRS volumes.\u0027),"},{"line_number":139,"context_line":"    cfg.BoolOpt("},{"line_number":140,"context_line":"        \u0027hitachi_use_drs_volumes\u0027,"},{"line_number":141,"context_line":"        default\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":5,"id":"b3d1a810_d6dd52ca","line":138,"range":{"start_line":132,"start_character":3,"end_line":138,"end_character":41},"in_reply_to":"7d6b6345_86b06f0d","updated":"2026-05-08 08:53:01.000000000","message":"Acknowledged.","commit_id":"ffb2eb23581c2263f16369d4f232d8ace0458ba0"}]}
