)]}'
{"manila/scheduler/host_manager.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"46033626774b2c513a1de7d9750b0646c3af9400","unresolved":false,"context_lines":[{"line_number":435,"context_line":"            # NOTE(jose-castro-leon): Only calculate provisioned_capacity_gb"},{"line_number":436,"context_line":"            # on thin provisioned pools"},{"line_number":437,"context_line":"            self.provisioned_capacity_gb \u003d capability.get("},{"line_number":438,"context_line":"                \u0027provisioned_capacity_gb\u0027, 0)"},{"line_number":439,"context_line":""},{"line_number":440,"context_line":"            if self.thin_provisioning and not self.provisioned_capacity_gb:"},{"line_number":441,"context_line":"                self.provisioned_capacity_gb \u003d ("}],"source_content_type":"text/x-python","patch_set":1,"id":"df33271e_6d37374f","line":438,"range":{"start_line":438,"start_character":43,"end_line":438,"end_character":44},"updated":"2020-04-07 17:39:57.000000000","message":"it\u0027s possible this is really 0 for some pools, let\u0027s avoid the calculation completely... \n\n self.provisioned_capacity_gb \u003d capability.get(\n                \u0027provisioned_capacity_gb\u0027)\n\n if self.thin_provisioning and self.provisioned_capacity_gb is None:\n    self.provisioned_capacity_gb \u003d (\n                    self._estimate_provisioned_capacity(self.host,\n                                                        context\u003dcontext))","commit_id":"2de4f7ebaf2862053aeeaefeca72467fc9a55c59"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"37aecdf24bef89f480b4a3f57d0359da41815676","unresolved":false,"context_lines":[{"line_number":435,"context_line":"            # NOTE(jose-castro-leon): Only calculate provisioned_capacity_gb"},{"line_number":436,"context_line":"            # on thin provisioned pools"},{"line_number":437,"context_line":"            self.provisioned_capacity_gb \u003d capability.get("},{"line_number":438,"context_line":"                \u0027provisioned_capacity_gb\u0027, 0)"},{"line_number":439,"context_line":""},{"line_number":440,"context_line":"            if self.thin_provisioning and not self.provisioned_capacity_gb:"},{"line_number":441,"context_line":"                self.provisioned_capacity_gb \u003d ("}],"source_content_type":"text/x-python","patch_set":1,"id":"df33271e_cd264317","line":438,"range":{"start_line":438,"start_character":43,"end_line":438,"end_character":44},"in_reply_to":"df33271e_6d37374f","updated":"2020-04-07 17:41:55.000000000","message":"your change currently matches what was being done before - but, we could improve this while we\u0027re here even for those thinly provisioned backends that report 0.","commit_id":"2de4f7ebaf2862053aeeaefeca72467fc9a55c59"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"428d623c3fd74d340df67e8dd5a015f42e201bac","unresolved":false,"context_lines":[{"line_number":435,"context_line":"            # NOTE(jose-castro-leon): Only calculate provisioned_capacity_gb"},{"line_number":436,"context_line":"            # on thin provisioned pools"},{"line_number":437,"context_line":"            self.provisioned_capacity_gb \u003d capability.get("},{"line_number":438,"context_line":"                \u0027provisioned_capacity_gb\u0027, 0)"},{"line_number":439,"context_line":""},{"line_number":440,"context_line":"            if self.thin_provisioning and not self.provisioned_capacity_gb:"},{"line_number":441,"context_line":"                self.provisioned_capacity_gb \u003d ("}],"source_content_type":"text/x-python","patch_set":1,"id":"df33271e_5bd30dd6","line":438,"range":{"start_line":438,"start_character":43,"end_line":438,"end_character":44},"in_reply_to":"df33271e_cd264317","updated":"2020-04-07 19:07:21.000000000","message":"OK, I agree with this  -- don\u0027t default to zero, a so-far-unused share might have zero actually provisioned.","commit_id":"2de4f7ebaf2862053aeeaefeca72467fc9a55c59"}]}