From 2a50107606918c3f887619c4d73f05618fae6c42 Mon Sep 17 00:00:00 2001 From: Eric Fried Date: Tue, 28 Aug 2018 14:46:00 -0500 Subject: [PATCH] reshaper gabbit: Nix comments re doubled max_unit As noted in a review [1] on the reshaper series, the comments about doubling max_unit were obsolete (fixed via [2]), and are hereby removed. [1] https://review.openstack.org/#/c/576927/35/nova/tests/functional/api/openstack/placement/gabbits/reshaper.yaml@222 [2] I8ba378ff5eeaf6c9cca11c5874708a17d4640097 Change-Id: Iaf3df651a76705d0f64182c46c665f037ea51b68 --- .../openstack/placement/gabbits/reshaper.yaml | 44 ------------------- 1 file changed, 44 deletions(-) diff --git a/nova/tests/functional/api/openstack/placement/gabbits/reshaper.yaml b/nova/tests/functional/api/openstack/placement/gabbits/reshaper.yaml index e5752043ff6c..59d9916b0ab9 100644 --- a/nova/tests/functional/api/openstack/placement/gabbits/reshaper.yaml +++ b/nova/tests/functional/api/openstack/placement/gabbits/reshaper.yaml @@ -155,10 +155,6 @@ tests: total: 2048 step_size: 10 min_unit: 10 - # this was 600 originally but we reset it to 1200 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 1200 # This resource provider does not exist. '39bafc00-3fff-444d-b87a-2ead3f866e05': @@ -166,10 +162,6 @@ tests: inventories: VCPU: total: 10 - # this was 4 originally but we reset it to 8 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 8 # these consumer generations are all 1 because they have # previously allocated @@ -216,10 +208,6 @@ tests: total: 2048 step_size: 10 min_unit: 10 - # this was 600 originally but we reset it to 1200 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 1200 bad_field: moo $ENVIRON['ALT_RP_UUID']: @@ -227,10 +215,6 @@ tests: inventories: VCPU: total: 10 - # this was 4 originally but we reset it to 8 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 8 # these consumer generations are all 1 because they have # previously allocated @@ -278,20 +262,12 @@ tests: total: 2048 step_size: 10 min_unit: 10 - # this was 600 originally but we reset it to 1200 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 1200 $ENVIRON['ALT_RP_UUID']: resource_provider_generation: 0 inventories: VCPU: total: 10 - # this was 4 originally but we reset it to 8 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 8 # these consumer generations are all 1 because they have # previously allocated @@ -342,20 +318,12 @@ tests: total: 2048 step_size: 10 min_unit: 10 - # this was 600 originally but we reset it to 1200 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 1200 $ENVIRON['ALT_RP_UUID']: resource_provider_generation: 0 inventories: VCPU: total: 10 - # this was 4 originally but we reset it to 8 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 8 # these consumer generations are all 1 because they have # previously allocated @@ -409,10 +377,6 @@ tests: inventories: VCPU: total: 10 - # this was 4 originally but we reset it to 8 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 8 # these consumer generations are all 1 because they have # previously allocated @@ -464,20 +428,12 @@ tests: total: 2048 step_size: 10 min_unit: 10 - # this was 600 originally but we reset it to 1200 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 1200 $ENVIRON['ALT_RP_UUID']: resource_provider_generation: 0 inventories: VCPU: total: 10 - # this was 4 originally but we reset it to 8 - # here because the fixture allocated twice for - # the same consumer and we can't do that from - # the api. max_unit: 8 # these consumer generations are all 1 because they have # previously allocated