yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #78565
[Bug 1829877] [NEW] avoid lazy-loading instance.flavor during cold migration
Public bug reported:
During a cold migration the compute API resize() method checks the
instance.flavor:
https://github.com/openstack/nova/blob/53d42de9d970abc290064ab86df272b4d933abc9/nova/compute/api.py#L3587
But this results in a query to the DB:
May 16 19:46:21.368204 ubuntu-bionic-rax-dfw-0006201125 devstack@n-api.service[19505]: DEBUG nova.objects.instance [None req-bde576dc-5de2-4113-982d-025c65b97e8d tempest-MigrationsAdminTest-391674513 tempest-MigrationsAdminTest-391674513] Lazy-loading 'flavor' on Instance uuid 21c45b79-b0f4-48a8-8949-9f480beb5864 {{(pid=19507) obj_load_attr /opt/stack/nova/nova/objects/instance.py:1106}}
May 16 19:46:21.410776 ubuntu-bionic-rax-dfw-0006201125 devstack@n-api.service[19505]: DEBUG nova.compute.api [None req-bde576dc-5de2-4113-982d-025c65b97e8d tempest-MigrationsAdminTest-391674513 tempest-MigrationsAdminTest-391674513] [instance: 21c45b79-b0f4-48a8-8949-9f480beb5864] flavor_id is None. Assuming migration. {{(pid=19507) resize /opt/stack/nova/nova/compute/api.py:3700}}
This could be avoided if we join the flavor when we fetch the instance
from the DB initially:
https://github.com/openstack/nova/blob/53d42de9d970abc290064ab86df272b4d933abc9/nova/api/openstack/compute/migrate_server.py#L55
By passing expected_attrs=['flavor'] to common.get_instance.
** Affects: nova
Importance: Low
Status: Confirmed
** Tags: low-hanging-fruit
** Summary changed:
- avoid lazy-lodaing instance.flavor during cold migration
+ avoid lazy-loading instance.flavor during cold migration
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1829877
Title:
avoid lazy-loading instance.flavor during cold migration
Status in OpenStack Compute (nova):
Confirmed
Bug description:
During a cold migration the compute API resize() method checks the
instance.flavor:
https://github.com/openstack/nova/blob/53d42de9d970abc290064ab86df272b4d933abc9/nova/compute/api.py#L3587
But this results in a query to the DB:
May 16 19:46:21.368204 ubuntu-bionic-rax-dfw-0006201125 devstack@n-api.service[19505]: DEBUG nova.objects.instance [None req-bde576dc-5de2-4113-982d-025c65b97e8d tempest-MigrationsAdminTest-391674513 tempest-MigrationsAdminTest-391674513] Lazy-loading 'flavor' on Instance uuid 21c45b79-b0f4-48a8-8949-9f480beb5864 {{(pid=19507) obj_load_attr /opt/stack/nova/nova/objects/instance.py:1106}}
May 16 19:46:21.410776 ubuntu-bionic-rax-dfw-0006201125 devstack@n-api.service[19505]: DEBUG nova.compute.api [None req-bde576dc-5de2-4113-982d-025c65b97e8d tempest-MigrationsAdminTest-391674513 tempest-MigrationsAdminTest-391674513] [instance: 21c45b79-b0f4-48a8-8949-9f480beb5864] flavor_id is None. Assuming migration. {{(pid=19507) resize /opt/stack/nova/nova/compute/api.py:3700}}
This could be avoided if we join the flavor when we fetch the instance
from the DB initially:
https://github.com/openstack/nova/blob/53d42de9d970abc290064ab86df272b4d933abc9/nova/api/openstack/compute/migrate_server.py#L55
By passing expected_attrs=['flavor'] to common.get_instance.
To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1829877/+subscriptions
Follow ups