| opendevreview | melanie witt proposed openstack/nova-specs master: Make 'tox -e fast-specs' fail on sphinx warnings https://review.opendev.org/c/openstack/nova-specs/+/835930 | 04:53 |
|---|---|---|
| *** kashyap` is now known as kashyap | 07:47 | |
| kashyap | stephenfin: Thanks for the review here! - https://review.opendev.org/c/openstack/nova/+/762330/. Will address | 07:49 |
| noonedeadpunk | sean-k-mooney: well... if openstackclient tended to use highest version available instead of lowest version supported... | 08:05 |
| amoralej | hi kashyap, we need to discuss wrt https://review.rdoproject.org/r/c/openstack/nova-distgit/+/38029 it doesn't work with aarch64 as it seems there is no virtio-vga for it? | 08:56 |
| kashyap | amoralej: Hi | 08:57 |
| * kashyap clicks | 08:57 | |
| amoralej | kashyap, see https://gitlab.com/redhat/centos-stream/rpms/qemu-kvm/-/blob/c9s/qemu-kvm.spec#L582 | 08:58 |
| kashyap | amoralej: Hmm, I don't top off my head, lemme check w/ AArch64 folks. | 08:59 |
| * kashyap clicks the QEMU spec too | 08:59 | |
| amoralej | it seems it's only for x86_64 and power64 | 08:59 |
| kashyap | amoralej: Isn't that line you linked to for 'power64'? | 08:59 |
| kashyap | Hmm, yea; did you run into failures on AArch64 hosts? | 09:00 |
| kashyap | amoralej: Maybe we can wrap the "Requires(pre):" in Nova under "%ifarch x86_64 %{power64}" | 09:01 |
| amoralej | yes, but nova is noarch | 09:02 |
| amoralej | and iirc %ifarch is only applied at build time, not at install time | 09:02 |
| amoralej | but i need to check | 09:03 |
| kashyap | amoralej: I have joined #rdo on Libera net, if you want to sort it out there. :) | 09:08 |
| amoralej | kashyap, better #rdo in oftc, please :) | 09:09 |
| kashyap | Ah, I didn't know it's on OFTC, joining | 09:09 |
| 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 |
| 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:35 |
| sean-k-mooney | you proably missed somethign in the requestspec in the api db | 09:38 |
| sean-k-mooney | without knowing what line/file that was raised form (preferable the full traceback) i really cant say what is missing | 09:40 |
| andry_g | https://paste.openstack.org/show/bmnRZsPkgpPjdIhwWUss/ | 09:43 |
| 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 |
| sean-k-mooney | request_spec.instance_group.hosts = list(group_info.hosts) | 09:44 |
| 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:44 |
| andry_g | we try to fix it, I''ll let you know. Thank you | 09:48 |
| 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 |
| sean-k-mooney | L1206 specifcally | 09:49 |
| sean-k-mooney | did you update the request spec to set request_spec.instance_group | 09:50 |
| sean-k-mooney | on the instance in addtion to insterting it into that db | 09:50 |
| andry_g | Thank you Sean, it's working fine! | 10:04 |
| sean-k-mooney | cool | 10:04 |
| andry_g | sean-k-mooney: there is another method on new release to do it without modify db? | 10:10 |
| sean-k-mooney | no there was a feature request to add support about a year ago but it was not implemented | 10:10 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/782353 | 10:13 |
| 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 |
| sean-k-mooney | no one is currenlty working on it as far as i am aware | 10:14 |
| 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 | 10:20 |
| *** bhagyashris_ is now known as bhagyashris | 11:09 | |
| *** amoralej is now known as amoralej|lunch | 11:54 | |
| 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 | 12:59 |
| 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 |
| sean-k-mooney | or this one if its affinity https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L1796-L1799 | 13:04 |
| 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:05 |
| *** amoralej|lunch is now known as amoralej | 13:06 | |
| 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:06 |
| sean-k-mooney | well actully here https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L2333 | 13:07 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/yoga: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836014 | 13:09 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | 13:10 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836016 | 13:10 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/victoria: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836017 | 13:10 |
| opendevreview | Erlon R. Cruz proposed openstack/nova stable/ussuri: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836018 | 13:10 |
| 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 |
| viks__ | also i'm using openstack-train .. is the upcall supposed to work in this release? | 13:14 |
| sean-k-mooney | am i think the upcall might just be for hard affinity looking at it | 13:18 |
| sean-k-mooney | well hard affinity and anti affinity | 13:18 |
| sean-k-mooney | soft-anti-affinity is just a best effort schduler weigher | 13:18 |
| opendevreview | Kashyap Chamarthy proposed openstack/nova master: Libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | 13:19 |
| 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 |
| 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 |
| 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:21 |
| sean-k-mooney | it wont nessalry have up call in it | 13:23 |
| viks__ | oh ok | 13:24 |
| sean-k-mooney | looking at the implemeation it does not really log much | 13:24 |
| viks__ | ok | 13:24 |
| sean-k-mooney | there is some logging in the faliure path | 13:24 |
| sean-k-mooney | for the reschdule | 13:24 |
| 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 |
| sean-k-mooney | i.e. the policy is not violated | 13:26 |
| sean-k-mooney | but this code also only checks 'anti-affinity' == group.policy and 'affinity' == group.policy | 13:26 |
| sean-k-mooney | it does nothing for soft-anti-affinity or soft-affinity | 13:27 |
| viks__ | ok... thanks a lot | 13:39 |
| *** dasm|off is now known as dasm | 14:00 | |
| *** dasm is now known as dasm|ruck | 16:38 | |
| *** amoralej is now known as amoralej|off | 17:15 | |
| opendevreview | Gorka Eguileor proposed openstack/nova master: Fix extending non LUKSv1 encrypted volumes https://review.opendev.org/c/openstack/nova/+/836064 | 18:00 |
| 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:08 |
| 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 |
| 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 | 19:06 |
| opendevreview | Merged openstack/nova stable/xena: [stable-only] Set nova-tox-functional-centos8-py36 non-voting https://review.opendev.org/c/openstack/nova/+/834854 | 20:57 |
| opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Test aborting queued live migration https://review.opendev.org/c/openstack/nova/+/835853 | 21:42 |
| 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 |
| opendevreview | Alexey Stupnikov proposed openstack/nova stable/xena: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/835855 | 21:43 |
| *** dasm|ruck is now known as dasm|off | 22:59 | |
| opendevreview | Julia Kreger proposed openstack/nova master: Ironic - Handle instance/node host on rebalance https://review.opendev.org/c/openstack/nova/+/813897 | 23:44 |
| opendevreview | Julia Kreger proposed openstack/nova master: Ironic - Don't query the API for instance counts https://review.opendev.org/c/openstack/nova/+/829613 | 23:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!