Compute schedulers in nova - wrong volume scheduler link

[Matt Riedemann]

- [x] This doc is inaccurate in this way: ______

Where it says:

"For information about the volume scheduler, see the Block Storage section of OpenStack Administrator Guide."

The link is broken, it should point to this now:

https://docs.openstack.org/cinder/latest/admin/blockstorage-manage-volumes.html

However, I'm not really sure why the compute scheduler docs are even linking to the volume scheduler docs...

-----------------------------------
Release: 17.0.0.0rc2.dev42 on 2018-02-13 12:00
SHA: fba4161f71e47e4155e7f9a1f0d3a41b8107cef5
Source: https://git.openstack.org/cgit/openstack/nova/tree/doc/source/admin/configuration/schedulers.rst
URL: https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html

[Matt Riedemann (mriedem)]

The link to https://docs.openstack.org/admin-guide/compute-node-down.html#evacuate-instances is also wrong, and there are a bunch of other problems in this doc so I'll fix those all together.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/544015

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/544010
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=8be8d02aa77ebbe6687fa9ec4c74949cde0bf596
Submitter: Zuul
Branch: master

commit 8be8d02aa77ebbe6687fa9ec4c74949cde0bf596
Author: Matt Riedemann <email address hidden>
Date: Tue Feb 13 12:12:12 2018 -0500

    Fix and update compute schedulers config guide

    There were several things wrong with this doc since the migration
    in-tree in pike, which are handled here:

    - Updated options to use the current names.
    - Any mention of the periodic_task_interval option is removed
      since this doc is really about the FilterScheduler, and the
      periodic task interval is only used by the CachingScheduler to
      cache and refresh hosts, something the FilterScheduler doesn't do.
      The note about service_down_time is already covered in the help
      text for the periodic_task_interval option.
    - Removed a broken (and oddly placed) link to the block storage
      docs; it's unclear why the compute scheduler docs need to randomly
      link to the block storage docs for schedluing (maybe that was
      super legacy from the nova-volume days).
    - Fixed a broken link to the 'evacuate instances' docs.

    This change does not get into details for the individual filters
    or weighers, only the overview and filter scheduler description.

    Change-Id: Ie5bb9781e1997ad12521b5df5245a78b587a4a20
    Closes-Bug: #1749235

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/544015
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=4746a0d63862a4bb06a04636e6c0e4be9b38e82e
Submitter: Zuul
Branch: master

commit 4746a0d63862a4bb06a04636e6c0e4be9b38e82e
Author: Matt Riedemann <email address hidden>
Date: Tue Feb 13 12:19:35 2018 -0500

    Clarify the help text for [scheduler]periodic_task_interval

    There was some confusing/incorrect information in
    the help text about the scheduler driver disabling a service
    via heartbeat, which is not accurate. What this is really trying
    to say is that the ComputeFilter might think a cached compute
    service has outlasted it's report interval and consider it
    down when using the CachingScheduler, which is the only in-tree
    driver that uses this periodic task config option. So this change
    updates the wording a bit.

    Change-Id: I5a31f2c6c9b1bbe892a423ea4e1820eec19c2117
    Related-Bug: #1749235

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/548873

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/548873
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=2ba48f3fa0c7665bb29c4f8de3e9efd130ae12a3
Submitter: Zuul
Branch: stable/queens

commit 2ba48f3fa0c7665bb29c4f8de3e9efd130ae12a3
Author: Matt Riedemann <email address hidden>
Date: Tue Feb 13 12:12:12 2018 -0500

    Fix and update compute schedulers config guide

    There were several things wrong with this doc since the migration
    in-tree in pike, which are handled here:

    - Updated options to use the current names.
    - Any mention of the periodic_task_interval option is removed
      since this doc is really about the FilterScheduler, and the
      periodic task interval is only used by the CachingScheduler to
      cache and refresh hosts, something the FilterScheduler doesn't do.
      The note about service_down_time is already covered in the help
      text for the periodic_task_interval option.
    - Removed a broken (and oddly placed) link to the block storage
      docs; it's unclear why the compute scheduler docs need to randomly
      link to the block storage docs for schedluing (maybe that was
      super legacy from the nova-volume days).
    - Fixed a broken link to the 'evacuate instances' docs.

    This change does not get into details for the individual filters
    or weighers, only the overview and filter scheduler description.

    Change-Id: Ie5bb9781e1997ad12521b5df5245a78b587a4a20
    Closes-Bug: #1749235
    (cherry picked from commit 8be8d02aa77ebbe6687fa9ec4c74949cde0bf596)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/nova 17.0.2 release.

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/nova 18.0.0.0b1 development milestone.