)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"1db61c89e7ab91155705e23e5dc11c836cfb6c43","unresolved":true,"context_lines":[{"line_number":10,"context_line":" / avail): \" can be printed correctly when a provisioning"},{"line_number":11,"context_line":"volume is created successfully."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1927321"},{"line_number":14,"context_line":"Change-Id: Idcaf657692c2597887476ec1834d790f39b1fe94"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a7cb828_ed92df57","line":13,"updated":"2021-05-06 11:51:01.000000000","message":"This change doesn\u0027t match what is being described in the bug.","commit_id":"7bf7c2da17367e14c28e6e83aa2fb04f20561d6c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"58bb721f10ed0d292093486f29d10fc84211afba","unresolved":true,"context_lines":[{"line_number":10,"context_line":" / avail): \" can be printed correctly when a provisioning"},{"line_number":11,"context_line":"volume is created successfully."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1927321"},{"line_number":14,"context_line":"Change-Id: Idcaf657692c2597887476ec1834d790f39b1fe94"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"06eee986_920099e5","line":13,"in_reply_to":"7a7cb828_ed92df57","updated":"2021-05-06 12:24:49.000000000","message":"Agree with Sean that this commit message is misleading.  What you\u0027ve fixed is:\n\n  Log message contains incorrect values\n\n  Correct a log message that incorrectly reports real free space\n  instead of virtual free space for thin provisioned volumes.","commit_id":"7bf7c2da17367e14c28e6e83aa2fb04f20561d6c"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"c817ba1883728901b19519dacefdbb7b4e3ab388","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     zhaoleilc \u003c15247232416@163.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-08-25 08:37:06 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Erroneous log location causes a misconception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch modifies log location so that the"},{"line_number":10,"context_line":"log message can report free space correctly."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"b154d140_0214b353","line":7,"range":{"start_line":7,"start_character":10,"end_line":7,"end_character":22},"updated":"2021-09-17 14:40:03.000000000","message":"youre not changing the log itself, just the message args. So it could be better as \u0027log message args location\u0027","commit_id":"8b56093b70cf7718cc0e3e1785e6e2a05aeef950"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"c817ba1883728901b19519dacefdbb7b4e3ab388","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Erroneous log location causes a misconception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch modifies log location so that the"},{"line_number":10,"context_line":"log message can report free space correctly."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idcaf657692c2597887476ec1834d790f39b1fe94"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"cb3644e5_316e2a21","line":9,"range":{"start_line":9,"start_character":20,"end_line":9,"end_character":32},"updated":"2021-09-17 14:40:03.000000000","message":"same here","commit_id":"8b56093b70cf7718cc0e3e1785e6e2a05aeef950"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"397dcbdaf9e50510116202c118fd6d9ffe28a862","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"481b6167_479cf861","updated":"2021-11-19 14:15:20.000000000","message":"Revisions look good.","commit_id":"4efd313d419253f5c3718867f17b8d229b3c2df8"}],"cinder/scheduler/filters/capacity_filter.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"58bb721f10ed0d292093486f29d10fc84211afba","unresolved":true,"context_lines":[{"line_number":169,"context_line":"                    LOG.debug(\"Space information for volume creation \""},{"line_number":170,"context_line":"                              \"on %(grouping)s %(grouping_name)s \""},{"line_number":171,"context_line":"                              \"(requested / avail): \""},{"line_number":172,"context_line":"                              \"%(requested)s/%(available)s\", msg_args)"},{"line_number":173,"context_line":"                return res"},{"line_number":174,"context_line":"        elif thin and backend_state.thin_provisioning_support:"},{"line_number":175,"context_line":"            LOG.warning(\"Filtering out %(grouping)s %(grouping_name)s \""}],"source_content_type":"text/x-python","patch_set":1,"id":"51f2ad49_a183fcf8","side":"PARENT","line":172,"range":{"start_line":172,"start_character":61,"end_line":172,"end_character":69},"updated":"2021-05-06 12:24:49.000000000","message":"this is using the values set at line 121, not the appropriate values for when thin-provisioning is enabled, which are set at line 160","commit_id":"ff6a967a5c207e3af7eb48c3cd72a5c965dd3caf"},{"author":{"_account_id":30696,"name":"zhaoleilc","email":"15247232416@163.com","username":"zhaoleilc"},"change_message_id":"7eb7ed93f8ea5252d39e592918fe599b9d9daf90","unresolved":false,"context_lines":[{"line_number":169,"context_line":"                    LOG.debug(\"Space information for volume creation \""},{"line_number":170,"context_line":"                              \"on %(grouping)s %(grouping_name)s \""},{"line_number":171,"context_line":"                              \"(requested / avail): \""},{"line_number":172,"context_line":"                              \"%(requested)s/%(available)s\", msg_args)"},{"line_number":173,"context_line":"                return res"},{"line_number":174,"context_line":"        elif thin and backend_state.thin_provisioning_support:"},{"line_number":175,"context_line":"            LOG.warning(\"Filtering out %(grouping)s %(grouping_name)s \""}],"source_content_type":"text/x-python","patch_set":1,"id":"f809f42a_a3c74f72","side":"PARENT","line":172,"range":{"start_line":172,"start_character":61,"end_line":172,"end_character":69},"in_reply_to":"0aa8a095_49ce73f2","updated":"2021-09-21 08:50:01.000000000","message":"Done","commit_id":"ff6a967a5c207e3af7eb48c3cd72a5c965dd3caf"},{"author":{"_account_id":30696,"name":"zhaoleilc","email":"15247232416@163.com","username":"zhaoleilc"},"change_message_id":"b3bb4bd2013191e4bb007500a41d7285cabc8514","unresolved":true,"context_lines":[{"line_number":169,"context_line":"                    LOG.debug(\"Space information for volume creation \""},{"line_number":170,"context_line":"                              \"on %(grouping)s %(grouping_name)s \""},{"line_number":171,"context_line":"                              \"(requested / avail): \""},{"line_number":172,"context_line":"                              \"%(requested)s/%(available)s\", msg_args)"},{"line_number":173,"context_line":"                return res"},{"line_number":174,"context_line":"        elif thin and backend_state.thin_provisioning_support:"},{"line_number":175,"context_line":"            LOG.warning(\"Filtering out %(grouping)s %(grouping_name)s \""}],"source_content_type":"text/x-python","patch_set":1,"id":"0aa8a095_49ce73f2","side":"PARENT","line":172,"range":{"start_line":172,"start_character":61,"end_line":172,"end_character":69},"in_reply_to":"51f2ad49_a183fcf8","updated":"2021-08-25 00:30:10.000000000","message":"That\u0027s right!","commit_id":"ff6a967a5c207e3af7eb48c3cd72a5c965dd3caf"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"58bb721f10ed0d292093486f29d10fc84211afba","unresolved":true,"context_lines":[{"line_number":186,"context_line":"            LOG.warning(\"Insufficient free space for volume creation \""},{"line_number":187,"context_line":"                        \"on %(grouping)s %(grouping_name)s (requested / \""},{"line_number":188,"context_line":"                        \"avail): %(requested)s/%(available)s\","},{"line_number":189,"context_line":"                        msg_args)"},{"line_number":190,"context_line":"            return False"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"        LOG.debug(\"Space information for volume creation \""}],"source_content_type":"text/x-python","patch_set":1,"id":"c7130980_dd286033","line":189,"range":{"start_line":189,"start_character":24,"end_line":189,"end_character":32},"updated":"2021-05-06 12:24:49.000000000","message":"check me on this, but the values set at line 121 are correct here and at line 194","commit_id":"7bf7c2da17367e14c28e6e83aa2fb04f20561d6c"}]}