| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-11-13 | |||
| 16:23:25 | dansmith | gibi: those *should* be places where there isn't a specific rpc version in play, | |
| 16:23:36 | dansmith | in which case the service version is specifically what we want | |
| 16:23:50 | gibi | dansmith: yeah, that would be the proper usage of the service version only checks | |
| 16:23:55 | dansmith | service version alone should be used for cases where "I'm not saying anything new over rpc, but I depend on some action being done on the compute" | |
| 16:24:06 | gibi | dansmith: I will look through these checks to be sure | |
| 16:27:27 | tacco | sean-k-mooney: i guess this should be debian-cloud-image most things kernel related should be default | |
| 16:28:34 | tacco | ok flavor has now 248vcpus and | properties | hw:cpu_sockets='2', hw:cpu_theads='2' | | |
| 16:29:32 | sean-k-mooney | tacco: ya im not sure what the default sockets is upstream but 64 sound like a number peopel would choose as a default | |
| 16:29:49 | tacco | sean-k-mooney: yes this was also in my mind as first :D | |
| 16:30:08 | tacco | thats why i asked here.. because if this is known.. you should know it. :) | |
| 16:30:21 | tacco | this is the first time i have a HV with so many CPUs | |
| 16:30:43 | tacco | and i know some people here should have way larger setups and way more experience than i have. :) | |
| 16:31:07 | sean-k-mooney | i havent gone over 128 but i alway make my flavor mirror the host toplogy in terms or threads and sockets | |
| 16:31:18 | tacco | anyway. Thanks for your initial help. Know in know this could be related to the image. Will digg aroung and see what i can find. | |
| 16:31:29 | gibi | dansmith: does an ovo always travels through RPC with all its data and only backlevelled on the receiving side? So if a new field is added to an o.vo that is sent via RPC then we don't need to bump RPC version and the reciving side gets the new field if the code on the reciving side has the new field definition in its own ovo class independently of the RPC api version? | |
| 16:31:37 | sean-k-mooney | tacco: did that work by the way | |
| 16:31:41 | sean-k-mooney | the updated flavor | |
| 16:31:43 | tacco | ususaly you also don't want such huge flavors. | |
| 16:31:53 | tacco | nope updated flavor also has only 64cpus in the cm | |
| 16:31:57 | tacco | s/cm/vm/ | |
| 16:32:07 | sean-k-mooney | really | |
| 16:32:12 | sean-k-mooney | that is odd | |
| 16:32:31 | sean-k-mooney | am can you quickly check the qemu string jut to triple check | |
| 16:32:33 | tacco | and i double checked the xml if that change affected the xml and reflects my change to the flavor | |
| 16:32:47 | dansmith | gibi: we always send the version of the object we have. If the receiving side determines it's too new, it calls to conductor and asks for conductor to backlevel it. Conductor can either return an object that has an older version (i.e. if it _can_ backlevel it) or it can refuse | |
| 16:32:52 | sean-k-mooney | it likely a qemu or guest kernel limiation however | |
| 16:33:13 | tacco | yes. Thanks thats where i would like to digg more deeper | |
| 16:33:35 | gibi | dansmith: good to know, that in this case there is an extra call back to the conductor | |
| 16:34:15 | dansmith | gibi: but that's just for the object(s), not the rpc signature itself, and the idea was that during an upgrade you have extra conductor load to handle all the backports, but as you upgrade everything that just disappears | |
| 16:35:50 | gibi | dansmith: yes, it is clear that this is only possible for o.vos itself, not for the whole RPC method signature. | |
| 16:36:05 | gibi | dansmith: but then in a new-field-in-an-ovo case the service version check is enough | |
| 16:36:29 | gibi | as if the compute service version is new enough then it will understand a new ovo version with the extra field | |
| 16:36:30 | sean-k-mooney | tacco: https://github.com/torvalds/linux/blob/master/arch/x86/Kconfig#L994-L1005 | |
| 16:36:53 | dansmith | yeah, and in some cases, it's possible to backlevel the object so we can just deal with it on the receiving end, but not if you require specific behavior | |
| 16:37:15 | sean-k-mooney | tacco: so it should be 512 so proably a qemu issue | |
| 16:38:41 | tacco | i see. Thanks. | |
| 16:39:08 | gibi | dansmith: thanks again, this make sense now | |
| 16:39:41 | dansmith | cool | |
| 16:40:14 | sean-k-mooney | tacco: it look like the max cpus depends on the machine type you enable | |
| 16:41:01 | gibi | I think this is a good time to finish my week and let the new understanding solidifies :) | |
| 16:41:14 | gibi | have a nice weekend folks o/ | |
| 16:41:24 | lyarwood | \o | |
| 16:44:11 | tacco | sean-k-mooney: here is also the capa list of virsh http://paste.openstack.org/show/nxpSZUUrICfvvRJBgosL/ | |
| 16:45:20 | tacco | this machine type? <type arch='x86_64' machine='pc-i440fx-4.0'>hvm</type> | |
| 16:45:47 | sean-k-mooney | ya you are using the pc machine type but it should in theory support up to 256 | |
| 16:47:00 | sean-k-mooney | tacco: if you look in teh output it has the limits | |
| 16:47:11 | sean-k-mooney | line 1193 | |
| 16:47:18 | sean-k-mooney | <machine maxCpus='255'>pc-i440fx-4.0</machine> | |
| 16:47:28 | tacco | yes i also found this in there. | |
| 16:47:45 | tacco | ok. so no "real"limitations more of a missconfiguration or bug. :) | |
| 16:48:01 | sean-k-mooney | so this is looking like a guest issue or a bug ya | |
| 16:48:13 | sean-k-mooney | you could try with another image | |
| 16:48:26 | tacco | i already struggled about that limitations that you can only have 8 disks inside a qemu vm. :) | |
| 16:48:48 | tacco | yes will do so but in general this was the debian cloud image with only minimal changes. but will test next week. | |
| 16:48:49 | sean-k-mooney | like a fedora or tubleweed image | |
| 16:49:01 | tacco | ok. will do so. but for today im done. | |
| 16:49:09 | sean-k-mooney | cool | |
| 16:49:13 | sean-k-mooney | enjoy your weekend | |
| 16:49:31 | tacco | thanks a lot for your help so far. Will get back to you next week. have a nice weekend as well | |
| 17:04:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Restore retrying the RPC connection to conductor https://review.opendev.org/762633 | |
| 17:17:54 | dopereira | Hi, everyone. I'm a new contributor and just joined the OpenStack community. | |
| 17:18:08 | dopereira | I recently started to work on a project that develops a Openstack distribuition with comercial support. Hopefully I will be contributing to the upstream Openstack as well. | |
| 17:18:19 | dopereira | I recently started to work on a project that develops a Openstack distribuition with comercial support. Hopefully I will be contributing to the upstream Openstack as well. | |
| 17:18:40 | dopereira | As part of my ramp up process, I'm learning how to contribute with Openstack, and I was asked to take care of a low-hanging-fruit bug. | |
| 17:18:58 | openstack | Launchpad bug 1888927 in OpenStack Compute (nova) "cell_v2 update_cell cell0 get transport_url from config file" [Low,In progress] - Assigned to Daniel de Oliveira Pereira (danielpereira01) | |
| 17:18:58 | dopereira | I choose this one: https://bugs.launchpad.net/nova/+bug/1888927 and already have a patch for it: https://review.opendev.org/#/c/762433/4 | |
| 17:19:10 | dopereira | Could you guys please take a look and help to review it? | |
| 17:21:51 | dopereira | Also, the VMware NSX CI check failed. How can I started recheck for it? It does not provide this information | |
| 17:23:18 | sean-k-mooney | dopereira: have you submitted the patch to gerrit | |
| 17:23:29 | sean-k-mooney | ah you have | |
| 17:24:23 | sean-k-mooney | dopereira: when a third party ci fails it leaves a comment telling you how to recheck that specific ci | |
| 17:24:47 | sean-k-mooney | dopereira: you have to click the toggle extra ci button to see it | |
| 17:24:59 | sean-k-mooney | oh but they dont | |
| 17:25:24 | sean-k-mooney | it should be listed here | |
| 17:25:25 | dopereira | it seems that's not the case for VMware NSX CI | |
| 17:25:26 | sean-k-mooney | https://wiki.openstack.org/w/index.php?title=ThirdPartySystems | |
| 17:25:47 | sean-k-mooney | vmware-recheck-patch | |
| 17:26:07 | sean-k-mooney | acorrding to https://wiki.openstack.org/wiki/ThirdPartySystems/VMware_CI | |
| 17:27:01 | dopereira | I saw, thanks | |
| 17:27:42 | sean-k-mooney | so i havent done a full review but i think you need to add a release note for the bug fix | |
| 17:28:16 | sean-k-mooney | looks like you have added tests | |
| 17:29:34 | sean-k-mooney | so ya the main thing i think is needed is a realse note but im not that familar with that part fo the code otherwise it looks fine | |
| 17:30:27 | dopereira | could you point me some documentation about release notes? | |
| 17:31:35 | gmann | dopereira: here - https://docs.openstack.org/reno/latest/user/usage.html | |
| 17:31:53 | sean-k-mooney | basically you create a new one form the template via tox. so tox -e venv -- reno new cell_0-transport-url | |
| 17:32:00 | sean-k-mooney | then you edit the file it creates | |
| 17:32:14 | sean-k-mooney | you can check its correct with tox -e releasenotes | |
| 17:32:38 | sean-k-mooney | you just need to fill out the fixes section in this case and remove the rest | |
| 17:34:26 | dopereira | thanks, will take a look | |
| 17:54:16 | tacco | sean-k-mooney: can't hold myself and tested with fedora. same same.. 64vcpus in /proc/cpuinfo.. d | |
| 17:54:42 | tacco | nproc also 64. | |
| 17:59:40 | sean-k-mooney | tacco: you try a dmidecode in the vm | |
| 17:59:55 | sean-k-mooney | or check dmsge to see if it prints anything on kernel start | |
| 18:00:15 | sean-k-mooney | but ya nova seams to be doing the right thing so its a qemu or guest issue | |
| 18:00:30 | sean-k-mooney | looking more like qemu since it happens on multiple distoros | |
| 18:12:01 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: [WIP] Migrate nova-grenade-multinode job to zuulv3 native https://review.opendev.org/742056 | |
| 19:05:44 | openstackgerrit | Daniel de Oliveira Pereira proposed openstack/nova master: Avoid changing transport_url when updating Cell0 https://review.opendev.org/762433 | |
| 20:06:14 | openstackgerrit | Merged openstack/nova stable/train: add [libvirt]/max_queues config option https://review.opendev.org/740064 | |
| 20:52:04 | openstackgerrit | Merged openstack/nova stable/victoria: Handle disabled CPU features to fix live migration failures https://review.opendev.org/758760 | |
| 21:27:47 | openstackgerrit | Merged openstack/nova stable/ussuri: docs: Resolve issue with deprecated extra specs https://review.opendev.org/748386 | |