Sahara deployment fails

[Sergii Golovatiuk]

Deployment Sahara nodes fails as Nova can't find appropriate node. It's stuck on "Scheduling" without any progress. This happened due to mutually exclusive filters ( 'DifferentHostFilter', 'SameHostFilter') in scheduler_default_filters.

[OpenStack Infra (hudson-openstack)]

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

[Dmitry Mescheryakov (dmitrymex)]

As far as I know, DifferentHostFilter and SameHostFilter can co-exist together nicely. Take a look at the docs:
http://docs.openstack.org/trunk/config-reference/content/section_compute-scheduler.html
It explicitly states that " To take advantage of this filter, the requester must pass a scheduler hint" for both filters. I.e. the filters are not invoked unless user explicitly set specific hints in 'nova boot' parameters.

Also, the line with two filters are here for almost a month:
https://github.com/stackforge/fuel-library/blame/master/deployment/puppet/osnailyfacter/manifests/cluster_ha.pp#L457
I know that Vadim Rovachev tests Sahara not once since then and it mostly worked.

[Dmitry Mescheryakov (dmitrymex)]

I think the issue should be investigated further. Sahara do needs DifferentHostFilter and should not be a problem since it worked so far.

[Dmitry Mescheryakov (dmitrymex)]

Also, it would help if you clarify which env and test scenario you've used

[Sergii Golovatiuk (sgolovatiuk)]

Here is a pick from nova scheduler documentation

DifferentHostFilter - allows to put the instance on a different host from a set of instances.
SameHostFilter - puts the instance on the same host as another instance in a set of instances.

It's mutual exclusion which makes no sense to nova scheduler. I left DifferentHost in Sahara manifest. However, it's a big topic and area for investigation as all keys have own weight and own impact.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/94632
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=71b112d8442c5a60b9ebe95b99b7b2bf9eb34b2a
Submitter: Jenkins
Branch: master

commit 71b112d8442c5a60b9ebe95b99b7b2bf9eb34b2a
Author: Sergii Golovatiuk <email address hidden>
Date: Wed May 21 14:34:49 2014 +0000

    Replace concat.rb with latest stdlib equivalent

    Sync concat puppet function to fix some bugs

    Related-Bug: #1320184
    Related-Bug: #1321284

    Change-Id: I49a614122469b37b1c4eb2fbc29b0877a4427fbd

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/94366
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=b1f06a3416c0323ca742149c5257b47d616f9072
Submitter: Jenkins
Branch: master

commit b1f06a3416c0323ca742149c5257b47d616f9072
Author: Sergii Golovatiuk <email address hidden>
Date: Tue May 20 13:20:01 2014 +0000

    Fix Nova scheduler_default_filters

    Fix nova scheduler_default_filters filters for Sahara Deployment

    - Change Sahara scheduler_default_filters strategy to DifferentHost
    - Refactor to remove unused variables

    Change-Id: I2a908e1b3e704326d10ff1867971048769df29f5
    Closes-Bug: 1321284

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/4.1
Review: https://review.openstack.org/96858

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/96858
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=18d81e17cfd13c135bb433a11dcc7d2c69f2c84d
Submitter: Jenkins
Branch: stable/4.1

commit 18d81e17cfd13c135bb433a11dcc7d2c69f2c84d
Author: Sergii Golovatiuk <email address hidden>
Date: Wed May 21 14:34:49 2014 +0000

    Replace concat.rb with latest stdlib equivalent

    Sync concat puppet function to fix some bugs

    Related-Bug: #1320184
    Related-Bug: #1321284

    Change-Id: I49a614122469b37b1c4eb2fbc29b0877a4427fbd

[Meg McRoberts (dreidellhasa)]

Marked as fixed in 4.1.1 Release Notes