| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-03 | |||
| 10:35:17 | kashyap | lyarwood: When I tried to live-modify a guest's name from s/cvm1/64ba587c-57a7-44c0-94ec-5444e5268c4e/ --> libvirt tells me: "well, the guest already has a UUID" | |
| 10:35:59 | kashyap | I tried removing dashes and plug the dash-less UUID into <name/>, it still detects. | |
| 10:36:05 | lyarwood | okay, we can't really change this anyway, it's just annoying | |
| 10:37:08 | kashyap | Yeah, libvirt explicitly rejects a UUID for a domain "name" | |
| 10:37:21 | kashyap | s/Yeah,/Yeah, and/ | |
| 10:38:17 | lyarwood | not sure how we'd do that after the test run | |
| 10:38:56 | kashyap | lyarwood: You mean, you want a file logged a mapping of UUIDs with instance names? | |
| 10:40:11 | lyarwood | yeah but I don't think that's going to be possible *after* tempest has already deleted everything | |
| 10:40:35 | kashyap | Hmm, true | |
| 10:43:51 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Improve logging https://review.opendev.org/c/openstack/nova/+/778407 | |
| 10:55:12 | lyarwood | gibi: https://review.opendev.org/q/topic:spec/libvirt-default-machine-type+status:open should be ready for review this week btw, not sure if you want to move it into a runway slot or not. | |
| 11:06:24 | gibi | lyarwood: I added to my queue, I see stephenfin is already +2 so I prioritize this | |
| 11:07:34 | lyarwood | gibi: yup thanks, I'm around all day for respins etc so let me know if you have any questions | |
| 11:07:42 | gibi | OK | |
| 11:23:07 | slaweq | hi nova team :) | |
| 11:23:11 | gibi | slaweq: hi! | |
| 11:23:14 | slaweq | I need Your help with some ci issue | |
| 11:23:31 | slaweq | we saw, mostly in stable/train errors like in https://a574f9c0fd4ca92b7603-2045be852d43868eb95da6cc3429b40d.ssl.cf2.rackcdn.com/777334/2/check/neutron-tempest-dvr-ha-multinode-full/44d0207/testr_results.html recently | |
| 11:23:59 | slaweq | do You have any idea why there is no host for cold-migration or resize in that job? | |
| 11:25:17 | slaweq | hi gibi :) | |
| 11:25:22 | bauzas | stephenfin: saw my weak -1 on your apidb change for Ocata ? https://review.opendev.org/c/openstack/nova/+/759402/4 | |
| 11:25:55 | bauzas | I don't really to vote -1 on a commit msg miss, but I feel this is important to comment the fact that we don't care about a specific DB migration | |
| 11:26:03 | bauzas | really like* | |
| 11:26:19 | bauzas | and a commit msg can't be patched by a FUP... | |
| 11:28:41 | gibi | slaweq: is it happens all the time or just sometimes? | |
| 11:29:46 | slaweq | gibi: on train I think that all the time | |
| 11:29:51 | slaweq | or at least very often | |
| 11:30:12 | slaweq | gibi: in scheduler log I see something like: | |
| 11:30:15 | slaweq | Mar 03 10:32:31.142569 ubuntu-bionic-rax-ord-0023272867 nova-scheduler[24545]: INFO nova.scheduler.host_manager [None req-657ecf60-4fba-4168-9613-3c57d01bdcb7 tempest-MinBwAllocationPlacementTest-1587030703 tempest-MinBwAllocationPlacementTest-1587030703-project-admin] Host filter ignoring hosts: ubuntu-bionic-rax-ord-0023272867 | |
| 11:30:34 | gibi | slaweq: ack, give me some time to dive into the logs | |
| 11:30:39 | slaweq | and then next message that " There are 0 hosts available but 1 instances requested to build." | |
| 11:30:52 | slaweq | gibi: sure, I will open LP for that to track it | |
| 11:30:55 | slaweq | thx for help | |
| 11:31:02 | gibi | slaweq: thanks for reporting it | |
| 11:31:10 | slaweq | it's not very urgent for us as this is non-voting job :) | |
| 11:31:16 | slaweq | so take Your time | |
| 11:31:36 | brinzhang | stephenfin: *(songwenping_)from none -> vnc need to reboot, from vnc -> vnc donnot need to reboot* I still think from docs or a bug trace this is a good choice | |
| 11:40:24 | slaweq | gibi: https://bugs.launchpad.net/nova/+bug/1917610 | |
| 11:40:26 | openstack | Launchpad bug 1917610 in OpenStack Compute (nova) "Migration and resize tests from tempest.scenario.test_minbw_allocation_placement.MinBwAllocationPlacementTest failing in neutron-tempest-dvr-ha-multinode-full" [Undecided,New] | |
| 11:40:26 | slaweq | reported | |
| 11:54:51 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add missed accel_uuids for _poll_shelved_instances https://review.opendev.org/c/openstack/nova/+/778440 | |
| 11:57:41 | stephenfin | brinzhang: I don't :( | |
| 11:57:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Ocata database migrations https://review.opendev.org/c/openstack/nova/+/759402 | |
| 11:58:03 | stephenfin | bauzas: Is that okay? ^ | |
| 11:58:12 | stephenfin | I'll rebase the rest of the series on top if so | |
| 11:58:24 | brinzhang | bauzas: fixed that bug, pls see https://review.opendev.org/c/openstack/nova/+/778440 | |
| 12:22:04 | brinzhang | stephenfin: we will re-search, and make sure whether can reset passwrod and without restart the server | |
| 12:22:44 | brinzhang | s/and// | |
| 12:28:08 | gibi | slaweq: responded in https://bugs.launchpad.net/nova/+bug/1917610 with alternatives to either fix the job definition in neutron or fix the test precoditions in tempest | |
| 12:28:09 | openstack | Launchpad bug 1917610 in OpenStack Compute (nova) "Migration and resize tests from tempest.scenario.test_minbw_allocation_placement.MinBwAllocationPlacementTest failing in neutron-tempest-dvr-ha-multinode-full" [Undecided,New] | |
| 12:37:28 | slaweq | gibi: thx | |
| 12:37:42 | slaweq | gibi: proposed fix https://review.opendev.org/c/openstack/tempest/+/778451 and dnm test patch https://review.opendev.org/c/openstack/neutron/+/778452 | |
| 12:37:49 | slaweq | let's check if that will help | |
| 12:38:01 | gibi | tht was fast :) | |
| 12:38:50 | gibi | wondering why this is only hit on stable/train. It should fail on any newer than train too | |
| 12:47:30 | gibi | lyarwood: I see an issue in https://review.opendev.org/c/openstack/nova/+/769548/12/nova/cmd/manage.py#2631 | |
| 12:53:53 | lyarwood | ah sorry, I thought I had sorted that out | |
| 12:54:05 | gibi | the update command looks good, you only missed the get | |
| 12:56:26 | lyarwood | yeah my bad | |
| 13:47:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Fix ImageBackendFixture not to support direct_snapshot https://review.opendev.org/c/openstack/nova/+/778462 | |
| 13:49:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Simplify device_path check in _detach_encryptor https://review.opendev.org/c/openstack/nova/+/778463 | |
| 13:49:55 | gibi | lyarwood: also could you check my question in https://review.opendev.org/c/openstack/nova/+/774898/7/nova/tests/functional/libvirt/test_machine_type.py#388 and the follow up I pushed? | |
| 13:50:38 | lyarwood | gibi: I was about to ask if that was related | |
| 13:50:42 | lyarwood | looking now | |
| 13:50:57 | lyarwood | just had to push the above fix before switching back to this | |
| 13:51:07 | gibi | yepp it is a fixture fix | |
| 13:51:15 | gibi | to enable the unshelve test | |
| 13:53:17 | lyarwood | wonderful | |
| 13:54:00 | lyarwood | gibi: would you mind if I just pulled in that fix ahead of the new func test | |
| 13:54:17 | gibi | lyarwood: sure | |
| 13:54:20 | lyarwood | thanks | |
| 13:59:25 | gibi | lyarwood: overall I'm +2 on the machine type series I will quickly re-review it once you respin it | |
| 14:00:32 | lyarwood | gibi: thanks, just waiting on a local func run now and I'll push | |
| 14:00:42 | gibi | ack | |
| 14:04:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-manage: Add libvirt get_machine_type command https://review.opendev.org/c/openstack/nova/+/769548 | |
| 14:04:51 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-manage: Add libvirt update_machine_type command https://review.opendev.org/c/openstack/nova/+/774896 | |
| 14:04:51 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-manage: Add libvirt list_unset_machine_type command https://review.opendev.org/c/openstack/nova/+/774897 | |
| 14:04:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: nova-status: Add hw_machine_type check for libvirt instances https://review.opendev.org/c/openstack/nova/+/770643 | |
| 14:04:52 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Fix ImageBackendFixture not to support direct_snapshot https://review.opendev.org/c/openstack/nova/+/778462 | |
| 14:04:53 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add a config update workflow test for [libvirt]hw_machine_type https://review.opendev.org/c/openstack/nova/+/774898 | |
| 14:04:53 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add admin docs for configuring and updating machine types https://review.opendev.org/c/openstack/nova/+/774899 | |
| 14:05:05 | gibi | on it ^^ | |
| 14:13:33 | gibi | stephenfin: you blessing is needed on some of the patches in the machine_type series as lyarwood respined it | |
| 14:13:45 | stephenfin | ack, will take a look shortly | |
| 14:13:51 | stephenfin | (in a meeting atm :() | |
| 14:16:00 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Simplify device_path check in _detach_encryptor https://review.opendev.org/c/openstack/nova/+/778463 | |
| 14:20:08 | stephenfin | gibi: Need your +2 again on https://review.opendev.org/c/openstack/nova/+/759402/ too. I just changed the commit message per bauzas' request | |
| 14:20:13 | stephenfin | Please and thanks :) | |
| 14:20:22 | gibi | on it | |
| 14:20:46 | bauzas | gibi: stephenfin: fast approved as this is only a commit msg update | |
| 14:20:53 | stephenfin | sweet | |
| 14:21:02 | stephenfin | I'll rebase the rest of that series on top of that shortly so | |
| 14:21:02 | gibi | stading down... | |
| 14:25:50 | gibi | anyhow it failed in zuul with a valid reason. unused import /o\ | |
| 14:27:01 | stephenfin | oh, weird | |
| 14:30:56 | stephenfin | I think I broke Gerrit | |
| 14:31:07 | stephenfin | Stuck on publishing edit for at least 2 minutes /o\ | |
| 14:31:14 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Ocata database migrations https://review.opendev.org/c/openstack/nova/+/759402 | |
| 14:31:18 | stephenfin | phew | |
| 14:31:32 | stephenfin | gibi: bauzas: Removed that import. Conflict was due to I532c7918a8e2c887f29d2f0e1e33b80f2b3a7507 | |