From 720dd3975850c7ba1bab29bee10aacbe61280993 Mon Sep 17 00:00:00 2001 From: "Chandan Kumar (raukadah)" Date: Fri, 26 Jun 2020 19:15:38 +0530 Subject: [PATCH] Update DEFAULT_FLAVOR_{RAM, _ALT} >= 128 On updating the cirros image to 0.5.1, volume boot pattern tests are failing as server while booting entering into kernel panic mode due to less RAM. We need RAM more than 64, will fix the issue. Let's update the value used in devstack testing https://opendev.org/openstack/devstack/src/commit/a13ab8db2dbd959b5a1f52ac03d3fca07a19eb04/lib/tempest#L206 and re-use the same as default. Related-Bug: #1885279 Change-Id: Ia172c9d719c51e72b320751e51b48861ab0c9fc9 Signed-off-by: Chandan Kumar (raukadah) --- config_tempest/constants.py | 4 ++-- config_tempest/tests/test_flavors.py | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/config_tempest/constants.py b/config_tempest/constants.py index 0a3721ea..c6df515e 100644 --- a/config_tempest/constants.py +++ b/config_tempest/constants.py @@ -27,8 +27,8 @@ DEFAULT_IMAGE = ("https://download.cirros-cloud.net/0.4.0/" "cirros-0.4.0-x86_64-disk.img") DEFAULT_IMAGE_FORMAT = 'qcow2' -DEFAULT_FLAVOR_RAM = 64 -DEFAULT_FLAVOR_RAM_ALT = 128 +DEFAULT_FLAVOR_RAM = 128 +DEFAULT_FLAVOR_RAM_ALT = 192 DEFAULT_FLAVOR_DISK = 1 DEFAULT_FLAVOR_VCPUS = 1 diff --git a/config_tempest/tests/test_flavors.py b/config_tempest/tests/test_flavors.py index 79281a18..05f86d2c 100644 --- a/config_tempest/tests/test_flavors.py +++ b/config_tempest/tests/test_flavors.py @@ -42,7 +42,7 @@ class TestFlavors(BaseConfigTempestTest): mock_function = mock.Mock(return_value=return_value) self.useFixture(MonkeyPatch(self.CLIENT_MOCK + '.list_flavors', mock_function)) - self.Service = Flavors(self.client, True, self.conf, 64, 1) + self.Service = Flavors(self.client, True, self.conf, 128, 1) def test_create_tempest_flavors(self): self.Service.flavor_list = [] @@ -52,8 +52,8 @@ class TestFlavors(BaseConfigTempestTest): self.Service.create_tempest_flavors() self.assertEqual(self.conf.get('compute', 'flavor_ref'), "FakeID") self.assertEqual(self.conf.get('compute', 'flavor_ref_alt'), "FakeID") - calls = [mock.call('m1.nano', 64, 1, 1, no_rng=False), - mock.call('m1.micro', 128, 1, 1, no_rng=False)] + calls = [mock.call('m1.nano', 128, 1, 1, no_rng=False), + mock.call('m1.micro', 192, 1, 1, no_rng=False)] mock_function.assert_has_calls(calls, any_order=True) def check_call_of_discover_smallest_flavor(self):