| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-12 | |||
| 14:58:46 | dansmith | sean-k-mooney: well, or we have to redesign the live migration flow to go through (super)conductor in a more push-pull sort of way | |
| 15:00:05 | sean-k-mooney | well i ment makign it so qemu on one host could talke to qemu on another via proxign the coneection. | |
| 15:01:23 | dansmith | oh, I dunno, I think you'd just make it a requirement that they can talk direct, | |
| 15:01:29 | dansmith | even if for a short time | |
| 15:01:55 | dansmith | implementing a well-performing proxy at the superconductor (which is likely just a container on a host) seems like more work than necessary | |
| 15:02:24 | sean-k-mooney | ya it does | |
| 15:02:35 | belmoreira | those are not limitations for me... | |
| 15:02:49 | sean-k-mooney | belmoreira: yep the were just limiation in general | |
| 15:02:59 | sean-k-mooney | e.g. if you map cells to edge sites | |
| 15:03:12 | sean-k-mooney | cross cell migration get a lot harder if its over the wan | |
| 15:04:38 | belmoreira | it looks like it's a lot of work... because its a live migration the instance entries need to be created in the dest cell DB, domain XML needs to be created, take care of the network logic, and possible ceph volumes attached... qemu starts the migration and it needs to be removed from the source DB, placement updated | |
| 15:04:55 | sean-k-mooney | i dont link config driven api behavior but i kind of feel like we woudl need a way to enabel/disabel this perhapes via the sschuler config in soem way | |
| 15:05:33 | sean-k-mooney | belmoreira: we would have to also update the instnace cell mapping in the api db | |
| 15:05:45 | belmoreira | true | |
| 15:05:50 | sean-k-mooney | which is parly why this need to be driven by the super conductor | |
| 15:05:57 | dansmith | sean-k-mooney: we disabled it via policy now | |
| 15:06:09 | sean-k-mooney | dansmith: cross cell live migration | |
| 15:06:29 | sean-k-mooney | how does the policy rule know its corsscell | |
| 15:07:06 | sean-k-mooney | are we actully doign a lookup of the host if we force the destionaion? | |
| 15:07:39 | belmoreira | Cross cell resize was added in Ussuri (i was searching for migration, that's why I missed it) | |
| 15:08:18 | dansmith | sean-k-mooney: we have a flag to the api that says "can cross cell boundaries" I think, and only allow that if policy passes | |
| 15:09:00 | sean-k-mooney | belmoreira: ah ya same code path internelly | |
| 15:12:13 | belmoreira | great... looks like I will be the oldest nova-network user around :) | |
| 15:12:38 | belmoreira | dansmith sean-k-mooney thanks a lot for the comments and insights | |
| 15:15:01 | dansmith | sean-k-mooney: well, I thought we expressed intent to the api that we wanted to cross, but apparently it's only the config flag, but yeah we check the policy deeper in compute/api to decide if the user is allowed | |
| 15:18:19 | sean-k-mooney | ya that would work. i dont think we modify the request body to request cross cell either, so ya it woudl have to be checked later in the compute api | |
| 15:35:18 | giblet | sean-k-mooney: Am I correct that nova uses the physnet of the neutron network in two cases 1) matching it with the physical_network property of the PciDevice (coming from the whitelist) during scheduling if there is InstancePciRequest 2) if numa aware vswitches are configured then matching against novas configuration | |
| 15:35:23 | giblet | https://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/numa-aware-vswitches.html#id33 ? | |
| 15:36:26 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 15:48:33 | elodilles | bauwser: sorry, I forgot to ping you earlier about the ussuri release patch (as today is the Ussuri Transition Day o:)) if you could approve this that would be great :) https://review.opendev.org/c/openstack/releases/+/817226 | |
| 15:50:12 | bauwser | elodilles: done :) | |
| 15:52:26 | elodilles | bauwser: cool, thanks! \o/ I'll update the nova ussuri-em patch after this has merged | |
| 15:52:38 | bauwser | ;) | |
| 16:50:06 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 16:52:49 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 16:55:02 | elodilles | bauwser if you are still here: ussuri relese patch has merged and the ussuri-em patch is updated: https://review.opendev.org/c/openstack/releases/+/817606 | |
| 16:55:21 | bauwser | elodilles: last call, indeed. | |
| 16:56:51 | elodilles | bauwser: uh, didn't want to leave it to end of the day, but somehow forgot to ping you earlier :S sorry :/ | |
| 16:57:05 | bauwser | np :) | |
| 17:08:15 | artom | dansmith, hey, if we do an instance.save() for example, but no fields have changed, we don't actually do the DB update, correct? | |
| 17:16:04 | bauwser | elodilles: I could be wrong but I said 'no' to https://review.opendev.org/c/openstack/releases/+/817606 | |
| 17:16:57 | dansmith | artom: https://github.com/openstack/nova/blob/171138146a648d22474b7021ac730e26f03455f8/nova/objects/instance.py#L810-L811 | |
| 17:17:38 | artom | dansmith, oh, it's only in instance, not provided by base ovo? | |
| 17:17:53 | dansmith | artom: not all objects even have a save | |
| 17:18:03 | artom | Doh, right | |
| 17:18:14 | dansmith | artom: base ovo provides dirty tracking for the fields, but what you do with it is up to you | |
| 17:20:07 | artom | dansmith, yeah, makes sense | |
| 17:58:38 | sean-k-mooney | giblet: yes those are the two cases it uses it | |
| 18:01:02 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 18:12:44 | elodilles | bauwser: sorry, yes, so those patches are not relevant from release perspective :) | |
| 18:14:06 | elodilles | bauwser: why would we force customers to upgrade those packages on production systems because of some TOX and zuul and gerrit related changes? o:) | |
| 18:15:06 | sean-k-mooney[m] | which packages | |
| 18:15:23 | elodilles | osc-placement and python-novaclient | |
| 18:15:48 | elodilles | sean-k-mooney[m]: see this patch: https://review.opendev.org/c/openstack/releases/+/817606 | |
| 18:18:49 | sean-k-mooney[m] | the oscp placment tox changes wont alter what is installed | |
| 18:19:16 | sean-k-mooney[m] | it just uses the ussuri upper-constratings instead of master | |
| 18:22:26 | elodilles | yepp, exactly :) | |
| 18:23:08 | sean-k-mooney[m] | elodilles i would personally update the shas to the latest if we are doing a release anyway but i dont see any issues that would need a release to be packaged | |
| 18:23:48 | sean-k-mooney[m] | there is a fix for one func test in novaclient | |
| 18:23:56 | sean-k-mooney[m] | but again wont affect end users | |
| 18:24:46 | elodilles | sean-k-mooney[m]: the *-em tag is not a "release" it is a tag against the latest release. and releases should be meaningful for end users as far as i know | |
| 20:25:35 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 20:26:39 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 21:53:52 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Test token expiration during live migration https://review.opendev.org/c/openstack/nova/+/817778 | |
| 23:00:24 | opendevreview | David Hill proposed openstack/nova stable/ussuri: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/817689 | |
| 23:00:38 | opendevreview | David Hill proposed openstack/nova stable/train: Ensure MAC addresses characters are in the same case https://review.opendev.org/c/openstack/nova/+/817830 | |
| #openstack-nova - 2021-11-13 | |||
| 03:52:18 | opendevreview | melanie witt proposed openstack/nova master: Replace eventlet.spawn_n() with eventlet.spawn() https://review.opendev.org/c/openstack/nova/+/813114 | |
| 17:38:38 | EugenMayer | on an instance with multiple interfaces, can i define that one should not push a gateway or just define which one is the default gateway? | |
| 18:09:37 | opendevreview | melanie witt proposed openstack/nova master: Replace eventlet.spawn_n() with eventlet.spawn() https://review.opendev.org/c/openstack/nova/+/813114 | |
| #openstack-nova - 2021-11-15 | |||
| 02:41:04 | opendevreview | Jun Chen proposed openstack/nova master: Catch an exception in power off procedure https://review.opendev.org/c/openstack/nova/+/817176 | |
| 08:31:48 | bauzas | good morning Nova | |
| 08:57:50 | elodilles | good morning bauzas o/ i've replied to your comment here: https://review.opendev.org/c/openstack/releases/+/817606 | |
| 08:58:06 | elodilles | so if you'll have time... o:) | |
| 09:00:17 | giblet | morning | |
| 09:14:38 | bauzas | elodilles: ack, saw your ping on Friday | |
| 09:16:39 | bauzas | elodilles: mmm, look at the last comment I just provided in https://review.opendev.org/c/openstack/releases/+/817606 | |
| 10:25:38 | opendevreview | Brin Zhang proposed openstack/nova master: WIP: Cyborg suspend/resume support https://review.opendev.org/c/openstack/nova/+/729945 | |
| 10:29:36 | kashyap | sean-k-mooney[m]: giblet: Hi, when you're about: can we move this forward? - https://review.opendev.org/c/openstack/nova/+/798680 (libvirt: Switch the default video model from 'cirrus' to 'virtio') | |
| 10:30:13 | songwenping_ | bauzas: i install nvidia-smi driver(470.82.00) for my vgpu ubuntu vm(20.04) and run nvidia-smi, but get the error NVIDIA-SMI has failed because it couldn't communicate with the NVIDIA driver. from the dmesg |grep -i nvidia, i found the error: probe of 0000:00:06.0 failed with error -1. | |
| 10:30:59 | songwenping_ | hava you met before and is there some solutions? | |
| 10:31:13 | kashyap | sean-k-mooney[m]: I have answered all the questions there. Please re-adjust your -1. I don't see any technical obstacles here | |
| 10:37:08 | gibi | kashyap: sure, I will re-read the patch | |
| 10:37:21 | gibi | kashyap: but you should get the bp approved https://blueprints.launchpad.net/nova/+spec/virtio-as-default-display-device | |
| 10:37:25 | kashyap | Thank you | |
| 10:37:27 | kashyap | Ohh, right | |
| 10:38:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 10:38:07 | kashyap | gibi: It's a tiny patch; also see my summary of testing from 04 Aug. | |
| 10:38:46 | gibi | bauzas: ^^ | |
| 10:39:18 | kashyap | (I've got some real live migration tests done by Red Hat QE folks to have Windows + Linux guests - and it all works as expected.) | |
| 10:40:14 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 10:42:02 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 10:45:12 | kashyap | gibi: Unrelated: Oddly, this simple class addition is failing only in the CI, not locally here. You see what I'm missing here? - https://paste.opendev.org/show/810997/ | |
| 10:46:28 | bauzas | songwenping_: sorry I was afk | |
| 10:46:57 | bauzas | songwenping_: honestly, I don't know, it looks like a nvidia driver issue | |
| 10:47:25 | bauzas | gibi: what do you want me to look ? | |
| 10:47:33 | songwenping_ | bauzas:ok, thanks. | |
| 10:58:40 | gibi | bauzas: kashyap: would like to get a specless bp approved | |
| 10:59:00 | kashyap | bauzas: This one - https://blueprints.launchpad.net/nova/+spec/virtio-as-default-display-device | |
| 10:59:03 | bauzas | gibi: ok, let's discuss this during tomorrow's meeting | |
| 10:59:10 | bauzas | kashyap: add it to the meeting agenda | |