)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"c4a746496eeb7311b8eac2329bd823b2b3a5a277","unresolved":true,"context_lines":[{"line_number":40,"context_line":"PASS: run host-fs-modify command successfully, wait for lvg available"},{"line_number":41,"context_line":"      size to be updated and run a controllerfs-modify, verify that"},{"line_number":42,"context_line":"      it runs successfully"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Regression:"},{"line_number":45,"context_line":"PASS: run host-fs-modify command that exceeds vg available space and"},{"line_number":46,"context_line":"      verify that it is not allowed to execute"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ab4ca2b3_b234cfa3","line":43,"updated":"2022-04-27 12:45:10.000000000","message":"recommend testcase to increase prior to (subcloud) upgrade and then upgrade, as per the initial LP scenario.","commit_id":"d7b15cfa808fa25840434d8723b7f72fd1cea2ec"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"0b2340df43885bd42d22877eef56696d8b4b1f9e","unresolved":false,"context_lines":[{"line_number":40,"context_line":"PASS: run host-fs-modify command successfully, wait for lvg available"},{"line_number":41,"context_line":"      size to be updated and run a controllerfs-modify, verify that"},{"line_number":42,"context_line":"      it runs successfully"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Regression:"},{"line_number":45,"context_line":"PASS: run host-fs-modify command that exceeds vg available space and"},{"line_number":46,"context_line":"      verify that it is not allowed to execute"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3156763d_005a800d","line":43,"in_reply_to":"ab4ca2b3_b234cfa3","updated":"2022-04-27 15:33:50.000000000","message":"Done, tests evidence: https://paste.opendev.org/show/bFNMe96uTZhARf0tzhUg/\n\nNote that The system blocks resizes that would not fit the vg, and I had to add an extra partition to cgts-vg to resize both docker to 40G and docker-distribution to 32G prior to the upgrade. More details on the paste link above","commit_id":"d7b15cfa808fa25840434d8723b7f72fd1cea2ec"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"4b7f692d357bdc1888a9a60500b101ca34c345c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"03e9d2ca_43f70d82","updated":"2022-04-26 13:59:01.000000000","message":"Test evidence: https://paste.opendev.org/show/bofKN1HnaSdb9ojKMrjq/","commit_id":"1fd035bc9e7ac0ef91be27de0aa263949a51d8a7"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"c7bfc17416767e736410dfaafa0833c00ab07e68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"16c60c8c_eeb1c524","updated":"2022-04-29 13:17:35.000000000","message":"Thanks for the review and comments, I\u0027ll address then on a future commit.","commit_id":"8c3ebd9def38c586143abc37b6aab6b30f630e4f"}],"sysinv/sysinv/sysinv/sysinv/api/controllers/v1/controller_fs.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"21d1fff0c086561224e08875f9cb0abfb5a459ce","unresolved":true,"context_lines":[{"line_number":574,"context_line":""},{"line_number":575,"context_line":"        controllers_lvg_updated \u003d True"},{"line_number":576,"context_line":"        for host in controller_hosts:"},{"line_number":577,"context_line":"            host_fs_list \u003d pecan.request.dbapi.host_fs_get_by_ihost(host.uuid)"},{"line_number":578,"context_line":"            host_lvg_list \u003d pecan.request.dbapi.ilvg_get_by_ihost(host.uuid)"},{"line_number":579,"context_line":"            controllers_lvg_updated \u003d controllers_lvg_updated and \\"},{"line_number":580,"context_line":"                utils.is_host_lvg_updated(host_fs_list, host_lvg_list)"}],"source_content_type":"text/x-python","patch_set":3,"id":"af8c8cab_2dc108b1","line":577,"updated":"2022-04-28 16:50:56.000000000","message":"would we ever want to short circuit this, and break out of the loop as soon as it changes to false?\nI am guessing that since its controller_hosts,  there are only ever 2 controllers so not really expensive to loop an extra time","commit_id":"8c3ebd9def38c586143abc37b6aab6b30f630e4f"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"c7bfc17416767e736410dfaafa0833c00ab07e68","unresolved":false,"context_lines":[{"line_number":574,"context_line":""},{"line_number":575,"context_line":"        controllers_lvg_updated \u003d True"},{"line_number":576,"context_line":"        for host in controller_hosts:"},{"line_number":577,"context_line":"            host_fs_list \u003d pecan.request.dbapi.host_fs_get_by_ihost(host.uuid)"},{"line_number":578,"context_line":"            host_lvg_list \u003d pecan.request.dbapi.ilvg_get_by_ihost(host.uuid)"},{"line_number":579,"context_line":"            controllers_lvg_updated \u003d controllers_lvg_updated and \\"},{"line_number":580,"context_line":"                utils.is_host_lvg_updated(host_fs_list, host_lvg_list)"}],"source_content_type":"text/x-python","patch_set":3,"id":"6b676b87_2e67cb30","line":577,"in_reply_to":"af8c8cab_2dc108b1","updated":"2022-04-29 13:17:35.000000000","message":"Ack","commit_id":"8c3ebd9def38c586143abc37b6aab6b30f630e4f"}],"sysinv/sysinv/sysinv/sysinv/api/controllers/v1/utils.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"21d1fff0c086561224e08875f9cb0abfb5a459ce","unresolved":true,"context_lines":[{"line_number":871,"context_line":"    \"\"\""},{"line_number":872,"context_line":"    last_resize \u003d None"},{"line_number":873,"context_line":""},{"line_number":874,"context_line":"    LOG.info(\"Checking host_fs: %s\" % [fs[\u0027name\u0027] for fs in host_fs_list])"},{"line_number":875,"context_line":"    for fs in host_fs_list:"},{"line_number":876,"context_line":"        if fs[\u0027name\u0027] in constants.FILESYSTEM_CONTROLLER_SUPPORTED_LIST:"},{"line_number":877,"context_line":"            if fs[\u0027updated_at\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"b238cff3_5b2d4ad2","line":874,"updated":"2022-04-28 16:50:56.000000000","message":"I think some of these INFO logs could be dropped to debug.  depending on how often we expect this method to be called.","commit_id":"8c3ebd9def38c586143abc37b6aab6b30f630e4f"},{"author":{"_account_id":33390,"name":"Heitor Matsui","display_name":"Heitor Matsui","email":"HeitorVieira.Matsui@windriver.com","username":"heitormatsui"},"change_message_id":"c7bfc17416767e736410dfaafa0833c00ab07e68","unresolved":false,"context_lines":[{"line_number":871,"context_line":"    \"\"\""},{"line_number":872,"context_line":"    last_resize \u003d None"},{"line_number":873,"context_line":""},{"line_number":874,"context_line":"    LOG.info(\"Checking host_fs: %s\" % [fs[\u0027name\u0027] for fs in host_fs_list])"},{"line_number":875,"context_line":"    for fs in host_fs_list:"},{"line_number":876,"context_line":"        if fs[\u0027name\u0027] in constants.FILESYSTEM_CONTROLLER_SUPPORTED_LIST:"},{"line_number":877,"context_line":"            if fs[\u0027updated_at\u0027]:"}],"source_content_type":"text/x-python","patch_set":3,"id":"edbf4901_f818b76d","line":874,"in_reply_to":"b238cff3_5b2d4ad2","updated":"2022-04-29 13:17:35.000000000","message":"Ack","commit_id":"8c3ebd9def38c586143abc37b6aab6b30f630e4f"}]}