)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8412,"name":"Mike Durnosvistov","email":"glacierr.dev@gmail.com","username":"mdurnosvistov"},"change_message_id":"ba2119b7ec7f9fbb761f221bf8d49b7b85ddbf50","unresolved":false,"context_lines":[{"line_number":7,"context_line":"limits don\u0027t update without using filters"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When creating a vm by specifying a host and the host has insufficient"},{"line_number":10,"context_line":"resouces, the user can scale resources by hand, such as the disk, cpu,"},{"line_number":11,"context_line":"memory or numa_topology. The resources scaled by hand can be able to"},{"line_number":12,"context_line":"refresh db, but the host_state.limits are not refreshed. At this time,"},{"line_number":13,"context_line":"if we creating a vm by specifying a host, it can\u0027t create a vm"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"baa041b7_59d755bf","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":8},"updated":"2015-04-03 13:31:46.000000000","message":"s/resouces/resources","commit_id":"ba114d3ec00ff19c990809527f54bbe98d297b23"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"69bec27f621eb97400c279af6e48d3d3eb36865f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"successfully because of the host_state.limits. This patch just a transition"},{"line_number":15,"context_line":"method and only deals with disk limits. If we want to solve these"},{"line_number":16,"context_line":"problems perfectly, we need to wait for the following bp implemented."},{"line_number":17,"context_line":"https://blueprints.launchpad.net/nova/+spec/allocation-ratio-to-resource-tracker"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ibbf5ffa2ee76d3948ee2de8bd1c64b8a57b6eaca"},{"line_number":20,"context_line":"Closes-Bug: #1422048"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9aa53dc9_966db59a","line":17,"updated":"2015-04-07 21:47:28.000000000","message":"I think there is a main problem with your approach : as filters are optional but provide those ratios, it means that you need to check the ratios within them and not directly within the HostManager.\n\nAs I said, I think it\u0027s just... horrible and hacky. I would rather put the efforts to providing those ratios in the RT, as you mentioned.","commit_id":"ba114d3ec00ff19c990809527f54bbe98d297b23"},{"author":{"_account_id":11530,"name":"zhangtralon","email":"ztralon@hotmail.com","username":"zhangtralon"},"change_message_id":"622499a1fbd8aa18a358db963f7596a910b839a8","unresolved":false,"context_lines":[{"line_number":14,"context_line":"successfully because of the host_state.limits. This patch just a transition"},{"line_number":15,"context_line":"method and only deals with disk limits. If we want to solve these"},{"line_number":16,"context_line":"problems perfectly, we need to wait for the following bp implemented."},{"line_number":17,"context_line":"https://blueprints.launchpad.net/nova/+spec/allocation-ratio-to-resource-tracker"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: Ibbf5ffa2ee76d3948ee2de8bd1c64b8a57b6eaca"},{"line_number":20,"context_line":"Closes-Bug: #1422048"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9aa53dc9_29b56730","line":17,"in_reply_to":"9aa53dc9_966db59a","updated":"2015-04-08 01:18:41.000000000","message":"yeah, I agree with you, but may be we need to waiting for the bp implemented. \thttps://blueprints.launchpad.net/nova/+spec/allocation-ratio-to-resource-tracker","commit_id":"ba114d3ec00ff19c990809527f54bbe98d297b23"}],"nova/scheduler/host_manager.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"92143920dcd1cc11b318218bb9ee3197dd503622","unresolved":false,"context_lines":[{"line_number":61,"context_line":"                help\u003d\u0027Which weight class names to use for weighing hosts\u0027),"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    cfg.FloatOpt(\"disk_allocation_ratio\", default\u003d1.0,"},{"line_number":64,"context_line":"                    help\u003d\"Virtual disk to physical disk allocation ratio\"),"},{"line_number":65,"context_line":"    ]"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"da86d52c_942409c7","line":64,"updated":"2015-02-18 15:52:07.000000000","message":"I already have serious concerns about overcommitment ratios, but here that\u0027s even worst because you\u0027re providing a global overcommit ratio at the Nova scheduler level which means that it can\u0027t be on a per-compute node basis","commit_id":"1d8d9feb1d9c27c0ae713489ec4bb2b25a159a6a"},{"author":{"_account_id":11530,"name":"zhangtralon","email":"ztralon@hotmail.com","username":"zhangtralon"},"change_message_id":"28d907f86df4fb137ed4ca804b053c5135834da9","unresolved":false,"context_lines":[{"line_number":61,"context_line":"                help\u003d\u0027Which weight class names to use for weighing hosts\u0027),"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    cfg.FloatOpt(\"disk_allocation_ratio\", default\u003d1.0,"},{"line_number":64,"context_line":"                    help\u003d\"Virtual disk to physical disk allocation ratio\"),"},{"line_number":65,"context_line":"    ]"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a80dd14_94e557aa","line":64,"in_reply_to":"da86d52c_942409c7","updated":"2015-04-07 06:19:14.000000000","message":"yeah, this problem can be solved after the following bp implemented.\nhttps://blueprints.launchpad.net/nova/+spec/allocation-ratio-to-resource-tracker","commit_id":"1d8d9feb1d9c27c0ae713489ec4bb2b25a159a6a"},{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"69bec27f621eb97400c279af6e48d3d3eb36865f","unresolved":false,"context_lines":[{"line_number":67,"context_line":"               help\u003d\u0027Determines if the Scheduler tracks changes to instances \u0027"},{"line_number":68,"context_line":"                    \u0027to help with its filtering decisions.\u0027),"},{"line_number":69,"context_line":"    cfg.FloatOpt(\"disk_allocation_ratio\", default\u003d1.0,"},{"line_number":70,"context_line":"                    help\u003d\"Virtual disk to physical disk allocation ratio\"),"},{"line_number":71,"context_line":"    ]"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa53dc9_36edc925","line":70,"updated":"2015-04-07 21:47:28.000000000","message":"Erm this opt is already provided by the DiskFilter...","commit_id":"ba114d3ec00ff19c990809527f54bbe98d297b23"},{"author":{"_account_id":8412,"name":"Mike Durnosvistov","email":"glacierr.dev@gmail.com","username":"mdurnosvistov"},"change_message_id":"ba2119b7ec7f9fbb761f221bf8d49b7b85ddbf50","unresolved":false,"context_lines":[{"line_number":182,"context_line":"            else:"},{"line_number":183,"context_line":"                LOG.warning(_LW(\"Metric name unknown of %r\"), item)"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"    def _update_limits_from_compute_node(self, compute):"},{"line_number":186,"context_line":"        total_usable_disk_mb \u003d compute.local_gb * 1024"},{"line_number":187,"context_line":"        disk_mb_limit \u003d total_usable_disk_mb * CONF.disk_allocation_ratio"},{"line_number":188,"context_line":"        disk_gb_limit \u003d disk_mb_limit / 1024"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_99eb9d78","line":185,"range":{"start_line":185,"start_character":8,"end_line":185,"end_character":40},"updated":"2015-04-03 13:31:46.000000000","message":"Please add test-case for this changes.","commit_id":"ba114d3ec00ff19c990809527f54bbe98d297b23"}]}