| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-11-12 | |||
| 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 | |
| 10:59:34 | bauzas | and please be around tomorrow during the meeting :) | |
| 10:59:52 | kashyap | Yes, I'll be around | |
| 10:59:57 | kashyap | Yea; thx | |
| 11:02:03 | gibi | ack, I will be on the meeting | |
| 11:17:59 | elodilles | bauzas: (sorry for the late reply) thanks, i've created the python-novaclient release: https://review.opendev.org/c/openstack/releases/+/817930 | |
| 11:18:29 | elodilles | and rebased the ussuri-em patch on top of this | |
| 11:43:03 | opendevreview | Takashi Kajinami proposed openstack/nova master: Deprecate [wsgi] secure_proxy_ssl_header https://review.opendev.org/c/openstack/nova/+/817927 | |
| 12:22:32 | sean-k-mooney | gibi: so technially the guest abi is mostly maintained when goign form cirrus to virtio as virtio has a vga compatiable interface that is compatiable with cirrus. it would be a guest visabel change | |
| 12:22:51 | sean-k-mooney | it apparently is not a harmful guest visable change | |
| 12:25:03 | sean-k-mooney | if we are ok with the change in default affectign existing instnace the kasahyap patch is workable if not then i would like him to adress the comments i left in the patch | |
| 12:27:28 | kashyap | Yep, this is a benign case. | |
| 12:27:52 | kashyap | sean-k-mooney[m]: What else comments? I addressed what you (incorrectly) thought was a problem | |
| 12:27:58 | kashyap | I don't see any outstanding questions there. | |
| 12:28:07 | kashyap | Also, recall: | |
| 12:28:27 | sean-k-mooney | you ignored all the other comment i made and you have not | |