)]}'
{"lib/nova_plugins/hypervisor-ironic":[{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"93303d0bf3233fdd39d3f6a54fcc38869dd794e4","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    iniset $NOVA_CONF DEFAULT scheduler_host_manager ironic_host_manager"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    if [[ \"$IRONIC_USE_RESOURCE_CLASSES\" \u003d\u003d \"False\" ]]; then"},{"line_number":47,"context_line":"      iniset $NOVA_CONF filter_scheduler use_baremetal_filters True"},{"line_number":48,"context_line":"    fi"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    iniset $NOVA_CONF DEFAULT ram_allocation_ratio 1.0"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"df3967d1_e2fb3976","line":47,"updated":"2017-08-03 13:08:04.000000000","message":"So if Ironic is using resource classes, this change will cause the non-ironic filters to be used instead of the ironic filters. I don\u0027t believe that that is the desired behavior. If resource classes are used, then the list of baremetal filters should have the ram/cpu/disk filters removed.","commit_id":"8227aede2e744d393c27639c3e4e7b92984e2883"},{"author":{"_account_id":1063,"name":"Ed Leafe","email":"ed@leafe.com","username":"ed-leafe"},"change_message_id":"bf8ffa22f81d64189899d2fe620f3eaf41d4e4c3","unresolved":false,"context_lines":[{"line_number":44,"context_line":"    iniset $NOVA_CONF DEFAULT scheduler_host_manager ironic_host_manager"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    if [[ \"$IRONIC_USE_RESOURCE_CLASSES\" \u003d\u003d \"False\" ]]; then"},{"line_number":47,"context_line":"      iniset $NOVA_CONF filter_scheduler use_baremetal_filters True"},{"line_number":48,"context_line":"    fi"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    iniset $NOVA_CONF DEFAULT ram_allocation_ratio 1.0"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"df3967d1_1da9008f","line":47,"in_reply_to":"df3967d1_e2fb3976","updated":"2017-08-03 13:31:41.000000000","message":"OK, after digging into this a bit more, with some help from sambetts, I think this should be OK. My concern was that there were other specific filters that could be configured for baremetal that would now be skipped.","commit_id":"8227aede2e744d393c27639c3e4e7b92984e2883"}]}