)]}'
{"nova/virt/vmwareapi/driver.py":[{"author":{"_account_id":19853,"name":"Nguyen Hung Phuong","email":"phuongnh@vn.fujitsu.com","username":"phuongnh"},"change_message_id":"d19028844b8f64a668d1a585a3e6a5f4275a58cb","unresolved":false,"context_lines":[{"line_number":347,"context_line":"        return [self._nodename]"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"    def update_provider_tree(self, provider_tree, nodename):"},{"line_number":350,"context_line":"        import pdb; pdb.set_trace()"},{"line_number":351,"context_line":"        inventory \u003d self.get_inventory(nodename)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        # it doesn\u0027t matter what we put for generation here, it is ignored by"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_84307109","line":350,"updated":"2018-01-23 11:14:05.000000000","message":"should be on 2 lines","commit_id":"8631f5b0e0864903b6b0570b75a16a6543a8a821"},{"author":{"_account_id":19853,"name":"Nguyen Hung Phuong","email":"phuongnh@vn.fujitsu.com","username":"phuongnh"},"change_message_id":"9728f4dc40e5679a7e2928ee8dd29e90874de50a","unresolved":false,"context_lines":[{"line_number":418,"context_line":"        mem_allocation \u003d res_pool_info.mem_allocation"},{"line_number":419,"context_line":"        cpu_max \u003d res_pool_info.cpu_max"},{"line_number":420,"context_line":"        mem_max \u003d res_pool_info.mem_max // (1024 * 1024)"},{"line_number":421,"context_line":"        cpu_limit \u003d cpu_allocation.limit if cpu_allocation.limit \u003e\u003d 0 else cpu_max"},{"line_number":422,"context_line":"        mem_limit \u003d mem_allocation.limit if mem_allocation.limit \u003e\u003d 0 else mem_max"},{"line_number":423,"context_line":"        if cpu_limit \u003e 0 and cpu_speed_mhz \u003e 0:"},{"line_number":424,"context_line":"            # Since cpu requests to the scheduler is in the form of number"},{"line_number":425,"context_line":"            # of vcpus, converting the speed configuration on resource pool"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f96bb07_9d718de1","line":422,"range":{"start_line":421,"start_character":15,"end_line":422,"end_character":30},"updated":"2018-01-24 03:04:44.000000000","message":"lines are too long, please check.","commit_id":"d7dcf0e73d45a4be03ed82efc2a827d6c2f312f7"},{"author":{"_account_id":17920,"name":"Jianghua Wang","email":"jianghua.wang@citrix.com","username":"wjhfresh"},"change_message_id":"78bbb9e22676e8e85d4e3bcbe351200ad05eeab1","unresolved":false,"context_lines":[{"line_number":392,"context_line":"        # this compute node"},{"line_number":393,"context_line":"        root_rp \u003d provider_tree.roots[0]"},{"line_number":394,"context_line":"        # delete all nested providers, we will add them again"},{"line_number":395,"context_line":"        root_rp.children.clear()"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        cpu_speed_mhz \u003d stats[\u0027cpu\u0027][\u0027vcpus_mhz\u0027] / stats[\u0027cpu\u0027][\u0027vcpus\u0027]"},{"line_number":398,"context_line":"        for res_pool in stats[\u0027resource_pools\u0027]:"}],"source_content_type":"text/x-python","patch_set":5,"id":"bf659307_865e2f10","line":395,"updated":"2018-04-10 06:43:13.000000000","message":"How about just delete the RPs which don\u0027t exist anymore so that we can keep the RP\u0027s id/uuids are consistent? Or is there any mechanism to guarantee the ids be consistent even with the clear-all\u0026re-generate RPs?","commit_id":"0a91dc0974ee0d65eb18ba51f3ba86d8599864d2"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"a7302f4888c694823e95b816ef56c58d788fa371","unresolved":false,"context_lines":[{"line_number":392,"context_line":"        # this compute node"},{"line_number":393,"context_line":"        root_rp \u003d provider_tree.roots[0]"},{"line_number":394,"context_line":"        # delete all nested providers, we will add them again"},{"line_number":395,"context_line":"        root_rp.children.clear()"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        cpu_speed_mhz \u003d stats[\u0027cpu\u0027][\u0027vcpus_mhz\u0027] / stats[\u0027cpu\u0027][\u0027vcpus\u0027]"},{"line_number":398,"context_line":"        for res_pool in stats[\u0027resource_pools\u0027]:"}],"source_content_type":"text/x-python","patch_set":5,"id":"bf659307_aba6850e","line":395,"in_reply_to":"bf659307_865e2f10","updated":"2018-04-10 22:40:34.000000000","message":"That\u0027s correct.  You can\u0027t assume you own all the children.  You have to identify the ones you know you own and delete only those.","commit_id":"0a91dc0974ee0d65eb18ba51f3ba86d8599864d2"}]}