| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-17 | |||
| 07:42:09 | bauzas | amorin: cool, what was the issue ? | |
| 07:42:14 | gibi | bauzas: good morning | |
| 08:09:10 | openstackgerrit | huanhongda proposed openstack/nova master: Fix instance delete stuck in deleting task_state https://review.openstack.org/598084 | |
| 10:38:48 | openstack | bug 1798163 in OpenStack Compute (nova) "Placement incomplete consumers online migration fails" [Critical,In progress] https://launchpad.net/bugs/1798163 - Assigned to Matt Riedemann (mriedem) | |
| 10:38:48 | openstackgerrit | Merged openstack/nova master: Add recreate test for bug 1798163 https://review.openstack.org/611113 | |
| 10:41:54 | kashyap | stephenfin: Hey, is there a less-strain-on-the-eye version of this? -- https://docs.openstack.org/nova/rocky/configuration/sample-config.html | |
| 10:42:15 | kashyap | Renderings for older releases had a table for config files | |
| 10:42:28 | kashyap | Now it is all one big sphaghetti noodle "sample-config.html" file | |
| 10:44:23 | kashyap | Ah, found it - https://docs.openstack.org/nova/rocky/configuration/config.html | |
| 10:44:26 | kashyap | stephenfin: Disregard me | |
| 12:04:51 | openstackgerrit | Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.openstack.org/608704 | |
| 12:18:21 | openstackgerrit | Merged openstack/nova master: Use unique consumer_id when doing online data migration https://review.openstack.org/611115 | |
| 13:02:25 | openstackgerrit | Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.openstack.org/608704 | |
| 13:05:02 | openstackgerrit | Merged openstack/nova master: Fix formatting non-templated cell URLs with no config https://review.openstack.org/611094 | |
| 13:21:18 | pooja_jadhav | In this functional tests, is it creating instance in real?? | |
| 13:21:18 | pooja_jadhav | mriedem: https://review.openstack.org/#/c/608771/1/nova/tests/functional/test_boot_from_volume.py | |
| 13:22:41 | mriedem | it creates a server in the sqlite database using the fake virt driver | |
| 13:22:45 | mriedem | i'm not sure what you mean by "real" | |
| 13:22:53 | amorin | bauzas the issue is that we are upgrading our openstack from juno to newton | |
| 13:23:02 | pooja_jadhav | real means in actual environment | |
| 13:23:06 | pooja_jadhav | like | |
| 13:23:17 | amorin | and some of our hosts were not configured correctly AFAIK | |
| 13:23:18 | pooja_jadhav | when we do nova list, we can see that instance | |
| 13:23:53 | amorin | maybe that was because nova rely on glanceclient, and glanceclient was not upgraded | |
| 13:23:55 | mriedem | pooja_jadhav: well the test doesn't list instances, but it's able to show the instance, otherwise it wouldn't be able to wait for its status to be ACTIVE | |
| 13:25:04 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Use unique consumer_id when doing online data migration https://review.openstack.org/611315 | |
| 13:25:04 | openstack | bug 1798163 in OpenStack Compute (nova) rocky "Placement incomplete consumers online migration fails" [Critical,Confirmed] https://launchpad.net/bugs/1798163 - Assigned to lihuayuliang (lihuayuliang) | |
| 13:25:04 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Add recreate test for bug 1798163 https://review.openstack.org/611314 | |
| 13:25:25 | mriedem | lyarwood: bauzas: pretty serious upgrade issue if you can review those backports quick ^ | |
| 13:26:11 | pooja_jadhav | mriedem: then after getting test passed, these instance should be deleted. Is it deleting by any existing mechanism or we need to delete by calling delete API.?? | |
| 13:26:43 | mriedem | pooja_jadhav: each test case has the db setup and torn down | |
| 13:26:56 | mriedem | so we don't need to explicitly delete resources since each test case has a fresh db | |
| 13:27:25 | pooja_jadhav | mriedem: Ohk, Thank you :) | |
| 13:30:42 | pooja_jadhav | mriedem: Can u plz locate to the code where this db setup and tear down is added for the functional tests? | |
| 13:31:04 | mriedem | pooja_jadhav: see nova.test.TestCase.setUp | |
| 13:31:08 | mriedem | the database fixtures in there | |
| 13:32:45 | pooja_jadhav | mriedem : thnnks :) | |
| 13:53:30 | openstackgerrit | Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.openstack.org/608704 | |
| 13:54:40 | openstackgerrit | Merged openstack/nova master: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/609518 | |
| 13:54:50 | mriedem | tssurya: you want to backport this to rocky? https://review.openstack.org/#/c/611094/ | |
| 13:55:37 | tssurya | mriedem: yea, doing it | |
| 13:55:58 | mriedem | artom: i see you wasted no time https://review.openstack.org/#/q/Icb0bdaf454935b3713c35339394d260b33520de5 | |
| 13:56:42 | dansmith | mriedem: we have some pressure on that one | |
| 13:56:49 | dansmith | rocky should be okay now yeah? | |
| 13:56:54 | tssurya | yea | |
| 13:57:52 | mriedem | dansmith: i just +2d it | |
| 13:58:34 | dansmith | ah was just looking at the conflict, | |
| 13:58:38 | dansmith | but I see you called it out | |
| 14:03:03 | mriedem | anyone know if tripleo is running nova-status upgrade check yet? | |
| 14:03:07 | mriedem | either on install and/or upgrade? | |
| 14:04:34 | mriedem | looks like only OSA and kolla-ansible so far | |
| 14:08:07 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/611326 | |
| 14:09:19 | openstackgerrit | Surya Seetharaman proposed openstack/nova stable/rocky: Fix formatting non-templated cell URLs with no config https://review.openstack.org/611327 | |
| 14:11:15 | bauzas | mriedem: lyarwood is on PTO for the next two weeks, but lemme look | |
| 14:16:46 | tssurya | if any cores have some time (other than mriedem and efried): would be good to have this in https://review.openstack.org/#/c/611162/ | |
| 14:17:03 | mriedem | i was just going to lobby for that :) | |
| 14:17:06 | mriedem | since it's an upgrade impact | |
| 14:17:19 | tssurya | :) | |
| 14:21:19 | bauzas | tssurya: mriedem: mmm, is that only an Ironic thing ? | |
| 14:21:40 | bauzas | mriedem: I was looking for like 2 weeks why some ComputeNode UUID was changing from a libvirt node | |
| 14:21:53 | tssurya | its an only ironic driver thing | |
| 14:21:55 | bauzas | anyway, +Wing | |
| 14:22:01 | tssurya | thanks bauzas | |
| 14:22:42 | bauzas | tssurya: mriedem: can't see a stable/rocky one, please ping me the change once done | |
| 14:22:52 | tssurya | yea proposing one now | |
| 14:22:59 | bauzas | of course, I'll +2 it once the master lands | |
| 14:24:15 | mriedem | bauzas: this is a fun reqspec regression thing too https://review.openstack.org/#/q/topic:bug/1797580+(status:open+OR+status:merged) | |
| 14:24:27 | mriedem | goes back to queens | |
| 14:24:35 | bauzas | mriedem: after 2 weeks of internal bug triage, I love you | |
| 14:24:43 | bauzas | because you help me doing other stuff | |
| 14:24:49 | tssurya | yea we hit that too the marker stuff :( | |
| 14:24:56 | mriedem | ^ isn't marker stuff | |
| 14:25:02 | bauzas | oh this one | |
| 14:25:07 | bauzas | the requested_destination field | |
| 14:25:21 | mriedem | tssurya: https://review.openstack.org/#/c/610974/ | |
| 14:25:51 | tssurya | ah hehe yea | |
| 14:25:53 | tssurya | sorry | |
| 14:26:20 | mriedem | but yeah would be good to get that in also since it affects upgrades | |
| 14:26:24 | mriedem | johnthetubaguy: are you around? | |
| 14:26:58 | openstackgerrit | Surya Seetharaman proposed openstack/nova stable/rocky: Ignore uuid if already set in ComputeNode.update_from_virt_driver https://review.openstack.org/611337 | |
| 14:27:09 | artom | mriedem, yeah, like I said, downstream pressure :) | |
| 14:27:25 | tssurya | bauzas, efried: ^ the backport to rocky for after the master merges | |
| 14:28:11 | mriedem | +1 | |
| 14:34:26 | stephenfin | Can someone help me understand what's going on here? paste.openstack.org/show/732338/ | |
| 14:34:56 | stephenfin | Was discussing it with bauzas earlier today. We've configured 'cpu_allocation=1' and yet it still seems like overallocation is happening | |
| 14:36:04 | bauzas | stephenfin: like I said to you, we should check the instances number | |
| 14:36:49 | stephenfin | bauzas: Yeah, there's one instance on that host before we do the host-evacuate. Once we do that, two instances land on it | |
| 14:37:09 | stephenfin | One instance has 2 vcpu, the other has 1 vcpu | |
| 14:37:12 | bauzas | given vcpus_used is generated by looking up the instances | |
| 14:37:24 | bauzas | host-evacuate is racy | |
| 14:37:44 | bauzas | you should really avoid it, and do two evacuations concurrently | |
| 14:37:57 | bauzas | I thought we commented that btw. | |
| 14:38:21 | stephenfin | Yup, vcpus_used appears to be correct. Compare lines 24-29 to 90-95 | |
| 14:38:42 | bauzas | https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/contrib/host_evacuate.py | |
| 14:39:04 | stephenfin | The second hypervisor (dcfb3fb7) goes from 5/6 used to 7/6 | |
| 14:41:02 | stephenfin | bauzas: But its done client side. It seems odd that the scheduler wouldn't honour overcommit ratios just because the requests came in at the same time. Is this known behavior? | |
| 14:42:13 | mriedem | is the target host in an aggregate with an allocation ratio metadata? | |
| 14:42:38 | stephenfin | sec - checking | |
| 14:42:56 | mriedem | compute-0.localdomain was forced down but was the service actually stopped? | |
| 14:44:31 | bauzas | otp but looking | |