| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-19 | |||
| 09:19:15 | kashyap | gibi: Ah, nod | |
| 09:27:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Add nova-live-migration-ceph job https://review.opendev.org/c/openstack/nova/+/768466 | |
| 09:28:46 | lyarwood | finally \o/ | |
| 09:29:15 | lyarwood | still needs some devstack-plugin-ceph and devstack changes to land but it's almost there | |
| 09:29:36 | lyarwood | then it's just the multinode grenade job iirc | |
| 09:33:18 | openstackgerrit | Merged openstack/nova-specs master: Libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116 | |
| 09:36:43 | stephenfin | lyarwood: looking | |
| 10:01:57 | lyarwood | did we agree on a spec review day before the deadline btw? | |
| 10:02:17 | lyarwood | I'm trying to finish the libvirt eph encryption spec today for review but appreciate it's pretty late | |
| 10:20:38 | bauzas | lyarwood: no, we said no during the meeting | |
| 10:21:01 | lyarwood | bauzas: ack thanks, and the deadline is Thursday for specs right? | |
| 10:21:01 | bauzas | but we can continue to look at the specs | |
| 10:21:05 | bauzas | correct | |
| 10:21:17 | lyarwood | bauzas: ack thanks, I'll get this posted today and go from there | |
| 10:21:19 | bauzas | ++ | |
| 10:24:11 | sean-k-mooney | lyarwood: yep just have a meeting in a at half past but ill look at it after. i am familar with the backport and it is something we shoudl do but ill do an actual review after the meeting | |
| 10:24:57 | bauzas | stephenfin: do you want to have +2/+W for all the series before merging https://review.opendev.org/c/openstack/nova/+/758155 or do you prefer to merge each of them | |
| 10:24:59 | bauzas | ? | |
| 10:25:01 | openstackgerrit | Merged openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394 | |
| 10:25:06 | bauzas | honestly, I'm fine with both | |
| 10:25:27 | stephenfin | bauzas: they can go in individually | |
| 10:25:49 | bauzas | sure, I was asking for the series, but at least the first can be merged now | |
| 10:26:58 | stephenfin | yeah, I don't think there'll be a case where patch N has to change because of something in M, if that's what you mean. No dependencies | |
| 10:27:10 | stephenfin | at least not in that way | |
| 10:36:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Liberty database migrations https://review.opendev.org/c/openstack/nova/+/759399 | |
| 10:36:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Mitaka database migrations https://review.opendev.org/c/openstack/nova/+/759400 | |
| 10:36:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Newton database migrations https://review.opendev.org/c/openstack/nova/+/759401 | |
| 10:36:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Ocata database migrations https://review.opendev.org/c/openstack/nova/+/759402 | |
| 10:36:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Pike database migrations https://review.opendev.org/c/openstack/nova/+/759403 | |
| 10:36:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Queens database migrations https://review.opendev.org/c/openstack/nova/+/759404 | |
| 10:36:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Rocky database migrations https://review.opendev.org/c/openstack/nova/+/759405 | |
| 10:36:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Stein database migrations https://review.opendev.org/c/openstack/nova/+/759406 | |
| 10:36:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Add manage.py script https://review.opendev.org/c/openstack/nova/+/771419 | |
| 10:36:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: apidb: Compact Train database migrations https://review.opendev.org/c/openstack/nova/+/771420 | |
| 10:43:22 | bauzas | stephenfin: no worries, for the moment, I don't see any issue with this | |
| 11:04:51 | elod | lyarwood: I've commented on the train backport patch you mentioned yesterday | |
| 11:06:35 | elod | lyarwood: this one: https://review.opendev.org/c/openstack/nova/+/770944 | |
| 11:07:01 | elod | lyarwood: I'd rather not backport those 2 patches :/ | |
| 11:09:08 | lyarwood | elod: ack, I'll add context in the review but tl;dr downstream our OSP 16 release maps to stable/train and we plan on keeping it around until ~2025 https://access.redhat.com/support/policy/updates/openstack/platform/ | |
| 11:09:26 | lyarwood | elod: we've backported a few things downstream and hit issues with pyflakes 1.2.3 not supporting f-strings | |
| 11:09:41 | lyarwood | elod: and assume that in the future we could easily also land things upstream that hit the same issue | |
| 11:10:04 | lyarwood | elod: as this is just a lint tooling bump we thought we'd try to land this in stable/train first before doing anything downstream only | |
| 11:10:29 | lyarwood | maybe that wasn't a true tl;dr but hopefully you see what I was trying to do now ;) | |
| 11:15:42 | elod | lyarwood: in train python 2 is still there, and f-sting is not in python2 | |
| 11:16:20 | elod | I guess you support train with python3 (of course) | |
| 11:16:55 | lyarwood | elod: yeah indeed only on py36 downstream | |
| 11:17:12 | lyarwood | elod: was it a supported runtime in stable/train? | |
| 11:17:29 | lyarwood | ah it was | |
| 11:17:33 | lyarwood | okay then ignore me | |
| 11:17:35 | elod | the job is still there | |
| 11:17:37 | lyarwood | sorry I thought it wasn't | |
| 11:17:39 | lyarwood | https://governance.openstack.org/tc/reference/runtimes/train.html | |
| 11:17:45 | lyarwood | I'll just backport these downstream | |
| 11:17:53 | lyarwood | apologies for the noise | |
| 11:18:38 | elod | lyarwood: ussuri is the 1st one without py2 | |
| 11:18:46 | elod | lyarwood: no problem | |
| 11:19:15 | lyarwood | yup ETOOMANYVERSIONS :) | |
| 11:19:22 | elod | lyarwood: I just don't feel it appropriate upstream :/ | |
| 11:19:45 | lyarwood | yup it isn't if we support py27 still as f-strings are not valid in stable/train as a result | |
| 11:19:58 | lyarwood | I'll just modify the backports downstream to avoid this and move on | |
| 11:20:07 | lyarwood | as it's only f-strings | |
| 11:20:31 | elod | lyarwood: ok, thanks | |
| 11:24:05 | bauzas | stephenfin: man, I don't know how other people do for reviewing your DB changes, but it'll need time for me for looking at all the modifications for https://review.opendev.org/c/openstack/nova/+/758394/4 | |
| 11:25:37 | stephenfin | bauzas: You can put this script in the root of your nova repo, run it on master and then it run after the change is applied, comparing the diff (there shouldn't be one) https://review.opendev.org/c/openstack/nova/+/769796 | |
| 11:25:46 | stephenfin | I've done that a few times now | |
| 11:49:41 | openstackgerrit | Merged openstack/nova stable/ussuri: Use cell targeted context to query BDMs for metadata https://review.opendev.org/c/openstack/nova/+/765748 | |
| 12:46:40 | openstackgerrit | Rico Lin proposed openstack/nova master: add openstack-python3-wallaby-jobs-arm64 job https://review.opendev.org/c/openstack/nova/+/742094 | |
| 12:46:45 | openstackgerrit | Rico Lin proposed openstack/nova master: Fix Unit test for arm64 https://review.opendev.org/c/openstack/nova/+/768202 | |
| 13:05:02 | openstackgerrit | Andrey Volkov proposed openstack/nova master: [WIP] Add conf option for scatter_gather_cells error handling https://review.opendev.org/c/openstack/nova/+/771440 | |
| 13:11:46 | openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: `socket` PCI NUMA-affinity Policy https://review.opendev.org/c/openstack/nova-specs/+/765551 | |
| 13:12:32 | artom | sean-k-mooney, stephenfin, gibi ^^ pretty please :) | |
| 13:26:36 | bauzas | stephenfin: question here https://review.opendev.org/c/openstack/nova/+/758394/4/nova/db/sqlalchemy/migrate_repo/versions/231_add_ephemeral_key_uuid.py#b32 with a procedural -1 | |
| 13:59:19 | sean-k-mooney | artom: im going to refill my coffee then ill take a look | |
| 14:02:29 | gibi | artom: refined my question in https://review.opendev.org/c/openstack/nova-specs/+/765551 | |
| 14:22:28 | stephenfin | bauzas: replied | |
| 14:22:36 | bauzas | ta, looking | |
| 14:22:48 | stephenfin | sean-k-mooney: ta is catching on :P ^ | |
| 14:23:18 | sean-k-mooney | hehe i know bauzas and efried started using it | |
| 14:23:34 | bauzas | ta doesn't mean thanks ? | |
| 14:23:38 | sean-k-mooney | eventually ye will all start spelling in sean speak and we will all be doomed | |
| 14:24:09 | sean-k-mooney | bauzas: kind of yes but its an irish/british thing | |
| 14:24:09 | stephenfin | bauzas: Yup, it does. sean-k-mooney just made a joke about me using it often some time back | |
| 14:24:15 | stephenfin | and I said it would catch on :) | |
| 14:24:19 | bauzas | ok, je peux parler en français sinon | |
| 14:24:24 | bauzas | :p | |
| 14:24:39 | bauzas | anyway, saw your reply | |
| 14:24:54 | bauzas | honestly, I'm not a SQLA expert | |
| 14:25:30 | bauzas | so, yeah, the column is *nullable* but I don't know whether the default value is NULL for the SQL | |
| 14:26:20 | bauzas | anyway, maybe it's just a bikeshed | |
| 14:26:22 | sean-k-mooney | gibi: artom seams to have disconencted but i think the spec is wrong | |
| 14:26:40 | sean-k-mooney | if you have hw_numa_nodes=2 N0 and N2 shoudl be fine | |
| 14:26:55 | stephenfin | yeah, gone since 14:10 | |
| 14:27:04 | stephenfin | probably on breakfast/kid duty | |
| 14:27:06 | gibi | sean-k-mooney: that answer I can accept | |
| 14:27:29 | gibi | sean-k-mooney: if at least one CPU is on the same socket as the allocated PCI | |
| 14:27:31 | sean-k-mooney | if you have hw:numa_nodes=2 and the socket policy provide the pci device come form one of the socket of any of the guest numa nodes its fine | |
| 14:27:59 | sean-k-mooney | hw_pci_numa_affinity_policy=socket should not require that all guest numa nodes come from teh same socket | |
| 14:28:17 | sean-k-mooney | gibi: yep | |
| 14:28:44 | gibi | sean-k-mooney: I agre | |
| 14:29:24 | sean-k-mooney | ill comment that on the spec and i guess you can too but i need to read the rest too. i just started with your comment since i saw your ping to artom | |