Add missing Core FIlter in V9 deployment Nova Playbooks

Bug #1399366 reported by Ian Cordasco
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Medium
Matt Thompson
Juno
Fix Released
Medium
Matt Thompson
Trunk
Fix Released
Medium
Matt Thompson

Bug Description

Opened by cbruf82 on 2014-11-25 22:21:32+00:00 at https://github.com/rcbops/ansible-lxc-rpc/issues/652

------------------------------------------------------------

In the nova_all.yml file:

nova_scheduler_default_filters: RetryFilter,AvailabilityZoneFilter,RamFilter,ComputeFilter,CoreFilter,ComputeCapabilitiesFilter,ImagePropertiesFilter,ServerGroupAntiAffinityFilter,ServerGroupAffinityFilter,AggregateCoreFilter

Tags:

====================== COMMENTS ============================

Comment created by odyssey4me on 2014-11-25 22:35:03+00:00

We could, of course, add every scheduler filter into the config - thus providing the capability of any existing filter/scheduler hint to be used without further customisation. That would probably result in a bit of extra load on the DB and scheduler server though. Thoughts?

------------------------------------------------------------

Comment created by cloudnull on 2014-11-26 00:21:59+00:00

@cbruf82 while the filter was not set in the default config you can override it to be whatever you'd like. Simply set the variable nova_scheduler_default_filters as the comma separated list of filters you want in the ``/etc/rpc_deploy/user_variables.yml** file and then (re)run the nova-scheduler plays to update your configs.

Default filters set for v9.0.x: "https://github.com/rcbops/ansible-lxc-rpc/blob/1bbea028674a2b5e0278232cb74e7047f4f60c4d/rpc_deployment/inventory/group_vars/nova_all.yml#L81"

tags: added: icehouse-backport-potential
Changed in openstack-ansible:
importance: Undecided → Medium
Changed in openstack-ansible:
milestone: none → 10.1.0
status: New → Confirmed
Revision history for this message
Matt Thompson (mattt416) wrote :

Hi Justin,

The nova_scheduler_default_filters in the original comment adds CoreFilter and removes AggregateDiskFilter. Are we adjusting the defaults across all branches to match this?

Thanks!

--Matt

Revision history for this message
Matt Thompson (mattt416) wrote :

We have a similar request in https://bugs.launchpad.net/openstack-ansible/+bug/1399364 asking us to add DiskFilter, trying to get an exhaustive list of the default filters required so I can make a single commit.

Revision history for this message
Andy McCrae (andrew-mccrae) wrote :

I'm in favour of using the nova.conf default from upstream, and then allowing the value to be adjusted via a variable.

Matt Thompson (mattt416)
Changed in openstack-ansible:
assignee: nobody → Matt Thompson (mattt416)
Revision history for this message
Matt Thompson (mattt416) wrote :

Looks like adding CoreFilter should be unnecessary when AggregateCoreFilter is present. From http://docs.openstack.org/juno/config-reference/content/section_compute-scheduler.html:

AggregateCoreFilter

Filters host by CPU core numbers with a per-aggregate cpu_allocation_ratio value. If the per-aggregate value is not found, the value falls back to the global setting.

Changed in openstack-ansible:
status: Confirmed → Invalid
Matt Thompson (mattt416)
Changed in openstack-ansible:
status: Invalid → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (master)

Fix proposed to branch: master
Review: https://review.openstack.org/141017

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (master)

Reviewed: https://review.openstack.org/141017
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=f3af86445f2aafc2b42faeb79a1bfb60cc99af0c
Submitter: Jenkins
Branch: master

commit f3af86445f2aafc2b42faeb79a1bfb60cc99af0c
Author: Matt Thompson <email address hidden>
Date: Thu Dec 11 11:15:15 2014 +0000

    Change nova scheduler filters

    This commit changes AggregateCoreFilter to CoreFilter and
    AggregateDiskFilter to DiskFilter. Whilst these filters work even if
    aggregates are not being used, icehouse does not have
    AggregateDiskFilter so there CoreFilter and DiskFilter need to be used.
    This change is more to bring the two branches in line, and if anyone
    does explicitly need the aggregate filter then that can be added via a
    local override.

    Change-Id: I6cc01139db7274f80d138020c97368a0faece771
    Closes-Bug: 1399366

Changed in openstack-ansible:
status: In Progress → Fix Committed
Changed in openstack-ansible:
milestone: 10.1.0 → 10.1.2
no longer affects: openstack-ansible/icehouse
tags: removed: icehouse-backport-potential
no longer affects: openstack-ansible/icehouse
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.