)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28486,"name":"Tee Ngo","email":"tee.ngo@windriver.com","username":"teewrs"},"change_message_id":"a20c0bf5b6a9dbbada6dd86b0f4e04d7f17418dd","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://opendev.org/starlingx/stx-puppet/src/branch/master/puppet-manifests/src/modules/platform/manifests/filesystem.pp#L146"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Test Plan:"},{"line_number":24,"context_line":"PASS: Execute controllerfs-modify"},{"line_number":25,"context_line":"      Execute host-fs-modify when status\u003ddrbd_fs_resizing_in_progress"},{"line_number":26,"context_line":"      Verify that system blocks the host-fs-modify command"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"aedb2558_cee24b3d","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":10},"updated":"2022-04-07 21:24:38.000000000","message":"B\u0026R and Subcloud upgrade tests are required.","commit_id":"a4550209ae221454cd67d1b8a20e97e6aca0ab10"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"69ae6600eb2b9a923e98102a39e3895b2049ef01","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://opendev.org/starlingx/stx-puppet/src/branch/master/puppet-manifests/src/modules/platform/manifests/filesystem.pp#L146"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Test Plan:"},{"line_number":24,"context_line":"PASS: Execute controllerfs-modify"},{"line_number":25,"context_line":"      Execute host-fs-modify when status\u003ddrbd_fs_resizing_in_progress"},{"line_number":26,"context_line":"      Verify that system blocks the host-fs-modify command"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"8f19a09e_450f70bd","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":10},"in_reply_to":"28c1ecf0_77de4b50","updated":"2022-04-11 13:30:35.000000000","message":"Done, thanks Tee.","commit_id":"a4550209ae221454cd67d1b8a20e97e6aca0ab10"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"19e25e74607fa7caecab3c79cda8a3c790e8b78b","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://opendev.org/starlingx/stx-puppet/src/branch/master/puppet-manifests/src/modules/platform/manifests/filesystem.pp#L146"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Test Plan:"},{"line_number":24,"context_line":"PASS: Execute controllerfs-modify"},{"line_number":25,"context_line":"      Execute host-fs-modify when status\u003ddrbd_fs_resizing_in_progress"},{"line_number":26,"context_line":"      Verify that system blocks the host-fs-modify command"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"28c1ecf0_77de4b50","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":10},"in_reply_to":"419302ea_248c4d9b","updated":"2022-04-11 13:28:05.000000000","message":"Thanks, LP was updated with the investigation. I\u0027ll let this comment unresolved until Tee provides feedback about the Test Plan","commit_id":"a4550209ae221454cd67d1b8a20e97e6aca0ab10"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"a4c3d01c9bbafad53fc7438cdcc112d4ed1fb4af","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://opendev.org/starlingx/stx-puppet/src/branch/master/puppet-manifests/src/modules/platform/manifests/filesystem.pp#L146"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Test Plan:"},{"line_number":24,"context_line":"PASS: Execute controllerfs-modify"},{"line_number":25,"context_line":"      Execute host-fs-modify when status\u003ddrbd_fs_resizing_in_progress"},{"line_number":26,"context_line":"      Verify that system blocks the host-fs-modify command"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fbd8849e_9b2dfac3","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":10},"in_reply_to":"aedb2558_cee24b3d","updated":"2022-04-08 19:14:54.000000000","message":"Is the Subcloud Upgrade test really needed?","commit_id":"a4550209ae221454cd67d1b8a20e97e6aca0ab10"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"ee800318be877a8c6a3152f5a66d5e5d59fc7efa","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"[1] https://opendev.org/starlingx/stx-puppet/src/branch/master/puppet-manifests/src/modules/platform/manifests/filesystem.pp#L146"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Test Plan:"},{"line_number":24,"context_line":"PASS: Execute controllerfs-modify"},{"line_number":25,"context_line":"      Execute host-fs-modify when status\u003ddrbd_fs_resizing_in_progress"},{"line_number":26,"context_line":"      Verify that system blocks the host-fs-modify command"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"419302ea_248c4d9b","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":10},"in_reply_to":"fbd8849e_9b2dfac3","updated":"2022-04-08 21:14:26.000000000","message":"Based on the analysis performed as part of resolution of this bug, the issue was that the config change was allowed prior to the upgrade when it shouldn\u0027t have been.  It may also help to add additional relevant investigation notes into the LP.  Code changes look ok.  Will update based upon Tee\u0027s confirmation.","commit_id":"a4550209ae221454cd67d1b8a20e97e6aca0ab10"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"eb9ea36af58da4692ee80c98d033feaca2abcecc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ca53ffa6_8d4b2657","updated":"2022-04-07 21:17:29.000000000","message":"Marking as WIP to test changes suggested on the comments","commit_id":"e833a1a14702d135437d1865d219f457fdf140ba"}],"sysinv/sysinv/sysinv/sysinv/api/controllers/v1/host_fs.py":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"8794d76e677b0c361116b6a391060f365472311a","unresolved":true,"context_lines":[{"line_number":129,"context_line":"    return cgtsvg_growth_gib"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"LOCK_NAME \u003d \u0027HostFsController\u0027"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"class HostFsController(rest.RestController):"}],"source_content_type":"text/x-python","patch_set":4,"id":"e38e38aa_e7a1698a","line":132,"updated":"2022-04-07 21:05:35.000000000","message":"Since controller_fs.py can also set CONTROLLER_FS_RESIZING_IN_PROGRESS ; \nconsider enforcing the mutex by using the same LOCK_NAME.\n\nbootstrap and deployment would need to be confirmed its ok to do so.","commit_id":"e833a1a14702d135437d1865d219f457fdf140ba"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"a4c3d01c9bbafad53fc7438cdcc112d4ed1fb4af","unresolved":false,"context_lines":[{"line_number":129,"context_line":"    return cgtsvg_growth_gib"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"LOCK_NAME \u003d \u0027HostFsController\u0027"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"class HostFsController(rest.RestController):"}],"source_content_type":"text/x-python","patch_set":4,"id":"72683439_2f295006","line":132,"in_reply_to":"e38e38aa_e7a1698a","updated":"2022-04-08 19:14:54.000000000","message":"Done, thanks John.","commit_id":"e833a1a14702d135437d1865d219f457fdf140ba"}]}