)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"db33eb0088623eb0170e57c705c9a564df8badcc","unresolved":true,"context_lines":[{"line_number":7,"context_line":"scheduler: add migration_ignore_scheduler config option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a new boolean config option migration_ignore_scheduler (default"},{"line_number":10,"context_line":"False) to scheduler_driver_opts. When enabled, migrate_share_to_host"},{"line_number":11,"context_line":"bypasses host_passes_filters validation and uses the destination host"},{"line_number":12,"context_line":"directly, allowing migrations to proceed even when filters such as"},{"line_number":13,"context_line":"CapacityFilter would normally reject the target."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"85858d9d_7b824f6e","line":10,"range":{"start_line":10,"start_character":47,"end_line":10,"end_character":68},"updated":"2026-06-08 13:08:49.000000000","message":"migration_ignore_scheduler instead?\n\nAlso, can we may be rename to something like force_migrate_share_to_host so that it ensures user ensures that this is a force migration which will ignore the scheduling. I am just worried this may be misused to bypass the scheduling.\n\n@gouthampravi@gmail.com, @ces.eduardo98@gmail.com can weigh in on this behaviour may be?","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"34e498e6874aafbb148e61b8177991746f8ccdb2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"scheduler: add migration_ignore_scheduler config option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Adds a new boolean config option migration_ignore_scheduler (default"},{"line_number":10,"context_line":"False) to scheduler_driver_opts. When enabled, migrate_share_to_host"},{"line_number":11,"context_line":"bypasses host_passes_filters validation and uses the destination host"},{"line_number":12,"context_line":"directly, allowing migrations to proceed even when filters such as"},{"line_number":13,"context_line":"CapacityFilter would normally reject the target."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"80297705_b9275b31","line":10,"range":{"start_line":10,"start_character":47,"end_line":10,"end_character":68},"in_reply_to":"85858d9d_7b824f6e","updated":"2026-06-09 14:22:10.000000000","message":"\u003emigration_ignore_scheduler instead?\nYes, this is config option name only.\n\n\u003emigrate_share_to_host\nthis is function and not the config option. I dont think we need to rename that functon.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"0ef511a9e6eb384515eb22cd39ca83354c4ad8e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"39ecff3a_9f4de62e","updated":"2026-08-27 10:53:29.000000000","message":"Added option to migration start API.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"fd2103c246637c29e2a8b6ca360261d37984cb06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2136ba91_9ee5443d","updated":"2026-07-14 14:07:54.000000000","message":"Goutham/Carloss,\nCan you please review this PR ?","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"c2fa012e3cd9f8dc950ccab4920ccdd7d463d745","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"2927f085_fa9a2e21","updated":"2026-05-14 07:13:52.000000000","message":"I am wondering if there is a better way to solve this issue because we are bypassing scheduler.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"f1b002643f34e785cc6005fbaa106e7c024a7a5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8dffe5db_0587f27d","updated":"2026-08-26 17:18:11.000000000","message":"Resetting my +2 as Goutham has a valid argument that needs resolving.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"eca767bc626e34303b7fc6888dd83b7b8a817689","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4f24428_19c40bd1","updated":"2026-06-18 16:19:16.000000000","message":"The patch looks good to me, but it also needs documentation and call out risks of bypassing the scheduler so that the options are used with due diligence.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"23fba5a2c4dc4095afa577411d683a7bd507b044","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"796aa9c0_34a198c6","in_reply_to":"2927f085_fa9a2e21","updated":"2026-05-14 07:39:22.000000000","message":"This is simple approach otherwise needs lot of changes across backends. The default is False.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"08eac4089fbc285b0eae9497b358f46c74985556","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"7b61ced2_6925668e","updated":"2026-09-07 07:37:06.000000000","message":"recheck","commit_id":"12d4d4f1a0bb05770c69cee24e2f4e47ef9a4463"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"49cdfe30b10406ba11a277927564a7afef28255f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ea0fd824_9089b324","updated":"2026-08-31 09:49:29.000000000","message":"recheck","commit_id":"12d4d4f1a0bb05770c69cee24e2f4e47ef9a4463"}],"manila/scheduler/drivers/base.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":"c2fa012e3cd9f8dc950ccab4920ccdd7d463d745","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"d768ba23_8236b80f","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"updated":"2026-05-14 07:13:52.000000000","message":"Isn\u0027t this risky since the CapacityFilter was supposed to avoid any overprovisioning on any backend. If allowed, it may overwhelm the destination site with more capacity expectations and also end up creating issues on the destination site which was supposed to be filtered and user would expect the filter to work?","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"f6e66d8d0583f155df48636e5b89bb91b5812ef5","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ba7bcc30_09d7f37e","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"in_reply_to":"09df0174_6076f08a","updated":"2026-07-14 15:03:28.000000000","message":"The configuration option is intentionally a \"giant knob\" because the problem it solves is deployment-wide. We are migrating share with provided destination-host, so intention is to get that host only no matter what. API flag is also acceptable, but I dont think new microversion + client changes + documentation and all is needed for such small thing. with default as False, its pretty much used where such deployment are known.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"5515d38e4e0b11382f379ba14e92199178f577d3","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"09df0174_6076f08a","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"in_reply_to":"3ee51da7_3e71f78e","updated":"2026-07-14 14:17:11.000000000","message":"Why not introduce a flag instead in the API to do this on a per share basis?\n\nThis doesn\u0027t seem useful as a giant knob that you\u0027ll have to enable/disable based on what you\u0027re trying to achieve","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":38059,"name":"Anoop Kumar Shukla","display_name":"Anoop Shukla","email":"anoop.shukla@netapp.com","username":"anoop2","status":"NetApp"},"change_message_id":"f42c7610c89d810d201f3e482c5d2c2a75f54435","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3ee51da7_3e71f78e","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"in_reply_to":"60c04881_5962f026","updated":"2026-06-08 13:09:18.000000000","message":"Acknowledged.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1a93c708657fa900f572e59e78f1e24c7fd59a29","unresolved":true,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"d5fa638e_a82c88c0","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"in_reply_to":"ba7bcc30_09d7f37e","updated":"2026-07-19 06:13:36.000000000","message":"\u003e because the problem it solves is deployment-wide. \n\nI disagree with this point; A large deployment like the one you work on has lots of shares and backends, I can\u0027t imagine your solution to one overloaded backend is to force every migration to skip the scheduler. \n\nI don\u0027t think \"skipping the scheduler\" a small thing; When we designed migration, we initially designed it without the scheduler, and then, rewrote the design to work with the scheduler because we realized the pitfall of designing migrations around the scheduler - the lack of validations.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"0ef511a9e6eb384515eb22cd39ca83354c4ad8e9","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"84808468_ebc021be","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"in_reply_to":"d5fa638e_a82c88c0","updated":"2026-08-27 10:53:29.000000000","message":"Done","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"23fba5a2c4dc4095afa577411d683a7bd507b044","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    cfg.BoolOpt(\u0027migration_ignore_scheduler\u0027,"},{"line_number":39,"context_line":"                default\u003dFalse,"},{"line_number":40,"context_line":"                help\u003d\u0027If True, skip scheduler filter checks when validating \u0027"},{"line_number":41,"context_line":"                     \u0027the destination host for share migration. This allows \u0027"},{"line_number":42,"context_line":"                     \u0027migrations to proceed even when the destination host \u0027"},{"line_number":43,"context_line":"                     \u0027would normally be rejected by filters such as \u0027"},{"line_number":44,"context_line":"                     \u0027CapacityFilter. Useful for manual load balancing in \u0027"},{"line_number":45,"context_line":"                     \u0027heavily utilized environments.\u0027),"},{"line_number":46,"context_line":"]"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"60c04881_5962f026","line":44,"range":{"start_line":41,"start_character":64,"end_line":44,"end_character":36},"in_reply_to":"d768ba23_8236b80f","updated":"2026-05-14 07:39:22.000000000","message":"We are intentionaly doing this to avoid CapactiyFilter, the use case is to manually rebalance pools specially during filer decommissioning. This is not recommended to use for production deloyment, so admin must be careful enough to deploy such config as its applicable for manual rebalancing. The default is False, so we are good to consider the migration with this option enabled is just an individual use-case based deployment and issues of overprovisioning will not be considered as bug.","commit_id":"4779fdbd349c67b962caa9c56df325b5e4764a71"}]}