Compute schedulers in Configuration Reference - AggregateImagePropertiesIsolation filter example

[Matt Riedemann]

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

The description of the AggregateImagePropertiesIsolation filter:

https://docs.openstack.org/ocata/config-reference/compute/schedulers.html#aggregateimagepropertiesisolation

Is trying to show a link between aggregate metadata and image metadata for a windows operating system, but the CLI output in the example showing the details of the aggregate and the image don't actually have anything about windows in them, which is confusing. So it appears the example is incomplete.

-----------------------------------
Release: 15.0.0 on 2017-05-24 10:51
SHA: f02a55a3b69bf41f54efa19617f15a824c4f25aa
Source: https://git.openstack.org/cgit/openstack/openstack-manuals/tree/doc/config-reference/source/compute/schedulers.rst
URL: https://docs.openstack.org/ocata/config-reference/compute/schedulers.html

[Ben Silverman (tersian)]

You don't think the text before the examples is enough to point out they are windows instances?

IE:

"For example, the following aggregate myWinAgg has the Windows operating system as metadata (named ‘windows’):"

or in the second example:

"In this example, because the following Win-2012 image has the windows property, it boots on the sf-devel host (all other filters being equal)"

With the command being: "$ openstack image show Win-2012"

What other indications would you suggest?

[Matt Riedemann (mriedem)]

The table output doesn't actually have any metadata in it, for the aggregate or image, to show a 'windows' metadata item. Descriptions with example output that don't match isn't helpful.

It also looks like this is semi duplicate of bug 1684261.

[Matt Riedemann (mriedem)]

e.g.

$ openstack aggregate show MyWinAgg
+-------------------+----------------------------+
| Field | Value |
+-------------------+----------------------------+
| availability_zone | zone1 |
| created_at | 2017-01-01T15:36:44.000000 |
| deleted | False |
| deleted_at | None |
| hosts | [u'sf-devel'] |
| id | 1 |
| name | test |
| properties | |
| updated_at | None |
+-------------------+----------------------------+

^ is from the docs bug "properties" is empty, which is really confusing.

And:

$ openstack image show Win-2012
+------------------+------------------------------------------------------+
| Field | Value |
+------------------+------------------------------------------------------+
| checksum | ee1eca47dc88f4879d8a229cc70a07c6 |
| container_format | bare |
| created_at | 2016-12-13T09:30:30Z |
| disk_format | qcow2 |
| ...

Doesn't show anything in there about OS.