From b01676c41859a40568d81f846e25ff88818cc27d Mon Sep 17 00:00:00 2001 From: Stephen Finucane Date: Thu, 8 Aug 2019 10:40:14 +0100 Subject: [PATCH] Remove 'hw:cpu_policy', 'hw:mem_page_size' extra specs from API samples In a future change, the use of 'hw:cpu_policy' will require a host to report PCPU inventory. Rather than modify the fake driver used in these tests to report such inventory, just use a different extra spec, 'hw:numa_nodes'. This has the added bonus of being supported by both the libvirt and Hyper-V virt drivers, unlike 'hw:cpu_policy' and 'hw:mem_page_size', which are only supported by the libvirt virt driver. Change-Id: Id203dc07f08557b1b094ec72e1df3493ec9524b1 Signed-off-by: Stephen Finucane --- api-guide/source/down_cells.rst | 6 ++---- doc/api_samples/flavors/v2.61/flavors-detail-resp.json | 3 +-- doc/api_samples/flavors/v2.75/flavors-detail-resp.json | 3 +-- doc/api_samples/servers/v2.47/server-get-resp.json | 3 +-- doc/api_samples/servers/v2.47/servers-details-resp.json | 3 +-- .../servers/v2.63/server-action-rebuild-resp.json | 3 +-- doc/api_samples/servers/v2.63/server-get-resp.json | 3 +-- doc/api_samples/servers/v2.63/server-update-resp.json | 3 +-- doc/api_samples/servers/v2.63/servers-details-resp.json | 3 +-- .../servers/v2.66/servers-details-with-changes-before.json | 3 +-- nova/tests/fixtures.py | 3 +-- .../api_samples/flavors/v2.61/flavors-detail-resp.json.tpl | 3 +-- .../api_samples/flavors/v2.75/flavors-detail-resp.json.tpl | 3 +-- .../api_samples/servers/v2.47/server-get-resp.json.tpl | 3 +-- .../api_samples/servers/v2.47/servers-details-resp.json.tpl | 3 +-- .../servers/v2.63/server-action-rebuild-resp.json.tpl | 3 +-- .../api_samples/servers/v2.63/server-get-resp.json.tpl | 3 +-- .../api_samples/servers/v2.63/server-update-resp.json.tpl | 3 +-- .../api_samples/servers/v2.63/servers-details-resp.json.tpl | 3 +-- .../v2.66/servers-details-with-changes-before.json.tpl | 3 +-- 20 files changed, 21 insertions(+), 42 deletions(-) diff --git a/api-guide/source/down_cells.rst b/api-guide/source/down_cells.rst index c7bdb83f5d..d77ad27312 100644 --- a/api-guide/source/down_cells.rst +++ b/api-guide/source/down_cells.rst @@ -130,8 +130,7 @@ behavior are described below: "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, @@ -266,8 +265,7 @@ behavior are described below: "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/flavors/v2.61/flavors-detail-resp.json b/doc/api_samples/flavors/v2.61/flavors-detail-resp.json index f615998043..980603f890 100644 --- a/doc/api_samples/flavors/v2.61/flavors-detail-resp.json +++ b/doc/api_samples/flavors/v2.61/flavors-detail-resp.json @@ -143,8 +143,7 @@ "rxtx_factor": 1.0, "description": null, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" } }, { diff --git a/doc/api_samples/flavors/v2.75/flavors-detail-resp.json b/doc/api_samples/flavors/v2.75/flavors-detail-resp.json index 58ca0e44ba..7adc9084df 100644 --- a/doc/api_samples/flavors/v2.75/flavors-detail-resp.json +++ b/doc/api_samples/flavors/v2.75/flavors-detail-resp.json @@ -143,8 +143,7 @@ "rxtx_factor": 1.0, "description": null, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" } }, { diff --git a/doc/api_samples/servers/v2.47/server-get-resp.json b/doc/api_samples/servers/v2.47/server-get-resp.json index e7d39431cd..406b6d41aa 100644 --- a/doc/api_samples/servers/v2.47/server-get-resp.json +++ b/doc/api_samples/servers/v2.47/server-get-resp.json @@ -36,8 +36,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/servers/v2.47/servers-details-resp.json b/doc/api_samples/servers/v2.47/servers-details-resp.json index 626a0f7d72..6c6c2804be 100644 --- a/doc/api_samples/servers/v2.47/servers-details-resp.json +++ b/doc/api_samples/servers/v2.47/servers-details-resp.json @@ -37,8 +37,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/servers/v2.63/server-action-rebuild-resp.json b/doc/api_samples/servers/v2.63/server-action-rebuild-resp.json index 546e7cb8bb..e1db18192a 100644 --- a/doc/api_samples/servers/v2.63/server-action-rebuild-resp.json +++ b/doc/api_samples/servers/v2.63/server-action-rebuild-resp.json @@ -17,8 +17,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/servers/v2.63/server-get-resp.json b/doc/api_samples/servers/v2.63/server-get-resp.json index c644f70a85..82c3b9bcf2 100644 --- a/doc/api_samples/servers/v2.63/server-get-resp.json +++ b/doc/api_samples/servers/v2.63/server-get-resp.json @@ -36,8 +36,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/servers/v2.63/server-update-resp.json b/doc/api_samples/servers/v2.63/server-update-resp.json index 5a47c0c7ce..9f2bbe6728 100644 --- a/doc/api_samples/servers/v2.63/server-update-resp.json +++ b/doc/api_samples/servers/v2.63/server-update-resp.json @@ -17,8 +17,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/servers/v2.63/servers-details-resp.json b/doc/api_samples/servers/v2.63/servers-details-resp.json index 3810f0f19f..089a07ff02 100644 --- a/doc/api_samples/servers/v2.63/servers-details-resp.json +++ b/doc/api_samples/servers/v2.63/servers-details-resp.json @@ -37,8 +37,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/doc/api_samples/servers/v2.66/servers-details-with-changes-before.json b/doc/api_samples/servers/v2.66/servers-details-with-changes-before.json index 9d33052288..1f3c91460e 100644 --- a/doc/api_samples/servers/v2.66/servers-details-with-changes-before.json +++ b/doc/api_samples/servers/v2.66/servers-details-with-changes-before.json @@ -37,8 +37,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/fixtures.py b/nova/tests/fixtures.py index bee0ed04b8..a538841292 100644 --- a/nova/tests/fixtures.py +++ b/nova/tests/fixtures.py @@ -739,8 +739,7 @@ class DefaultFlavorsFixture(fixtures.Fixture): defaults = {'rxtx_factor': 1.0, 'disabled': False, 'is_public': True, 'ephemeral_gb': 0, 'swap': 0} extra_specs = { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" } default_flavors = [ objects.Flavor(context=ctxt, memory_mb=512, vcpus=1, diff --git a/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.61/flavors-detail-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.61/flavors-detail-resp.json.tpl index efc74ae6f1..23bfdf27e5 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.61/flavors-detail-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.61/flavors-detail-resp.json.tpl @@ -143,8 +143,7 @@ "rxtx_factor": 1.0, "description": null, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" } }, { diff --git a/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.75/flavors-detail-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.75/flavors-detail-resp.json.tpl index 88bd02e2cc..bb4092aebc 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.75/flavors-detail-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/flavors/v2.75/flavors-detail-resp.json.tpl @@ -143,8 +143,7 @@ "rxtx_factor": 1.0, "description": null, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" } }, { diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/server-get-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/server-get-resp.json.tpl index 809aa81234..5568f7ab5e 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/server-get-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/server-get-resp.json.tpl @@ -21,8 +21,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/servers-details-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/servers-details-resp.json.tpl index 53fe2c90bc..cba2f9d102 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/servers-details-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.47/servers-details-resp.json.tpl @@ -22,8 +22,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-action-rebuild-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-action-rebuild-resp.json.tpl index 7b0b9038ce..76b93ab767 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-action-rebuild-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-action-rebuild-resp.json.tpl @@ -16,8 +16,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-get-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-get-resp.json.tpl index 5dab6aa0e0..ac6c385ce6 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-get-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-get-resp.json.tpl @@ -21,8 +21,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:cpu_policy": "dedicated", - "hw:mem_page_size": "2048" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-update-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-update-resp.json.tpl index 402222ce29..d96971c4c4 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-update-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/server-update-resp.json.tpl @@ -17,8 +17,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/servers-details-resp.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/servers-details-resp.json.tpl index 2a97616333..a64afc4a34 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/servers-details-resp.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.63/servers-details-resp.json.tpl @@ -22,8 +22,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512, diff --git a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.66/servers-details-with-changes-before.json.tpl b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.66/servers-details-with-changes-before.json.tpl index 7cfb1ed2ef..9d42bbd499 100644 --- a/nova/tests/functional/api_sample_tests/api_samples/servers/v2.66/servers-details-with-changes-before.json.tpl +++ b/nova/tests/functional/api_sample_tests/api_samples/servers/v2.66/servers-details-with-changes-before.json.tpl @@ -22,8 +22,7 @@ "disk": 1, "ephemeral": 0, "extra_specs": { - "hw:mem_page_size": "2048", - "hw:cpu_policy": "dedicated" + "hw:numa_nodes": "1" }, "original_name": "m1.tiny.specs", "ram": 512,