| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-31 | |||
| 09:08:51 | kashyap | amoralej: I have joined #rdo on Libera net, if you want to sort it out there. :) | |
| 09:09:07 | amoralej | kashyap, better #rdo in oftc, please :) | |
| 09:09:18 | kashyap | Ah, I didn't know it's on OFTC, joining | |
| 09:35:02 | andry_g | Hi all, we had inserted some vms into an host group for anti-affinity rules after creation, the vms were been created some months ago, we know that it's not possible by a doc but we tried to insert 2 records into db nova_api table instance_group_member. It's works fine but we have a problem if we try to do a resize of vm, we have this error: 2022-03-31 11:12:23.455 45106 ERROR oslo_messaging.rpc.server [req-be200e2d-af30-4f16-8571-f3a04533dbd6 5 | |
| 09:35:33 | andry_g | Exception during message handling: AttributeError: 'NoneType' object has no attribute 'hosts' Seems that something is missing in db. Could you give us any suggestion please? | |
| 09:38:47 | sean-k-mooney | you proably missed somethign in the requestspec in the api db | |
| 09:40:14 | sean-k-mooney | without knowing what line/file that was raised form (preferable the full traceback) i really cant say what is missing | |
| 09:43:32 | andry_g | https://paste.openstack.org/show/bmnRZsPkgpPjdIhwWUss/ | |
| 09:44:21 | sean-k-mooney | ya so its the group_info.host that is not populated when its trying to poulate teh request spec for schduling | |
| 09:44:23 | sean-k-mooney | request_spec.instance_group.hosts = list(group_info.hosts) | |
| 09:44:32 | andry_g | the record that has bee insert is insert into instance_group_member(created_at,updated_at,id,instance_uuid,group_id) values('2022-03-31 08:53:19',NULL,'138','13052619-612f-4eea-b34d-59d3d473329f','67'); | |
| 09:48:04 | andry_g | we try to fix it, I''ll let you know. Thank you | |
| 09:49:04 | sean-k-mooney | this is the code where its failing https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L1168-L1208 | |
| 09:49:25 | sean-k-mooney | L1206 specifcally | |
| 09:50:15 | sean-k-mooney | did you update the request spec to set request_spec.instance_group | |
| 09:50:30 | sean-k-mooney | on the instance in addtion to insterting it into that db | |
| 10:04:21 | andry_g | Thank you Sean, it's working fine! | |
| 10:04:53 | sean-k-mooney | cool | |
| 10:10:15 | andry_g | sean-k-mooney: there is another method on new release to do it without modify db? | |
| 10:10:48 | sean-k-mooney | no there was a feature request to add support about a year ago but it was not implemented | |
| 10:13:40 | sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/782353 | |
| 10:14:27 | sean-k-mooney | andry_g:^ that was the most recent proposal but it did not have agreement yet on how to allow it | |
| 10:14:37 | sean-k-mooney | no one is currenlty working on it as far as i am aware | |
| 10:20:05 | sean-k-mooney | andry_g: just lookign that the comments it seams like we were convergin on extneing the move oeprations so resize, shelve, cold migrate and live migrate to allow you to migrate into a group | |
| 12:59:50 | viks__ | sean-k-mooney: i have tried to find the "upcall" in nova logs for investigating the anti-affinity, but did not find any even after enabling debug log.. not sure what i'm doing wrongly | |
| 13:04:16 | sean-k-mooney | so if the polciy is violated we shoudl raise thei exception https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L1782-L1786 | |
| 13:04:43 | sean-k-mooney | or this one if its affinity https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L1796-L1799 | |
| 13:05:24 | sean-k-mooney | you should atlest see that in the conductor log but i think you shoudl see that in the compute log too | |
| 13:06:36 | sean-k-mooney | it should be caut and logged here i think https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L2318-L2323 | |
| 13:07:06 | sean-k-mooney | well actully here https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L2333 | |
| 13:09:13 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/yoga: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836014 | |
| 13:10:07 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | |
| 13:10:18 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836016 | |
| 13:10:30 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/victoria: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836017 | |
| 13:10:42 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/ussuri: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836018 | |
| 13:14:08 | viks__ | sean-k-mooney: to get myself clarified, if i create 2 instance i.e. create 1st instance with soft-anti-affinity and then after that got created if i create one more with the same, it should go to the same node or different...? i'm not sure if this method is the proper way of testing the upcall.. | |
| 13:14:39 | viks__ | also i'm using openstack-train .. is the upcall supposed to work in this release? | |
| 13:18:08 | sean-k-mooney | am i think the upcall might just be for hard affinity looking at it | |
| 13:18:29 | sean-k-mooney | well hard affinity and anti affinity | |
| 13:18:48 | sean-k-mooney | soft-anti-affinity is just a best effort schduler weigher | |
| 13:19:07 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: Libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 13:21:07 | kashyap | stephenfin: --^ I've adressed most of your smaller comments (see my short log at the bottom). Also ... I can really make use of your laser-eyes here. I want to get this to completion for real | |
| 13:21:29 | kashyap | Today I need to be away a couple of hours early; so I won't be able to respond much today. But tomm onwards, yes | |
| 13:21:59 | viks__ | `anti-affinity` it goes in to two separate nodes if i try like above... i.e. it is working as expected... but i could not see any `upcall` string in any of the logs with this as well | |
| 13:23:47 | sean-k-mooney | it wont nessalry have up call in it | |
| 13:24:02 | viks__ | oh ok | |
| 13:24:02 | sean-k-mooney | looking at the implemeation it does not really log much | |
| 13:24:11 | viks__ | ok | |
| 13:24:16 | sean-k-mooney | there is some logging in the faliure path | |
| 13:24:21 | sean-k-mooney | for the reschdule | |
| 13:26:03 | sean-k-mooney | there is no log line when it works https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L1706-L1801 | |
| 13:26:12 | sean-k-mooney | i.e. the policy is not violated | |
| 13:26:48 | sean-k-mooney | but this code also only checks 'anti-affinity' == group.policy and 'affinity' == group.policy | |
| 13:27:10 | sean-k-mooney | it does nothing for soft-anti-affinity or soft-affinity | |
| 13:39:48 | viks__ | ok... thanks a lot | |
| 18:00:39 | opendevreview | Gorka Eguileor proposed openstack/nova master: Fix extending non LUKSv1 encrypted volumes https://review.opendev.org/c/openstack/nova/+/836064 | |
| 18:08:58 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose spec for ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/835877 | |
| 18:09:12 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose spec for ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/835877 | |
| 19:06:08 | opendevreview | melanie witt proposed openstack/nova-specs master: Re-propose spec for ephemeral encryption for libvirt https://review.opendev.org/c/openstack/nova-specs/+/836075 | |
| 20:57:29 | opendevreview | Merged openstack/nova stable/xena: [stable-only] Set nova-tox-functional-centos8-py36 non-voting https://review.opendev.org/c/openstack/nova/+/834854 | |
| 21:42:40 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/835853 | |
| 21:43:04 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/835854 | |
| 21:43:17 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/835855 | |
| 23:44:56 | opendevreview | Julia Kreger proposed openstack/nova master: Ironic - Handle instance/node host on rebalance https://review.opendev.org/c/openstack/nova/+/813897 | |
| 23:45:09 | opendevreview | Julia Kreger proposed openstack/nova master: Ironic - Don't query the API for instance counts https://review.opendev.org/c/openstack/nova/+/829613 | |
| #openstack-nova - 2022-04-01 | |||
| 00:36:15 | opendevreview | Merged openstack/nova stable/xena: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/828413 | |
| 01:52:57 | opendevreview | Merged openstack/nova stable/xena: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/828414 | |
| 04:02:07 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/836113 | |
| 08:39:26 | opendevreview | Tobias Urdin proposed openstack/nova stable/xena: Cleanup old resize instances dir before resize https://review.opendev.org/c/openstack/nova/+/828407 | |
| 09:32:49 | opendevreview | Jan Hartkopf proposed openstack/nova-specs master: update userdata https://review.opendev.org/c/openstack/nova-specs/+/816542 | |
| 11:34:20 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/836145 | |
| 11:34:21 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/836146 | |
| 11:34:22 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/836147 | |
| 13:02:26 | opendevreview | Takashi Kajinami proposed openstack/nova stable/xena: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/836145 | |
| 13:02:27 | opendevreview | Takashi Kajinami proposed openstack/nova stable/xena: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/836146 | |
| 13:02:28 | opendevreview | Takashi Kajinami proposed openstack/nova stable/xena: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/836147 | |
| 13:17:04 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/828418 | |
| 13:17:05 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/828419 | |
| 13:19:02 | artom | kashyap, yeah, different meaning: revert the commit that changed revert resize behaviour | |
| 13:26:02 | kashyap | artom: Ahh, yeah. (Damn, an example of my brain auto-completing things wrongly.) | |
| 13:27:33 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/828418 | |
| 13:27:34 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/828419 | |
| 13:43:43 | bauzas | folks, as I wrote, I started to provide a schedule for our PTG https://etherpad.opendev.org/p/nova-zed-ptg#L45 | |
| 13:44:01 | bauzas | look at it if you have concerns or meeting conflicts | |
| 13:44:35 | bauzas | I added a courtesy ping list for every topic, so add your nick if you want to be around | |
| 13:44:39 | bauzas | chateaulav: ^ | |
| 13:45:05 | bauzas | chateaulav: don't know if you will attend the whole PTG, I can understand if you only need to be around for say, one day | |
| 14:01:30 | gibi | bauzas: schedule looks good to me | |
| 14:06:43 | bauzas | thanks | |
| 14:40:38 | pslestang | /join #openstack-glance | |
| 15:12:22 | chateaulav | bauzas: I will be there | |
| 15:12:33 | bauzas | cool | |
| 15:17:44 | sean-k-mooney | bauzas: by the way the ptg i assume we will be using zoom or jitsi meet right | |
| 15:17:56 | sean-k-mooney | so we can just connect with a web client | |
| 15:18:17 | artom | sean-k-mooney, the current URLs on https://ptg.opendev.org/ptg.html are for Zoom | |
| 15:20:39 | sean-k-mooney | cool | |
| 15:20:50 | bauzas | sean-k-mooney: we will use Zoom | |
| 15:21:07 | bauzas | unless people want to use meetpad/Jitsi | |
| 15:21:24 | bauzas | but given we had 3 PTGs using Zoom, I think it's not a problem | |
| 15:22:14 | bauzas | artom: the foundation defaults all the rooms to Zoom but as a moderator, I can create a new meetpad which will use Jitsi (and then I can modify the url) | |