)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b3d5ee45c81d324e0f3aa9ce43cfd58153e60043","unresolved":false,"context_lines":[{"line_number":518,"context_line":"            size\u003dCONF.sync_power_state_pool_size)"},{"line_number":519,"context_line":"        self._syncs_in_progress \u003d {}"},{"line_number":520,"context_line":"        self.send_instance_updates \u003d ("},{"line_number":521,"context_line":"            CONF.filter_scheduler.track_instance_changes)"},{"line_number":522,"context_line":"        if CONF.max_concurrent_builds !\u003d 0:"},{"line_number":523,"context_line":"            self._build_semaphore \u003d eventlet.semaphore.Semaphore("},{"line_number":524,"context_line":"                CONF.max_concurrent_builds)"}],"source_content_type":"text/x-python","patch_set":24,"id":"7ffa3b31_82fb7a77","line":521,"updated":"2017-04-19 16:07:28.000000000","message":"shouldn\u0027t we also verify the workaround opt value and set send_instance_updates to False if the opt is True ?","commit_id":"17f2acc7322655e728e7c55a77b0e9b65820d90f"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"ffe0b17788072b472ffb49ef21991c8932f2d190","unresolved":false,"context_lines":[{"line_number":518,"context_line":"            size\u003dCONF.sync_power_state_pool_size)"},{"line_number":519,"context_line":"        self._syncs_in_progress \u003d {}"},{"line_number":520,"context_line":"        self.send_instance_updates \u003d ("},{"line_number":521,"context_line":"            CONF.filter_scheduler.track_instance_changes)"},{"line_number":522,"context_line":"        if CONF.max_concurrent_builds !\u003d 0:"},{"line_number":523,"context_line":"            self._build_semaphore \u003d eventlet.semaphore.Semaphore("},{"line_number":524,"context_line":"                CONF.max_concurrent_builds)"}],"source_content_type":"text/x-python","patch_set":24,"id":"7ffa3b31_4848f35a","line":521,"in_reply_to":"7ffa3b31_82fb7a77","updated":"2017-04-19 17:49:44.000000000","message":"The workaround is specifically for the check. You could want to enable that and still report this, for non-cell reasons I assume. The doc text ties them together, so i think it\u0027s okay.","commit_id":"17f2acc7322655e728e7c55a77b0e9b65820d90f"}],"nova/conf/scheduler.py":[{"author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"change_message_id":"b3d5ee45c81d324e0f3aa9ce43cfd58153e60043","unresolved":false,"context_lines":[{"line_number":234,"context_line":"NOTE: In a multi-cell (v2) setup where the cell MQ is separated from the"},{"line_number":235,"context_line":"top-level, computes cannot directly communicate with the scheduler. Thus,"},{"line_number":236,"context_line":"this option cannot be enabled in that scenario. See also the"},{"line_number":237,"context_line":"workarounds.disable_group_policy_check_upcall option."},{"line_number":238,"context_line":"\"\"\"),"},{"line_number":239,"context_line":"    cfg.MultiStrOpt(\"available_filters\","},{"line_number":240,"context_line":"        default\u003d[\"nova.scheduler.filters.all_filters\"],"}],"source_content_type":"text/x-python","patch_set":24,"id":"7ffa3b31_a29b1664","line":237,"range":{"start_line":237,"start_character":0,"end_line":237,"end_character":45},"updated":"2017-04-19 16:07:28.000000000","message":"nit: [workarounds]/disable_group_policy_check_upcall","commit_id":"17f2acc7322655e728e7c55a77b0e9b65820d90f"}],"nova/conf/workarounds.py":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"c4c7d8309dfe6395fde7745767a07a482c218c72","unresolved":false,"context_lines":[{"line_number":136,"context_line":"scheduler. It does this by making an upcall to the API database to list"},{"line_number":137,"context_line":"the instances in the server group for one that it is booting, which violates"},{"line_number":138,"context_line":"our api/cell isolation goals. Eventually this will be solved by proper affinity"},{"line_number":139,"context_line":"guarantees in the scheduler and placemetn service, but until then, this late"},{"line_number":140,"context_line":"check is needed to ensure proper affinity policy."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"Operators that desire api/cell isolation over this check should"}],"source_content_type":"text/x-python","patch_set":11,"id":"5a22e58a_13f3ee0b","line":139,"range":{"start_line":139,"start_character":32,"end_line":139,"end_character":41},"updated":"2017-03-14 12:15:44.000000000","message":"typo, \u0027placement\u0027","commit_id":"79318dc0de0dd3eadb3b708213cc4f8fde167453"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5eb9e981ff11d686dc8c3f10da6307f2f9e12310","unresolved":false,"context_lines":[{"line_number":126,"context_line":"\"\"\"),"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    cfg.BoolOpt("},{"line_number":129,"context_line":"        \u0027disable_group_policy_check_upcall\u0027,"},{"line_number":130,"context_line":"        default\u003dFalse,"},{"line_number":131,"context_line":"        help\u003d\"\"\""},{"line_number":132,"context_line":"Disable the server group policy check upcall in compute."}],"source_content_type":"text/x-python","patch_set":21,"id":"bff0334d_67032475","line":129,"updated":"2017-04-06 20:40:17.000000000","message":"We should probably call this out as a related option in the help text for the \u0027track_instance_changes\u0027 option which is tied to how this is used I think in the scheduler.","commit_id":"c4e39aa0dde19309965ad1443702047b81a05826"}]}