| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-12-03 | |||
| 01:03:08 | sean-k-mooney | not ussuri | |
| 01:03:27 | sean-k-mooney | dcapone2004: you can fix that in your current deployment via the glance api | |
| 01:03:33 | dcapone2004 | didn't pass hypervisor=no, but it did pass a whole bunch of other hv options....might essentially resolve the same issue...will know in about 6 minutes when hyper-v finishes installing...I greatly appreciate your help | |
| 01:03:36 | sean-k-mooney | you can update the metadata defs via the api | |
| 01:03:56 | dcapone2004 | yeah i can just update our internal MOPs | |
| 01:04:28 | sean-k-mooney | dcapone2004: yep os_type is used to enable the hyperv enlightenments too which enable the hyperv optimisations | |
| 01:04:38 | sean-k-mooney | it should improve you guest performance | |
| 01:05:30 | dcapone2004 | admittedly I don't overly care about nested VM performance....we basically use nested VM mostly to lab out potential deployments for customers | |
| 01:05:40 | dcapone2004 | but good to know anyway | |
| 01:06:11 | sean-k-mooney | dcapone2004: well it will impre the first level vm performance too and it shoudl reduce power use when the vms are ideling | |
| 01:07:58 | dcapone2004 | ok, maybe slight miscommunication.....these are being deployed on KVM hosts....not Hyper-V hosts....don't know if that makes a different in the configuration....I point that out because I cannot understand how exposes hyper-v features to the VM would aid performance when running on KVM | |
| 01:08:31 | sean-k-mooney | that is what i was expecting hyperv on kvm | |
| 01:09:13 | sean-k-mooney | window has special kernel code for runing on hyperv and kvm support some of the hyperv feature | |
| 01:09:41 | sean-k-mooney | https://libvirt.org/formatdomain.html#hypervisor-features has a table ot the hyperv feature | |
| 01:09:42 | dcapone2004 | i see | |
| 01:10:03 | sean-k-mooney | although we dont support enabling most of them basically because no one has asked or done it | |
| 01:11:49 | dcapone2004 | unfortunately the bad news is that didn't seem to resolve this specific situation.... Im about to try this out on a non openstack standalone kvm system to see if hypervisor=off does anything for it | |
| 01:12:18 | sean-k-mooney | so you could manally edit the xml and add that | |
| 01:12:21 | dcapone2004 | but sinc eyou mentioned performance improvements....is there anyway to add those flags to already existing, deployed/modified VMs? | |
| 01:12:27 | sean-k-mooney | then use libvirt to restart the vm | |
| 01:12:32 | sean-k-mooney | not openstack | |
| 01:12:53 | sean-k-mooney | dcapone2004: no not without a db update | |
| 01:13:03 | dcapone2004 | might try that as well first | |
| 01:13:21 | dcapone2004 | that is what I thought...db update and hard reboot? | |
| 01:13:32 | sean-k-mooney | if you were willing to do that then all you need to do is add img_os_type windows to each vm in the instance_system_metadata table | |
| 01:13:50 | sean-k-mooney | dcapone2004: yes db update and hard reboot | |
| 01:14:46 | dcapone2004 | alright, you have been very helpful, I am going to play around with a few things and see what happens, if the hypervisor=no option works, I might be asking for a feature request :-) | |
| 01:15:21 | sean-k-mooney | if you confim that adding the elemnt fixes the issue then we can simple add it here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5386 or here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5393 | |
| 01:15:23 | sean-k-mooney | as a bug fix | |
| 01:15:35 | sean-k-mooney | depending on if its jut for hyperv or for hyperv and kvm | |
| 01:15:56 | sean-k-mooney | sorry windows or windows and linux | |
| 01:16:17 | dcapone2004 | I think this is KVM specific based on the articles about the problem and how to fix it....I unfortunately don't have any openstack hyper-v nodes to test for sure | |
| 01:16:18 | sean-k-mooney | kvm/hyperv does not matter in this case | |
| 01:16:35 | dcapone2004 | linux nested virtualization works without issue | |
| 01:16:39 | sean-k-mooney | yep | |
| 01:16:47 | dcapone2004 | I've used it on this same cluster and with the same flavor without issue | |
| 01:16:49 | sean-k-mooney | i use it every day | |
| 01:17:08 | sean-k-mooney | i do all my openstack dev on my home openstack cluster | |
| 01:17:14 | sean-k-mooney | with nested virt and hypervior hiding | |
| 01:19:54 | dcapone2004 | 1 more question...to potentially save me some time searching....do you know where the xml files for VMs is in a kolla containerized deployment? | |
| 01:20:13 | sean-k-mooney | oh you dont edit the xmls | |
| 01:20:19 | sean-k-mooney | i use kolla | |
| 01:20:30 | sean-k-mooney | you need to docker exec into nova_libvirt | |
| 01:20:43 | sean-k-mooney | e.g. docker exec -it nova_libvirt bash | |
| 01:20:54 | sean-k-mooney | then do virsh edit <vm id> | |
| 01:21:33 | sean-k-mooney | if you look at the extended atibute in the server show as an admin you can get teh libvirt domain name | |
| 01:21:49 | sean-k-mooney | otherwise you just need to find the one with the uuid that matches | |
| 01:23:49 | dcapone2004 | ok, I thought you said, "so you could manally edit the xml and add that" "then use libvirt to restart the vm | |
| 01:24:06 | sean-k-mooney | ah yes when you do virsh edit | |
| 01:24:21 | sean-k-mooney | it will open the xml in an editor for you | |
| 01:24:24 | sean-k-mooney | that is what i ment | |
| 01:24:39 | dcapone2004 | got it | |
| 01:24:41 | sean-k-mooney | it will also validate it with libvirt and only allow it to save if the edit is valid | |
| 01:25:22 | sean-k-mooney | anywyay its 01:25 here so im heading to bed | |
| 01:25:45 | sean-k-mooney | if that works defintly file the bug and we can see if we can fix this | |
| 01:26:10 | sean-k-mooney | the nova meeting is tomorrow too if you wanted to bring it up in open dicusstion | |
| 01:26:42 | sean-k-mooney | if you can confirm either way if setting that option fixes your issue let me know an i can bring it up if you cant attend | |
| 01:27:30 | dcapone2004 | will do | |
| 01:27:42 | dcapone2004 | just ran virsh reboot 35, so we shall see | |
| 01:28:20 | sean-k-mooney | you might need to stop the domin and start it by the way | |
| 01:28:31 | sean-k-mooney | i dont rembere if reboot does a soft reboot or hard | |
| 01:28:57 | dcapone2004 | was going to iterate through all possible methods before quitting :-) .... get some rest...your assistant has been much appreciated | |
| 01:28:57 | sean-k-mooney | soft being a qemu monitor command to tell the os to reboot without restarting the qemu process | |
| 01:29:17 | sean-k-mooney | o/ | |
| 03:17:46 | openstackgerrit | chengsheng proposed openstack/nova master: CPU selection with hypervisor consideration https://review.opendev.org/c/openstack/nova/+/762330 | |
| 05:51:48 | openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | |
| 06:32:57 | brinzhang_ | gibi, dansmith: updated cyborg shelve/unshelve support patch, please review it while you are free https://review.opendev.org/c/openstack/nova/+/729563, thanks | |
| 07:05:11 | openstackgerrit | Xinran WANG proposed openstack/nova-specs master: Smart-nic Support Specification https://review.opendev.org/c/openstack/nova-specs/+/742785 | |
| 08:07:51 | openstackgerrit | Wenping Song proposed openstack/nova master: Nova supports password encrypted VNC https://review.opendev.org/c/openstack/nova/+/622336 | |
| 08:38:19 | bauzas | good morning Nova | |
| 08:47:54 | gibi | good morning bauzas | |
| 08:49:07 | bauzas | :-) | |
| 08:59:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Separate out the migration tests https://review.opendev.org/c/openstack/nova/+/765300 | |
| 09:10:47 | elod | good morning :) | |
| 09:11:11 | elod | bauzas: if you will have time can you have another look at this patch? https://review.opendev.org/c/openstack/nova/+/764432 | |
| 09:11:33 | elod | this would unblock the gate on stable/victoria | |
| 09:11:43 | bauzas | elod: for sure, I just want to progress on the routed networks impl, but I can do other urgent stuff | |
| 09:11:56 | bauzas | a-ha, good reason then | |
| 09:12:09 | bauzas | elod: and fwiw, lyarwood is on PTO those 2 days | |
| 09:12:24 | bauzas | damn, those RH folks who are always on vacation... | |
| 09:12:47 | elod | this is lyarwood's commit so it does not really matter ;) | |
| 09:13:46 | elod | actually, only a commit message change happened since your last +1 (and a successful zuul run \o/) | |
| 09:46:49 | elod | bauzas: thx! \o/ (and sorry for the interrupt o:)) | |
| 09:47:21 | bauzas | np | |
| 10:24:23 | openstackgerrit | Lucian Petrut proposed openstack/nova master: hyper-v rbd volume support https://review.opendev.org/c/openstack/nova/+/763550 | |
| 11:05:37 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace all_tenants with all_projects https://review.opendev.org/c/openstack/nova/+/765311 | |
| 11:27:21 | chengsheng | kashyap: I have updated: https://review.opendev.org/c/openstack/nova/+/762330/ | |
| 11:28:35 | kashyap | chengsheng: Hi; will look. (Side note: I'm juggling various things this and next week, as I'll be off from 14-Dec until 31-Dec) | |
| 11:29:10 | gibi | kashyap: thanks for the heads up, have a nice time off! | |
| 11:29:25 | kashyap | gibi: Thank you; lots of non-computer reading :-) | |
| 11:30:57 | gibi | nice! | |
| 11:47:12 | brinzhang_ | gmann: There is an small issue while I want the replace the list servers's policies, I am not find a good way to decrepate the old policy (https://github.com/openstack/oslo.policy/blob/master/oslo_policy/policy.py#L1299), I have submited a base patch, hope you can give some suggestions | |
| 11:48:40 | openstackgerrit | Brin Zhang proposed openstack/nova master: WIP: Replace tenants* with projects* in List Servers's policy https://review.opendev.org/c/openstack/nova/+/765315 | |
| 11:48:48 | brinzhang_ | gmann: this is the replace role name patch https://review.opendev.org/c/openstack/nova/+/765315 | |
| 11:48:57 | brinzhang_ | gmann: thanks | |
| 11:52:48 | brinzhang_ | gibi, stephenfin, sean-k-mooney, gmann: the blueprint remove-tenant-id need a microversion, and there are so many nova API need to be changed, so I would like to get a microversion to avoid many conflict, do you have time to review the first patch that adding a microversion 2.88 for this change. links:https://review.opendev.org/c/openstack/nova/+/764292 | |
| 12:00:40 | sean-k-mooney | brinzhang_: im not sure we have many patches tat are changing the api at present, i i would hope there wont be many conflicts | |
| 12:01:17 | sean-k-mooney | but yes ill try to review later today | |
| 12:02:24 | brinzhang_ | sean-k-mooney: thanks, but from the list nova apis in the spec, there are at least 3 APIs need a microversion, I thinks https://specs.openstack.org/openstack/nova-specs/specs/wallaby/approved/remove-tenant-id.html | |
| 12:03:16 | brinzhang_ | whatever, there is a microversion to be bound, it's easy to do ^^ | |